Форум программистов, компьютерный форум CyberForum.ru

C++

Войти
Регистрация
Восстановить пароль
 
 
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,250
Записей в блоге: 1
Завершенные тесты: 1
#1

Немного философии ООП - C++

17.09.2016, 07:15. Просмотров 973. Ответов 24
Метки нет (Все метки)

На днях на работе возник холивар с коллегами, к правильному решению пришли только на следующий день, почитав Мейерса и Саттера (один из участников холивара уже знал о правильном решении, но сторонников неправильного было большинство).
Рекомендаций от упомянутых гуру пока не говорю, предлагаю поразмышлять)

Итак - есть проект, в проекте используется ООП, скатываться до С считается мовитоном и не приветствуется.
Есть структура сеттингов
C++
1
2
3
4
5
6
struct Settings
{
    std::string param1;
    int param2;
    // etc
};
понадобилось сделать сериализацию/десериализацию этой структуры. По сути нужны 2 функции serialize(Settings)/deserialize(Settings). Но 2 функции - это С, обернуть их в класс и сделать статичными - это вроде как ООП, но с запашком.
C++
1
2
3
4
5
6
7
class Foo
{
public:
    static PackedObject serialize(Settings);
    
    static Settings deserialize(PackedObject);
};
По сути этот класс является неймспейсом и если из него реально сделать неймспейс (и это правильно), то даже остальной код трогать не придется. т.к. синтаксис использования не изменится. Но это сведет нас к тем же 2м сишным функциям, которых мы хотели избежать.

Вопрос - как сделать правильно, чтоб было хорошее ООП?

Не по теме:

rikimaru2013, хотел интересных вопросов?

Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
17.09.2016, 07:15     Немного философии ООП
Посмотрите здесь:

Обработка всех исключений TIdHTTP + немного ООП - C++ Builder
Приветствую. У меня возникла потребность в обработке всех исключений компонента TIdHTTP для более детального отчёта работы программы. А...

немного не обычно - C++
занявшись програмированием на C плавно добрался до работы с графикой, но тут меня ждало разочарование, мой кампилятор (dev-C++) обнаружил...

Немного теории - C++
Доброго дня, форумчане! Пишу курсач на тему Системы программирования. Препод сказал, что у меня нет информации о видах систем...

Немного о списках - C++
Помогите разобраться со списками(хотя бы односвязными) Подскажите пример простого кода, где реализован список как можно проще. Вот пока...

Немного не получается - C++
#include "stdafx.h" #include "iostream" #include <locale.h> using namespace std; class terms { public: double sum, razn, pow,...

немного поменять, - C++
#include <iostream> #include <cstdlib> #include <fstream> using namespace std; struct LIST { int info; LIST *next; ...

немного усовершенствовать... - C++
B]как сделать так чтобы пробег автобусов генерировался randomize а не вводился с клавиатуры...?...все время получаются какие то...

После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,250
Записей в блоге: 1
Завершенные тесты: 1
18.09.2016, 19:09  [ТС]     Немного философии ООП #16
castaway, мне на чисто интуитивном уровне кажется, что структура (любая) не должна уметь менять свое представление (упаковываться), этим должен заматься кто-то со стороны, кто знает о наличии этой структуры и формата представления (то, во что сериализуется структура). Сама же структура не должна ничего знать о, скажем, классе DataTransferObject, ей эти знания ни к чему, а без этих знаний она не может упаковывать сама себя в DataTransferObject.
castaway
Эксперт С++
4881 / 3017 / 370
Регистрация: 10.11.2010
Сообщений: 11,076
Записей в блоге: 10
Завершенные тесты: 1
18.09.2016, 19:17     Немного философии ООП #17
Kastaneda, так какой же вариант лучший?
IGPIGP
Комп_Оратор)
Эксперт по математике/физике
6444 / 3083 / 306
Регистрация: 04.12.2011
Сообщений: 8,495
Записей в блоге: 4
18.09.2016, 19:21     Немного философии ООП #18
Цитата Сообщение от Kastaneda Посмотреть сообщение
мне на чисто интуитивном уровне кажется, что структура (любая) не должна уметь менять свое представление (упаковываться), этим должен заматься кто-то со стороны,
Это значит, что кто-то со стороны должен знать о данной структуре. Я не помню точно, но кажется создатель работая в Центре компьютерных исследований фирмы Bell Labs решал задачи распределения вычислений между различными машинами и различными OS на них. Так как легче, научить внешние ресурсы работе со структурой или попросить разрабов поддержать структуру в разных средах? Конечно, это не о данной структуре, хотя из вопроса не видно, что она примитивна. Иначе вопрос о чём?
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,250
Записей в блоге: 1
Завершенные тесты: 1
18.09.2016, 19:23  [ТС]     Немного философии ООП #19
Цитата Сообщение от castaway Посмотреть сообщение
Kastaneda, так какой же вариант лучший?
Слово "лучший" тут наверное тоже не совсем уместно. Можно рассуждать так (поэтапно):
1. Мы решили, что Settings не должен уметь сам себя сериализовывать/десериализовывать
2. Значит выносим эти функции в отдельный класс
3. Эти функции не меняют состояние объекта класса, значит должны быть статичными
4. Класс, кроме этих двух статичных функций, больше ничего не содержит, а значит семантически это получается неймспейс
5. Делаем из класса неймспейс, получаем 2 сишные функции

Получаем 2 функции, которые являются интерфейсными (как пишут умные люди) для структуры Settings, при этом семантика Settings не нарушена.

Добавлено через 56 секунд
Цитата Сообщение от IGPIGP Посмотреть сообщение
Это значит, что кто-то со стороны должен знать о данной структуре.
да, и о выходном формате. Этого "кого-то" можно считать "специально обученным для этого дела" )
castaway
Эксперт С++
4881 / 3017 / 370
Регистрация: 10.11.2010
Сообщений: 11,076
Записей в блоге: 10
Завершенные тесты: 1
18.09.2016, 19:35     Немного философии ООП #20
Цитата Сообщение от Kastaneda Посмотреть сообщение
Получаем 2 функции, которые являются интерфейсными (как пишут умные люди) для структуры Settings, при этом семантика Settings не нарушена.
но ведь:
Цитата Сообщение от Kastaneda Посмотреть сообщение
скатываться до С считается мовитоном и не приветствуется.
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,250
Записей в блоге: 1
Завершенные тесты: 1
18.09.2016, 19:42  [ТС]     Немного философии ООП #21
Цитата Сообщение от castaway Посмотреть сообщение
но ведь:
это были изначальные условия, от которых мы отталкивались. Потом (прочитав Мейерса), пришли к тому, что я написал в предыдущем посте. Собственно это решение сразу было на виду, но не хотели его принимать, т.к. "не ООП же")
hoggy
6460 / 2658 / 460
Регистрация: 15.11.2014
Сообщений: 5,850
Завершенные тесты: 1
20.09.2016, 00:08     Немного философии ООП #22
boost.serialization/cereal
pav1uxa
1788 / 1628 / 622
Регистрация: 23.01.2014
Сообщений: 5,906
Завершенные тесты: 1
21.09.2016, 22:28     Немного философии ООП #23
Отклонюсь в сторону.

Почему 2 функции в неймспейсе это "сишный подход" и моветон? STL переполнена такими функциями. Или я чего-то не так понимаю?

Страуструп русским по белому писал, что моветон, это как раз когда методами класса (пусть и статическими) делают функции, которым не нужен доступ к приватным членам класса.

Все к тому и сводится: нужен доступ к приватным членам - статик, не нужен - отдельные функции.

ООП ради ООП, как уже сказали, это ведь в разы хуже чем "сишный подход"
rikimaru2013
C++ Game Dev
2419 / 1113 / 240
Регистрация: 30.11.2013
Сообщений: 3,661
22.09.2016, 18:14     Немного философии ООП #24
Я согласен с castaway - Settings хоть и структура, должна в первую очередь знать, что ей нужно сериализовать. Пусть она не имеет алгоритмы, но должна предоставить метод для тех кто умеет. И посему 22 пост, как по мне, лучше всего это делает. Ведь в идеале структура в одном файле, и добавив поле не хотелось бы искать, где её сериализатор. В СИ такого вопроса не возникало, потому что структура писалась как sizeof(), а тут нужен ООП подход: объект знает всю private кухню и подскажет алгоритму сохранению, что ему делать
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
26.09.2016, 12:22     Немного философии ООП
Еще ссылки по теме:

Немного подправить. - C++
Имеется код программы. Он рабочий, но почему то при вводе данных в программу, в документе database.txt выводятся какие то символы, но ни...

Немного о sizeof - C++
Не могу найти. Вот такая форма оператора, с несколькими параметрами: sizeof (323, 43241, 555) Что она возвращает?

Подправьте немного... - C++
Суть задачи: найти сумму ряда с точностью 0.001 общий член n!/3^n^n вот написал какой то бред #include "stdafx.h" #include...

Немного шаблонов - C++
Добрый день форумчане, у меня в порядке бреда родилась такая идея class B//базовый класс, содержит указатель на void { public: ...

подправить немного - C++
#include <iostream> using namespace std; int main() { int i, j, N, M; double *a,*b,*c; cout«"N = "; cin»N; cout«"M = ";...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
AlexVRud
442 / 152 / 38
Регистрация: 04.07.2014
Сообщений: 430
26.09.2016, 12:22     Немного философии ООП #25
1. Settings - стоит определить как class, т.к. он содержит не Pod-тип внутри себя, и для него будут сгенерированы конструкторы/деструкторы для управления памятью

2. Результат сериализации - нечто иное как представление класса, следовательно, она не должен фигурировать в нём.

3. Если хочешь ООП, то представь сериализацию как конфигурируемый процесс, которым кто-то управляет, тогда можно и так сделать:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
#include <iostream>
#include <sstream>
#include <string>
#include <utility>
 
class Settings {
  std::string param1_;
  int param2_;
 
public:
  Settings()
      : param1_("default")
      , param2_(0) {};
 
  template <class T>
  Settings(T &&param1, int param2)
      : param1_(std::forward<std::string>(param1))
      , param2_(param2)
  {
  }
 
  std::string getParam1() const
  {
    return param1_;
  };
  int getParam2() const
  {
    return param2_;
  };
};
 
class JsonStringPacker {
  bool need_new_line_;
  bool need_tab_to_spaces_;
  uint8_t tab_width_;
 
public:
  JsonStringPacker(bool need_new_line = true, bool need_tab_to_spaces = true, uint8_t tab_width = 4)
      : need_new_line_(need_new_line)
      , need_tab_to_spaces_(need_tab_to_spaces)
      , tab_width_(tab_width) {};
 
  std::string pack(const Settings &settings)
  {
    std::stringstream ss;
    ss << "{";
    if (need_new_line_) {
      ss << "\n" << (need_tab_to_spaces_ ? std::string(tab_width_, ' ') : std::string("\t"));
    }
    ss << "'param1': '" << settings.getParam1() << "',";
    if (need_new_line_) {
      ss << "\n" << (need_tab_to_spaces_ ? std::string(tab_width_, ' ') : std::string("\t"));
    } else {
      ss << " ";
    }
    ss << "'param2': " << settings.getParam2();
    if (need_new_line_) {
      ss << "\n}\n";
    } else {
      ss << "}";
    }
    return ss.str();
  };
};
 
int main()
{
  Settings settings = Settings("Hello world", 13);
  JsonStringPacker packer1;
  JsonStringPacker packer2 = JsonStringPacker(false);
  std::cout << packer1.pack(settings);
  std::cout << packer2.pack(settings) << std::endl;
  std::cout << JsonStringPacker(true, false).pack(settings);
  std::cout << JsonStringPacker(true, true, 2).pack(settings);
  return 0;
}
Результат:

JSON
1
2
3
4
5
6
7
8
9
10
11
12
13
{
    'param1': 'Hello world',
    'param2': 13
}
{'param1': 'Hello world', 'param2': 13}
{
        'param1': 'Hello world',
        'param2': 13
}
{
  'param1': 'Hello world',
  'param2': 13
}
Добавлено через 6 минут
Цитата Сообщение от rikimaru2013 Посмотреть сообщение
Settings хоть и структура, должна в первую очередь знать, что ей нужно сериализовать
Если исходить из этого, то должно знать "представление данных" и не сами "данные", что и как надо сериализовать, т.е. PacketObject (которые тогда правильнее назвать PacketSettings )
Yandex
Объявления
26.09.2016, 12:22     Немного философии ООП
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru