0

у меня есть общий класс File, который наследуется от класса FileWrite, который выполняют запись данных в файл, и от класса FileSearch, который выполняет поиск информации по файлу. Скажите есть ли тут нарушение принципа SRP(single-responsibility principle)? введите сюда описание изображения

9
  • 1
    solidworks - это такая программа (для которой часто пишут плагины), не имеющая отношения к SOLID.
    – Chorkov
    Commented 2 дек. 2021 в 7:30
  • Спасибо, буду знать
    – Maxim
    Commented 2 дек. 2021 в 7:39
  • Здесь классы ни за что не отвечают, в просто напиханы несвязанные статические функции, которым явно было бы лучше в пространстве имен. Commented 2 дек. 2021 в 7:44
  • я вот сильно сомневаюсь, что класс файл должен уметь искать в себе. Скорее всего должен быть класс искателя, который получает себе указатель/ссылку на интерфейс ридера (который файл должен реализовать) и уже читает и ищет.
    – KoVadim
    Commented 2 дек. 2021 в 7:49
  • 1
    в java это нормально - наплодить 100500 классов и говорить, что это нормально в ООП (хотя java не является ООП языком:) и пример с spring показывает ещё раз это). Но не будет зацикливаться на bad practice, а также тыкать:)
    – KoVadim
    Commented 2 дек. 2021 в 8:33

1 ответ 1

4
  1. Single responsibility principle - не означает что компонент должен отвечать за одно действие, но означает что должна быть одна (всмысле одна и та же) причина изменения для кода в этом классе. То есть если запись в файл и поиск по файлу меняется по одной и той же причине (потому что сменился формат файла например) - то эти функции должны быть помещены в один компонет, в один и тот же класс. Если меняются по разным причинам - например чтение более низкоуровневое и зависит от файловой системы или носителя, а поиск - не зависит, он зависит только от чтения - тогда можно разбить в разные компоненты.
  2. У вас сделанно наследование немного не логично получается что может быть класс умеющий искать по файлу, но не читать. Но поиск подразумевает чтение файла.
2
  • 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); };
    – Maxim
    Commented 2 дек. 2021 в 8:14
  • Примерно да. Если ваш этот класс сильно разрастется и его будет сложно поддерживать, тогда его можно будет разбить, только функции чтения-записи будут в родительском а разный поиск, зависящий от них, в дочернем Commented 2 дек. 2021 в 8:20

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.