JS
Может у него решарпер и Solution-Wide Analysis
О божечки! Снова эти стремные репозитории!
DbSet в DbContext это уже репозиторий!
Не делайте так, новички.
Size: a a a
2020 December 04
V
Хреновый это репозиторий
V
И множество причин почему
NS
Хреновый это репозиторий
Расскажи мне свою боль
IP
Может у него решарпер и Solution-Wide Analysis
кстати как вариант
🅰O
О божечки! Снова эти стремные репозитории!
DbSet в DbContext это уже репозиторий!
Не делайте так, новички.
DbSet в DbContext это уже репозиторий!
Не делайте так, новички.
а как делать?
AO
а как делать?
Просто используй dbcontext напрямую.
NS
а как делать?
Дело не в использовании самого патерна репозитрия, а в том, что люди пишут еще один DbSet а потом мучаются. Репозиторий должен функции какие-то выполнять, возвращать DTO или делать доп. фильтрации и прочее. Если надобности в таком нет, то инжектишь dbcontext сразу в сервис и в контроллере используешь его.
IP
а потом приходит кто то другой и начинает сношать что код без задумки о перспективне на будущее)
JS
Дело не в использовании самого патерна репозитрия, а в том, что люди пишут еще один DbSet а потом мучаются. Репозиторий должен функции какие-то выполнять, возвращать DTO или делать доп. фильтрации и прочее. Если надобности в таком нет, то инжектишь dbcontext сразу в сервис и в контроллере используешь его.
Почему тогда не инжектить контекст прямо в контоллер? )) Зачем прослойка в виде сервиса? ))
NS
Почему тогда не инжектить контекст прямо в контоллер? )) Зачем прослойка в виде сервиса? ))
))
JS
Ihor Ponomarenko
а потом приходит кто то другой и начинает сношать что код без задумки о перспективне на будущее)
А ты его отправляешь читать yagni principle
NZ
Почему тогда не инжектить контекст прямо в контоллер? )) Зачем прослойка в виде сервиса? ))
Тестировать тогда сложно. Сервис можно протестировать без обращения к rest api. А чтобы контроллер протестировать, нужно реально к нему обращаться. Плюс, сервис может содержать в себе дополнительную логику, кроме непосредственно работы с базой данных.
JS
Nikolay Zainchkovskiy
Тестировать тогда сложно. Сервис можно протестировать без обращения к rest api. А чтобы контроллер протестировать, нужно реально к нему обращаться. Плюс, сервис может содержать в себе дополнительную логику, кроме непосредственно работы с базой данных.
Ну так интегрейшн тестами вызывай его
NZ
Ну так интегрейшн тестами вызывай его
А что делать с юнит тестами? Так можно сказать, что вообще e2e - наше всё, и остальные тесты не нужны.
IP
А ты его отправляешь читать yagni principle
а он тебе начинает рассказывать о перспективе перехода на другую субд) и в добавок начинает рассказывать что принцып репозитаория нарушен потому что у тебя там есть еще какието доп функцыи в виде спецыфической выборки)
JS
Ihor Ponomarenko
а он тебе начинает рассказывать о перспективе перехода на другую субд) и в добавок начинает рассказывать что принцып репозитаория нарушен потому что у тебя там есть еще какието доп функцыи в виде спецыфической выборки)
О да, мне тоже иногда начинают рассказывать про датабейс агностик код, но дело в том что рано или поздно ты останешься прибитым гвоздями к своей базе, и тебе все равно приходится писать какой нибудь постгресс специфик код, чтоб ускорить выборку, потому что еф не может асилить что-то там
IP
ага
IP
тут просто часто выходит патерн ради патерна
JS
Ну или интерфейсами обмажутся имея всего одну имплементацию