Правильно ли я понял принцип одной обязанности в классе?

Question

Дмитрий @EvilsInterrupt

System programming, Reversing Engineering, C++

Правильно ли я понял принцип одной обязанности в классе?

Читая книгу Фриман и Фриман. Паттерны проектирования увидел прикольный принцип: «Класс должен иметь только одну причину для изменения» это стр. 361 в книге.

Мне хочется чтобы кто-нибудь проверил ход моих мыслей и указал на мои ошибки в рассуждениях, чтобы я мог понять насколько правильно понял этот принцип.

Возьмем к примеру простейший класс:

class CustomFormat
{
public:
  Header1 getHeader1();
  std::vector<Header2> getHeader2Array();
  void load();
  void save();  
};

Мне кажется что основной задачей класса является предоставление методов для работы со значениями заголовков. Но он также имеет и дополнительную задачу: класс имеет методы по чтению\записи.

Вопрос: Я правильно понимаю, что если следовать этому принципу, то мы должны методы load\save вынести в другой класс к примеру в какой-нить CustomFormatSerializer?

Меня не интересуют слова вида: «Вы знаете на практике… ». Меня очень волнует именно вопрос следования этому принципу, а не возможные компромиссы для достижения задач на практичеке.

Вопрос задан более трёх лет назад
2853 просмотра

2 комментария

Подписаться 3 Оценить 2 комментария

Дмитрий @EvilsInterrupt Автор вопроса
Спасибо @xSci. Уточню строение класса и с каким форматом он работает. Этот класс работает со интерфейсом стрима, к примеру std::iostream. Ссылка на интерфейс передается в конструкторе класса CustomFormat. Клиент открывает потомка std::iostream, любой из: файловый или память или сеть, не важно и передает этому классу CustomFormat. Строение хидеров простое, к примеру:

struct Header1 { uint32_t magic; uint8t reseverd[10]; };

struct Header2 { uint64_t offset; uint32_t size; };
Написано более трёх лет назад
ixSci @ixSci

Клиент открывает потомка std::iostream, любой из: файловый или память или сеть, не важно и передает этому классу CustomFormat

Надо понимать, что это должно звучать как «Клиент создает объект класса-потомка std::iostream, любой из: файловый или память или сеть, не важно и передает этот объект, объекту класса CustomFormat при создании». Так?

Написано более трёх лет назад

Спасибо @xSci. Уточню строение класса и с каким форматом он работает. Этот класс работает со интерфейсом стрима, к примеру std::iostream. Ссылка на интерфейс передается в конструкторе класса CustomFormat. Клиент открывает потомка std::iostream, любой из: файловый или память или сеть, не важно и передает этому классу CustomFormat. Строение хидеров простое, к примеру:

struct Header1 { uint32_t magic; uint8t reseverd[10]; };

struct Header2 { uint64_t offset; uint32_t size; };
Клиент открывает потомка std::iostream, любой из: файловый или память или сеть, не важно и передает этому классу CustomFormat

Надо понимать, что это должно звучать как «Клиент создает объект класса-потомка std::iostream, любой из: файловый или память или сеть, не важно и передает этот объект, объекту класса CustomFormat при создании». Так?

Answer 1 · 2012-06-28 12:39:01

Насколько я понимаю, это зависит от глубины разбиения задач. Например, стоит задача — сделать простую рисовалку. Вы можете засунуть все в класс типа CPaintBox. Но можно разбить эту большую задачу на несколько более мелких — работу с кистью, загрузку, сохранение, наложение фильтров. Работу с кистью тоже можно разбить — на работу с формой кисти и на работу с ее цветом.

Правильно ли я понял принцип одной обязанности в классе?

Войдите на сайт