|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
С коллегами на работе сложились разногласия относительно необходимости реализации паттерна Repository в ASP.NET MVC при наличии EntityFramework. Одни говорят, что не требуется, так как уже EF всё реализует, другие (и я в их числе) говорят что он (паттерн Repository) просто немного видоизменился и теперь это не чистый DAL, а еще и Pagination, Row Level Security и Model -> ViewModel mapping и прочие полезности этого слоя. А как вы считаете? ... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 18:39 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 19:08 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
ссылко не тот! ищи EF Repository ... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 19:08 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
EF + Repository там где постов > 100 ... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 19:10 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaА как вы считаете? Репозиторий - это полная абстракция от системы хранения, а не только от конкретной РСУБД. К примеру у нас в Абрикосе есть темы: https://www.wildapricot.com/membership-website-themes С точки зрения хранения - это набор файлов. С точки зрения домена приложения - это корень агрегации Theme. И вот ThemeRepository по указанной спецификации возвращает нужную тему: оригинальную, или кастомную. Первая - это набор файлов, что изначально создали мы; последняя - это частично расширенная, или полностью переработанная клиентами. И вот оригинальных тем относительно мало и они хранятся на машине локально, а кастомных относительно много и они хранятся на сетевой шаре. И, как Вы понимаете, при каждом запросе собирать корень агрегации с диска, или с сетевой шары, это чертовски нагладно. Поэтому реализован двухуровневый кэш: MemoryCache + Couchbase. Реализован он через шаблон Декоратор. Подключается/отключается через IoC. И при этом для вызывающего кода работа со всем этим выглядит как repository.GetTheme(specification) . И это не изменится, если сетевая шара будет послана и заменена на микросервис писанный на Go, или на MongoDB GridFS ... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 19:31 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaА как вы считаете? Вообще, при использовании EF, лепить сверху паттерн Репозиторий совсем не обязательно. В этом есть как и плюсы, так и минусы. Если вы совершенно точно никогда за рамки EF в своих проектах не выйдите, и за рамки одной единственной БД, то не нужен вам этот репозиторий. ... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 20:35 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
Calabonga, Можно запилить простой generic репозиторий, под каждую сущность, на IQueryable, и это решит ваши концептуальные проблемы в спорах, зато будет больше гибкости, и способности к изменениям вырастут на порядке. Конечно совсем по феншую это CQS, и отказ от EF, в пользу команд и запросов. Но это дорого на начальном этапе. ... |
|||
:
Нравится:
Не нравится:
|
|||
22.07.2018, 20:38 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
Calabongaразногласия относительно необходимостине нужен ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 07:32 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
hVosttCalabongaА как вы считаете? Вообще, при использовании EF, лепить сверху паттерн Репозиторий совсем не обязательно. В этом есть как и плюсы, так и минусы. Если вы совершенно точно никогда за рамки EF в своих проектах не выйдите, и за рамки одной единственной БД, то не нужен вам этот репозиторий. В общем-то, идея понятна. Но хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository??? Я-то себе сделал nuget обобщенный репозиторий. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 10:53 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaВ общем-то, идея понятна. Но хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository???Например для Pagination удобно использовать расширяющие методы для IQueryable. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 11:14 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaEntityFramework не реализует Pagination А почему нужно все пихать что даже не всегда нужно? Пагинация нужна строго для грида с пагинацией. И правильно сказали делается одной строчкой. Тоже самое и с остальным. .... Если бы ты показал as is to be и весь ужас сегодняшнего EF тогда конечно. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 11:23 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaЯ-то себе сделал nuget обобщенный Гы... https://aspnetboilerplate.com/Pages/Documents/EntityFramework-Integration ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 11:50 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaВ общем-то, идея понятна. Но хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository??? Я-то себе сделал nuget обобщенный репозиторий. EF реализует Pagination через интерфейсы IQueryable Take/Skip. В обязанности репозитория не входит обеспечение Row Level Security, реализовывать его там -- вколачивать кривые ржавые костыли. Посмотри паттерн Спецификация и его реализации на IQueryable (гуглится на раз). Это то, что тебе нужно для сесурыти ) Model -> ViewModel должен делать контроллер, с помощью AutoMapper, например. Это его задача, а не репозитория, который совершенно точно не должен знать и понятий иметь ни о каких вью моделях. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 11:53 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
skyANACalabongaЯ-то себе сделал nuget обобщенный Гы... https://aspnetboilerplate.com/Pages/Documents/EntityFramework-Integration В общем, кто во что горазд. Поэтому, на не больших проектах и даже средненьких можно легко обходится без доп. слоя репо, так как EF им и является. А в больших EF и не нужен вовсе ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 11:55 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaНо хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository??? One Domain-Driven-Design solution to the problem of where to place querying, sorting, and paging logic is to use a Specification . https://deviq.com/specification-pattern/ ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 11:59 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
Calabonga, Посмотрел на нугет, перешёл на сайт, глянул исходники. Я считаю, что разделение на ReadableRepositoryBase и Writeable это ошибка. Репозиторий по своей сути, это набор методов: GetById Add Remove Update Это идеальный во всех смыслах репозиторий. Отсадить выполнение запросов надо не в репозиторий "только для чтения" -- это немного асбурдно, так как репо хранит объекты, сущности, а результы запроса самые разные, далеко не сущности, а некие проекции, аггрегации, суммы, группировки. Это все не задачи репозитория. Т.е. надо IRepository и типа IQueryService если уж на то пошло. IQueryService принимает либо Query Object, либо работает со спецификациями и позволяет теребонькать за IQueryable. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 12:01 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
hVosttCalabonga, Посмотрел на нугет, перешёл на сайт, глянул исходники. Я считаю, что разделение на ReadableRepositoryBase и Writeable это ошибка. Репозиторий по своей сути, это набор методов: GetById Add Remove Update Это идеальный во всех смыслах репозиторий. Отсадить выполнение запросов надо не в репозиторий "только для чтения" -- это немного асбурдно, так как репо хранит объекты, сущности, а результы запроса самые разные, далеко не сущности, а некие проекции, аггрегации, суммы, группировки. Это все не задачи репозитория. Т.е. надо IRepository и типа IQueryService если уж на то пошло. IQueryService принимает либо Query Object, либо работает со спецификациями и позволяет теребонькать за IQueryable. Мы же это вроде как обсуждали уже в мае ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 12:06 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
skyANAМы же это вроде как обсуждали уже в мае Повторение матерь учения :) ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 12:07 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
hVostt, Выставлять IQueryable наружу это мовитон, ибо с юнит тестами к нему не подлезешь. А то что сборка разбита на две части, это только плюс, потому что в зависимости от ролей пользователя он получит только то, что можно. Очень удобно инжектить. Да реализация проще, например есть только сайты для чтения fate name.ru например. Думаю позже на два пакета разбить, чтобы ставить только то, что нужно. А что касается "идельного", ты вы же сказали, что модно и без него если есть EF... :) ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 17:50 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
hVostt, Думаю вам нужно ознакомиться с плюсами и минусами выставления наружу IQueryable. Не все так гладко в этом вопросе. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 17:52 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
А кто тут предлагал выставлять его наружу? Только в сервисный слой. Или что такое "ружа" в вашем понимании? ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 18:08 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaА что касается "идельного", ты вы же сказали, что модно и без него если есть EF... :) Можно вообще никакие паттерны не использовать в проекте, где есть EF, не пристрелят ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 18:10 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaА то что сборка разбита на две части, это только плюс, потому что в зависимости от ролей пользователя он получит только то, что можно.В плане? Запретить какой-то роли писать, какой-то читать в рамках одного репозитория? Легко. CalabongaОчень удобно инжектить.Один generic repository тоже очень удобно инжектить. CalabongaДа реализация проще, например есть только сайты для чтения fate name.ru например.Дык реализуйте метод GetById, а в методах Add, Remove, Update киньте исключение: "Ай, ай, ай, у нас сайт только для чтения, какой суке понадобилось дёрнуть Add?!" А вообще давно уже придумали CQS. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 18:17 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
Shocker.ProА кто тут предлагал выставлять его наружу? Только в сервисный слой. Или что такое "ружа" в вашем понимании?+1 Moжно и в модели, если она чуть толще голых сущностей. Ну и потом он десктоп забыл. Все его примеры от MVC web идут. ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 18:31 |
|
EntityFramework и паттерн Repository
|
|||
---|---|---|---|
#18+
CalabongaНе все так гладко в этом вопросе.увы. В жизни все не гладко). Вот MS взяло и сделало реализацию репозитария в виде DBSet<T> И поди докажи что у тебя репо лучше) и программистам легче). Дописывать EF это все равно что Мону Лизу дорисовывать). ... |
|||
:
Нравится:
Не нравится:
|
|||
23.07.2018, 19:05 |
|
|
start [/forum/topic.php?fid=17&msg=39677414&tid=1349162]: |
0ms |
get settings: |
10ms |
get forum list: |
15ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
52ms |
get topic data: |
12ms |
get forum data: |
3ms |
get page messages: |
69ms |
get tp. blocked users: |
2ms |
others: | 15ms |
total: | 186ms |
0 / 0 |