Правильно ли применён принцип единичной ответственности?

Question

[zlodiak]

Скажите пожалуйста, правильно ли я исправил скрипт при помощи принципа единственной ответственности?

Вот пример, в котором этот принцип не соблюдается.
Вот пример, в котором этот принцип соблюдается.

Сомнения возникают по поводу того, что во втором примере, несмотря на использование dependency injection, всё равно класс Summator выполняет два отличных действия:
1. вычисление суммы,
2. запись в лог

Не правильнее было бы запись в лог производить в клиентском коде так?

Comments

[Максим Федоров]

2. запись в лог

Это сквозная функциональность, вполне может быть...

Если сильно не нравится, то можете декоратор сделать... То есть сделать обертку, в которой будет логирование происходить

[zlodiak]

Максим Федоров, я не совсем понимаю что такое сквозная функциональность. Проверка авторизации например это сквозная функциональность? Её логика находится в одном месте, но проверки на основе этой логики осуществляются при заходе пользователя на любую из 10 страничек... я правильно понимаю?

Answer 1

[Антон Спирин]

По принципу единственной ответственности метод getSum должен лишь возвращать сумму. Никаких дополнительных действий в нем происходить не должно.

Насчет последнего примера трудно что-либо посоветовать, так как это имеет мало общего с реальным клиентским кодом. Сам Summator лишняя неудобная сущность которую можно смело заменить на:

const sum = arr => arr.reduce((acc, i) => acc + i);

const calculateSquaresSum = arr => sum(arr.map(i => i.calcArea()));

Для логгирования в dev режиме обычно используют декораторы или middleware.