Данный код нарушет первый принцип SOLID (Единая ответственность)?

Question

[HCEL]

async function updateUser (data) {
     const { password }  = data;
     
     if (password) {
          const hash = await Password.getHash(password);
         data.passwordHash = hash;
     }

    await UserRepo.update(data);
}

Comments

[Dmitry Bay]

password проверку нужно не в функции проводить.

[HCEL]

Dmitry Bay, то есть реализовать и вынести в отдельную бизнес логику, а эта функция будет только уметь обновлять уже заготовленные данные, верно?

[vladislav3101]

HCEL, да

[mkone112]

SOLID обычно применяется к ооп, я не вижу здесь ооп.

[Дмитрий Беляев]

кто Вам сказал такую глупость?

[HCEL]

ну сделаю так

class User {
  update() {}
}

Особо ничего не изменилось, логика кода внутри метода будет точно таким же. Даже если я буду принимать данные/зависимости в конструкторе, мой метод остается неизменным, разве что в конструкторе я опять же буду вызывать модуль генераций хеша, тогда вопрос будет немного задан по другому: - "А правильно ли что в экземпляре класса я дергаю модуль генераций пароля?".
Поэтому мой вопрос был задан в простом формате вынести модуль генераций хеша пароля (из куска кода, который взаимодействует с пользователем) или нет aka Должен ли этот метод тогда только уметь обновлять данные тем самым получив в параметрах этот hash, а не самому создавать его.
У тебя есть другой ответ?

[mkone112]

Дмитрий Беляев, тут две глупости. Какую вы имеете в виду?

[HCEL]

mkone112, аргуменитруй пожалуйста адекватно без пассивной агрессий. Есть ответ отвечай, если нет, то пожалуйста проигнорируй.

[mkone112]

HCEL, пассивная агрессия, рили?

Есть ответ отвечай, если нет, то пожалуйста проигнорируй.

Я это уже сделал, не стоит благодарности.
P.S. Я сам разберусь отвечать мне или нет.

[HCEL]

mkone112, ну значит ты слит раз не смог с 3-ей попытки аргументировать свои слова и наверное пишешь код в таком стиле: раз код не по ооп значит единая ответственность не распространяется, гений мыслей.

[mkone112]

HCEL,

раз код не по ооп значит единая ответственность не распространяется, гений мыслей.

Ты бы хоть читал ответы которые комментируешь, в нем нет ни слова про единую ответственность.

смог с 3-ей попытки

Тут не было не единой попытки, ответ дан - дальше сам.

ну значит ты слит

Что за детский сад...

[HCEL]

mkone112, да пофиг уже можешь не отвечать на мои байты, я чекнул твой профиль с вопросами питона :D не буду продолжать диалог.

[mkone112]

HCEL, было бы здорово, если бы ты смог ответить хоть на один из этих вопросов, но я на это не рассчитываю.

[vladislav3101]

Бред

[mkone112]

vladislav3101, что конкретно?

[Модератор]

Необходимо прекратить флуд и флейм. Немедленно. Не надо продолжать его в других вопросах.

[vladislav3101]

mkone112,

SOLID обычно применяется к ооп, я не вижу здесь ооп.

[Alexandroppolus]

mkone112,

SOLID обычно применяется к ооп

Только третий принцип. Остальные можно применять к другим подходам.

[mkone112]

Alexandroppolus, конечно. Но это уже не SOLID.

[mkone112]

vladislav3101, это два утверждения, с каким конкретно ты не согласен?

[vladislav3101]

mkone112, ты очень душный. Судя по комментариям и ответам других людей тебе все уже разложили по полочкам с довольно приемлемым примером, но ты всё отстаиваешь свою позицию как "клоун" без каких либо аргументов, видно что поддержка явно не на твоей стороне. В принципе таких как ты наверное лучше сразу кидать в игнор. В диалоге с такими обычно ничего не добьешься и не объяснишь.
Я ещё раз повторюсь, твоя цитата

SOLID обычно применяется к ооп, я не вижу здесь ооп.

к вопросу ТС звучит как бред не в тему или как сказал Дмитрий

глупость

[Модератор]

vladislav3101, у вас есть неиллюзорный шанс отправиться на некоторое время в readonly вместе с остальными персонажами этого безобразия.

ЭТО ПОСЛЕДНЕЕ «КИТАЙСКОЕ» ПРЕДУПРЕЖДЕНИЕ.

Answer 1

[Alexandroppolus]

На мой взгляд, небольшое нарушение в том, что модуль самостоятельно резолвит свою зависимость от UserRepo, а не получает её через DI. От Password, наверно, тоже, но это вроде как утилита и можно не заморачиваться. А вот UserRepo я бы заинжектил.