АП
юзать глобальные зависимости
Size: a a a
2020 June 06
АП
вот представьте у вас в проекте 100500 пакетов локальный и там везде блин logger
АП
потом как вы тестами покрывать без хаков будете?
AK
Без копания сложно разобраться с первого взгляда вообще
Ну да, я пока смотрю, я не очень даже могу поять домен. Т.е. предметную область. Там речь о каких то спец идентификаторах и тп
АП
все зависимости должны быть прокинуты
AK
вот представьте у вас в проекте 100500 пакетов локальный и там везде блин logger
Не, я конкретно вырвал из контекста, вот какой спор: Зачем повторять 3 строки отдельно, если можно объединить эти 3 строки в одну функцию и просто пробрасывать туда параметры каждый раз, тем самым как бы сделав по DRY.
АП
ну ничего что тебе текст там разный?
АП
и тебе придется либо хранить этот текст внутри функции и плодить функции либо прокидывать
AK
На мой взгляд так делать именно там нельзя, потому что логирование, возврат ошибки и инкремент метрики, это совершенно разные операции и объединение их в одну функцию нарушает SRP
АП
никакой DRY там не выйдет
АП
поэтому три вызова - это норма
AK
поэтому три вызова - это норма
https://youtu.be/8-U3xK7kBPA?t=2296 - вот таймкод. Ну т.е. даже в таком случае существуют разные мнения.
АП
да что-то меня на 3 минуты хватило
АП
где они начали мямлить почему собственно интерфейсы передавать в конструктор
АП
хотя там ответ очень простой "как ты мокать то будешь без интерфейсов"
AK
хотя там ответ очень простой "как ты мокать то будешь без интерфейсов"
На это тоже есть разные мнения, например что мокать не нужно.
АП
а тестить как будешь?
АП
это не мнение - это бред какой то
М
Не, я конкретно вырвал из контекста, вот какой спор: Зачем повторять 3 строки отдельно, если можно объединить эти 3 строки в одну функцию и просто пробрасывать туда параметры каждый раз, тем самым как бы сделав по DRY.
Есть такая тема, что если по какой-то причине у вас есть одинаковый код, то это ещё не значит что это один и тот же код.
Грубо говоря нужно поставить вопрос: "если я буду его менять, то мне это будет нужно сделать везде или только в одном месте?"
Грубо говоря нужно поставить вопрос: "если я буду его менять, то мне это будет нужно сделать везде или только в одном месте?"