Что такое избыточная ответственность метода или класса ("слишком много" в одном методе/классе)?

Что такое избыточная ответственность?

Избыточная ответственность (или нарушение Single Responsibility Principle) возникает, когда один класс или метод берет на себя выполнение нескольких логически не связанных между собой задач. Например, класс может одновременно обрабатывать данные, сохранять их в базу и отправлять email-уведомления. Это делает код жестким, трудным для понимания и изменения.

Пример проблемы

Рассмотрим класс, который отвечает и за расчет зарплаты, и за отправку отчета:

class EmployeeService {
  calculateSalary(employee) {
    // сложные расчеты
  }
  sendReport() {
    // отправка email
  }
  saveToDatabase() {
    // SQL запросы
  }
}

Здесь три разные ответственности смешаны в одном классе. Если изменится формат отчета, придется менять класс, который также отвечает за расчет зарплаты — это увеличивает риск ошибок.

Как исправить?

Разделите класс на несколько, каждый со своей четкой задачей:

class SalaryCalculator {
  calculate(employee) { /* ... */ }
}
class ReportSender {
  send(report) { /* ... */ }
}
class EmployeeRepository {
  save(employee) { /* ... */ }
}

Вывод

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