ПГ
А в вашей какие то ограничения для этого?
Ограниченй нет, просто у вас запросы харнятся типа в миграциях)
Size: a a a
2021 March 18
SB
Ну в одном месте норм) зато где репо вызывается - чисто
давайте все функции складывать а один класс чтобы не размазывать их по проекту)
ПГ
наоборот не размазано по всему проекту а сгруппировано там где оно надо
Сгруппированов репо....
КГ
Ограниченй нет, просто у вас запросы харнятся типа в миграциях)
Ну так у меня они тоже выполняются в миграциях
ПГ
давайте все функции складывать а один класс чтобы не размазывать их по проекту)
Ну реп то несколько
ПГ
в том месте где они делаются например
ну или в инвокабл функциях
ну или в инвокабл функциях
Момент в том, что инкокабл функция - все равно обертка
КГ
Ну реп то несколько
Всё равно SRP по бороде идёт, складывать всё в кучу плохо
ПГ
Всё равно SRP по бороде идёт, складывать всё в кучу плохо
Ну согласен, можно и еще репо размбить на 1 запрос 1 класс. Я больше к тому, что все равно это инкапуслироват в классе, а не QB на месте строится
SB
Момент в том, что инкокабл функция - все равно обертка
ну жто был второй варинт
а так где испоьзуется там и пишется
а зачем вам надо запрос выносить далеко от места использования не понятно
зачем увеличивать количество связей в системе
а так где испоьзуется там и пишется
а зачем вам надо запрос выносить далеко от места использования не понятно
зачем увеличивать количество связей в системе
КГ
Ну согласен, можно и еще репо размбить на 1 запрос 1 класс. Я больше к тому, что все равно это инкапуслироват в классе, а не QB на месте строится
Наверное стоит исходить из того, как ты это тестами покрываешь.
Мокнуть маленькую обёртку проще чем большую репу.
Если тесты не пишем, то проблемы не так остро стоят, пока)
Мокнуть маленькую обёртку проще чем большую репу.
Если тесты не пишем, то проблемы не так остро стоят, пока)
ПГ
ну жто был второй варинт
а так где испоьзуется там и пишется
а зачем вам надо запрос выносить далеко от места использования не понятно
зачем увеличивать количество связей в системе
а так где испоьзуется там и пишется
а зачем вам надо запрос выносить далеко от места использования не понятно
зачем увеличивать количество связей в системе
Ну далекои не надо. У меня больше был вопрос "как вообще без оберток над запросом" . 1 обертка на 10 запросов или 1 обертка на 1 зарос - у меня вопрос не стоит - вкусовщина и размах проекта.
SB
Ну далекои не надо. У меня больше был вопрос "как вообще без оберток над запросом" . 1 обертка на 10 запросов или 1 обертка на 1 зарос - у меня вопрос не стоит - вкусовщина и размах проекта.
А в чем приемущества обертки?
ПГ
Наверное стоит исходить из того, как ты это тестами покрываешь.
Мокнуть маленькую обёртку проще чем большую репу.
Если тесты не пишем, то проблемы не так остро стоят, пока)
Мокнуть маленькую обёртку проще чем большую репу.
Если тесты не пишем, то проблемы не так остро стоят, пока)
Какая разница мокать большую или маленькую если мокать один метод в итоге
ПГ
А в чем приемущества обертки?
Нейминг и сокрытие реализации?
КГ
Ну далекои не надо. У меня больше был вопрос "как вообще без оберток над запросом" . 1 обертка на 10 запросов или 1 обертка на 1 зарос - у меня вопрос не стоит - вкусовщина и размах проекта.
Можно вообще без обёрток. У нас такие места тестируются передачей реального ManagerRegistry. То есть не юнит тест получается, а с фикстурами работает
КГ
Какая разница мокать большую или маленькую если мокать один метод в итоге
Разница есть. Тут Фесор часто набрасывает, что важно все методы реализовывать. Я правда нупь и пока сам не понял суть проблемы
ПГ
Можно вообще без обёрток. У нас такие места тестируются передачей реального ManagerRegistry. То есть не юнит тест получается, а с фикстурами работает
Ну сначало надо будет замокать его, потом то что он возвращает (репо или ем) Ну или я не понимаю.
SB
Нейминг и сокрытие реализации?
Ну в плане вот запрос скрыт в функционале и даже не светится
а тут мы его в обертку и пользуйтесь други дорогие помоему сокрытие страдает
Ну и нейминг приватной функции против нейминга обертки не вижу разницы существенной
а тут мы его в обертку и пользуйтесь други дорогие помоему сокрытие страдает
Ну и нейминг приватной функции против нейминга обертки не вижу разницы существенной
ПГ
Разница есть. Тут Фесор часто набрасывает, что важно все методы реализовывать. Я правда нупь и пока сам не понял суть проблемы
Ну если кейс такой есть, то ок. Я такого кейса не знаю.
КГ
Ну сначало надо будет замокать его, потом то что он возвращает (репо или ем) Ну или я не понимаю.
чужие классы плохо мокать, тест просто идёт в базу и делает дела как и должен. Я же заранее знаю к каким фикстурам он пойдёт, чтобы проверить результат