у меня есть общий класс File, который наследуется от класса FileWrite, который выполняют запись данных в файл, и от класса FileSearch, который выполняет поиск информации по файлу. Скажите есть ли тут нарушение принципа SRP(single-responsibility principle)?
-
1solidworks - это такая программа (для которой часто пишут плагины), не имеющая отношения к SOLID.– ChorkovCommented 2 дек. 2021 в 7:30
-
Спасибо, буду знать– MaximCommented 2 дек. 2021 в 7:39
-
Здесь классы ни за что не отвечают, в просто напиханы несвязанные статические функции, которым явно было бы лучше в пространстве имен.– user7860670Commented 2 дек. 2021 в 7:44
-
я вот сильно сомневаюсь, что класс файл должен уметь искать в себе. Скорее всего должен быть класс искателя, который получает себе указатель/ссылку на интерфейс ридера (который файл должен реализовать) и уже читает и ищет.– KoVadimCommented 2 дек. 2021 в 7:49
-
1в java это нормально - наплодить 100500 классов и говорить, что это нормально в ООП (хотя java не является ООП языком:) и пример с spring показывает ещё раз это). Но не будет зацикливаться на bad practice, а также тыкать:)– KoVadimCommented 2 дек. 2021 в 8:33
|
Показать ещё 4 комментария
1 ответ
- Single responsibility principle - не означает что компонент должен отвечать за одно действие, но означает что должна быть одна (всмысле одна и та же) причина изменения для кода в этом классе. То есть если запись в файл и поиск по файлу меняется по одной и той же причине (потому что сменился формат файла например) - то эти функции должны быть помещены в один компонет, в один и тот же класс. Если меняются по разным причинам - например чтение более низкоуровневое и зависит от файловой системы или носителя, а поиск - не зависит, он зависит только от чтения - тогда можно разбить в разные компоненты.
- У вас сделанно наследование немного не логично получается что может быть класс умеющий искать по файлу, но не читать. Но поиск подразумевает чтение файла.
-
Cпасибо за ответ. Хотел бы уточнить, если я Вас правильно понял, то должно получиться примерно вот так? class FileAction { public: bool findOne(const string& fileName, TypeMethod Object::* method, any_type value, Object& object); void findAll(const string& nameFile, vector<T>& savedObject); void reWrite(const string& fileName, vector<T>& array); void addData(const string& fileName, T& object); void getUnicSeed(const string& nameFile, T& obj); bool is_file_exist(const string& fileName); };– MaximCommented 2 дек. 2021 в 8:14
-
Примерно да. Если ваш этот класс сильно разрастется и его будет сложно поддерживать, тогда его можно будет разбить, только функции чтения-записи будут в родительском а разный поиск, зависящий от них, в дочернем Commented 2 дек. 2021 в 8:20