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

C++

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

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

17.09.2016, 07:15. Просмотров 946. Ответов 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     Немного философии ООП
Посмотрите здесь:

Немного подправить. C++
немного поменять, C++
C++ Исправить немного
немного усовершенствовать... C++
Подправьте немного... C++
Немного теории C++
C++ Немного не получается
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kastaneda
Форумчанин
Эксперт С++
4470 / 2832 / 224
Регистрация: 12.12.2009
Сообщений: 7,202
Записей в блоге: 1
Завершенные тесты: 1
18.09.2016, 19:42  [ТС]     Немного философии ООП #21
Цитата Сообщение от castaway Посмотреть сообщение
но ведь:
это были изначальные условия, от которых мы отталкивались. Потом (прочитав Мейерса), пришли к тому, что я написал в предыдущем посте. Собственно это решение сразу было на виду, но не хотели его принимать, т.к. "не ООП же")
hoggy
6168 / 2534 / 444
Регистрация: 15.11.2014
Сообщений: 5,611
Завершенные тесты: 1
20.09.2016, 00:08     Немного философии ООП #22
boost.serialization/cereal
pav1uxa
1779 / 1619 / 617
Регистрация: 23.01.2014
Сообщений: 5,850
Завершенные тесты: 1
21.09.2016, 22:28     Немного философии ООП #23
Отклонюсь в сторону.

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

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

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

ООП ради ООП, как уже сказали, это ведь в разы хуже чем "сишный подход"
rikimaru2013
C++ Game Dev
2380 / 1074 / 235
Регистрация: 30.11.2013
Сообщений: 3,571
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     Немного философии ООП
Еще ссылки по теме:

Обработка всех исключений TIdHTTP + немного ООП C++ Builder
C++ немного об указателях
подправить немного C++
Немного шаблонов C++
Немного о sizeof C++

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

Или воспользуйтесь поиском по форуму:
AlexVRud
442 / 152 / 38
Регистрация: 04.07.2014
Сообщений: 426
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