Как можно заметить, что нарушен принцип единственной ответственности (Single Responsibility)?

Question

Класс нарушает SRP, если он выполняет несколько несвязанных задач. Явные признаки: класс имеет слишком много методов, которые делают разное; часто меняется по разным причинам; его зависимости относятся к разным предметным областям; или его название неконкретное (например, Manager, Processor), что указывает на смешение обязанностей.

YeaHub · Accepted Answer

Принцип единственной ответственности (SRP) гласит, что у класса должна быть только одна причина для изменения. Нарушение этого принципа приводит к появлению "божественных объектов" (God Object), которые сложно поддерживать, тестировать и понимать.Ключевые признаки нарушения SRP:Класс имеет множество несвязанных публичных методов.Если из класса можно "выделить" несколько независимых групп методов (например, методы для работы с базой данных, для логирования и для отправки email), это явный сигнал о нарушении SRP.Пример: Класс Order, который не только управляет данными заказа, но и сохраняет себя в БД, отправляет уведомление клиенту и генерирует PDF-чек.Класс часто изменяется по разным причинам.Если при изменении бизнес-правил, связанных с одной задачей, приходится менять класс, который также отвечает и за другие задачи, это нарушает SRP. Изменения в логике отправки email не должны затрагивать код, отвечающий за расчёт скидок.Класс имеет слишком много зависимостей.Большое количество внедряемых зависимостей (в конструкторе или методах) часто указывает на то, что класс пытается делать слишком много. Например, зависимость от репозитория, сервиса отправки почты, логгера и шаблонизатора в одном классе."Размытое" или обобщённое название класса.Имена вроде ServiceManager, ProductHelper или Utils часто скрывают классы, которые взяли на себя слишком много обязанностей. Чёткое название (например, EmailNotificationService) помогает контролировать фокус класса.Пример до и после рефакторинга:// Нарушение SRP
public class OrderProcessor {
    public void Process(Order order) {
        // 1. Валидация заказа
        // 2. Расчёт стоимости
        // 3. Сохранение в базу данных
        // 4. Отправка подтверждения по email
        // 5. Запись в лог
    }
}

// Соблюдение SRP
public class OrderValidator { ... } // Ответственность: валидация
public class PriceCalculator { ... } // Ответственность: расчёт цены
public class OrderRepository { ... } // Ответственность: сохранение
public class EmailService { ... }    // Ответственность: отправка email
public class Logger { ... }          // Ответственность: логирование

// Класс-координатор, который теперь зависит от абстракций
public class OrderProcessor {
    public OrderProcessor(IOrderValidator validator, IPriceCalculator calculator, ...) { ... }
    public void Process(Order order) {
        // Последовательно вызывает методы сервисов
    }
}Вывод:Нарушение SRP делает код хрупким и сложным для изменения. Рефакторинг заключается в разделении большого класса на несколько маленьких, каждый из которых отвечает за одну четко определённую задачу.