SOLID Единая ответственность — стоит ли мне в данном случае написать отдельный класс?

Question

[gomerIT]

Задача заключается в том, чтобы подсчитать сумму всех двух значений ключей a и b в объекте, которые находятся в массиве, но так же иметь возможность подсчитывать сумму только для одно объекта.

const data = [{"id": 1, "a": 99, "b": 100}, {"id": 2, "a": 1 "b": 2}];
const oneEl = {"id": 3, "a": 2, "b": 10};

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

class Calc {
  calcAmount (obj) {
    const {a, b} = obj;
    return a + b;
  }

 calcAmountOfArray(data) {
   let sum = 0;
   data.forEach((item) => {
     sum += this.calcAmount(item);
   });

  return sum;
 }
}

Вроде все логично, но разве этим я не нарушил единую ответственность? Допустим появится новый ключ "c" и придется подсчитывать и его сумму. Интуиция мне подсказывает создать отдельный класс для метода calcAmount, но стоит ли так делать?

Comments

[Сергей delphinpro]

вариант просто перебрать все ключи (исключив id) вы видимо не рассматриваете

[gomerIT]

Сергей delphinpro, с одной стороны да вот оно решение, но кто знает какие еще ключи могут появится, тогда и это решение поломается.

[Сергей delphinpro]

gomerIT, согласен. но ваш вариант вообще плолхо выглядит. Это хорошо, если у вас ключей 2-5 штук. А если больше? Не перечислять же их все.
Тут как вариант использовать массив ключей, участвующих в калькуляции суммы. Тогда его можно подредактировать в одном месте.

[gomerIT]

Сергей delphinpro, да возможно пример плохой, но тут вопрос больше по - "А правильно ли я делаю придерживаясь принципу единой ответсвтенности".
А насчет рещения: cейчас подумав над вашим комментарием, то получать data лучше с какого нибудь репозитория с нужными данными и пробрасывать в этот класс и перебирать уже все ключи которые пришли.

Answer 1

[pin2t]

Ну начнем с того, что класс вам тут вообще не нужен. Потому что в классе Calc нет никаких данных, только методы. Вот функции вам и нужны, а не класс.
Не надо сильно выдумывать и придумывать что-то связанное с этими принципами, они очень абстрактны. Даже сам автор SOLID сто раз переобувался что он имел в виду под этим S.
Простой функции

arrayAmount(data) {
   let sum = 0;
   data.forEach((item) => {
     const {a, b} = item;
     sum += a + b;
   });

  return sum;
 }

будет достаточно

Answer 2

[Aetae]

calcAmount по SOLID должен быть методом у объекта с этими ключами.
Т.е. допустим с сервера пришла

const data = [{"id": 1, "a": 99, "b": 100}, {"id": 2, "a": 1 "b": 2}];

Перед тем как с ней что-то делать, ты преобразуешь её в типизированную структуру:

interface Calculable {
  calcAmount(): number;
}

class CalculableAB implements Calculable {
  constructor(obj) {
    Object.assign(this, obj);
  }
  calcAmount() {
    const {a, b} = this;
    return a + b;
  }
}

struct = data.map(obj => new CalculableAB(obj));

Ну и дальше при работе с любым Calculable ты просто вызываешь у него calcAmount(). Как оно там реализовано - тебя не волнует.

Это конечно хардкорное ООП и в большинстве мелких случаев такого не надо, но суть думаю ясна.)

Comments

[gomerIT]

Точно - "Открытость/Закрытость".