Добро пожаловать в форум, Guest  >>   Войти | Регистрация | Поиск | Правила | В избранное | Подписаться
Все форумы / ADO.NET, LINQ, Entity Framework, NHibernate, DAL, ORM Новый топик    Ответить
Топик располагается на нескольких страницах: [1] 2 3 4 5   вперед  Ctrl      все
 EntityFramework и паттерн Repository  [new]
Calabonga
Member

Откуда: www.calabonga.net
Сообщений: 213
С коллегами на работе сложились разногласия относительно необходимости реализации паттерна Repository в ASP.NET MVC при наличии EntityFramework. Одни говорят, что не требуется, так как уже EF всё реализует, другие (и я в их числе) говорят что он (паттерн Repository) просто немного видоизменился и теперь это не чистый DAL, а еще и Pagination, Row Level Security и Model -> ViewModel mapping и прочие полезности этого слоя.

А как вы считаете?
22 июл 18, 18:39    [21592342]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
buser
Member

Откуда: Санкт-Петербург
Сообщений: 4305
как-то так
22 июл 18, 19:08    [21592369]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
buser
Member

Откуда: Санкт-Петербург
Сообщений: 4305
ссылко не тот! ищи EF Repository
22 июл 18, 19:08    [21592370]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
buser
Member

Откуда: Санкт-Петербург
Сообщений: 4305
EF + Repository там где постов > 100
22 июл 18, 19:10    [21592373]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
skyANA
Member

Откуда: Зеленоград
Сообщений: 25854
Calabonga
А как вы считаете?

Репозиторий - это полная абстракция от системы хранения, а не только от конкретной РСУБД.

К примеру у нас в Абрикосе есть темы: https://www.wildapricot.com/membership-website-themes

С точки зрения хранения - это набор файлов. С точки зрения домена приложения - это корень агрегации Theme.
И вот ThemeRepository по указанной спецификации возвращает нужную тему: оригинальную, или кастомную.
Первая - это набор файлов, что изначально создали мы; последняя - это частично расширенная, или полностью переработанная клиентами.
И вот оригинальных тем относительно мало и они хранятся на машине локально, а кастомных относительно много и они хранятся на сетевой шаре.
И, как Вы понимаете, при каждом запросе собирать корень агрегации с диска, или с сетевой шары, это чертовски нагладно.
Поэтому реализован двухуровневый кэш: MemoryCache + Couchbase. Реализован он через шаблон Декоратор.
Подключается/отключается через IoC.

И при этом для вызывающего кода работа со всем этим выглядит как repository.GetTheme(specification).
И это не изменится, если сетевая шара будет послана и заменена на микросервис писанный на Go, или на MongoDB GridFS
22 июл 18, 19:31    [21592403]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
hVostt
Member

Откуда:
Сообщений: 14975
Calabonga
А как вы считаете?


Вообще, при использовании EF, лепить сверху паттерн Репозиторий совсем не обязательно. В этом есть как и плюсы, так и минусы. Если вы совершенно точно никогда за рамки EF в своих проектах не выйдите, и за рамки одной единственной БД, то не нужен вам этот репозиторий.
22 июл 18, 20:35    [21592479]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
hVostt
Member

Откуда:
Сообщений: 14975
Calabonga,

Можно запилить простой generic репозиторий, под каждую сущность, на IQueryable, и это решит ваши концептуальные проблемы в спорах, зато будет больше гибкости, и способности к изменениям вырастут на порядке.

Конечно совсем по феншую это CQS, и отказ от EF, в пользу команд и запросов. Но это дорого на начальном этапе.
22 июл 18, 20:38    [21592482]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Petro123
Member

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)
Сообщений: 35700
Calabonga
разногласия относительно необходимости
не нужен
23 июл 18, 07:32    [21592926]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Calabonga
Member

Откуда: www.calabonga.net
Сообщений: 213
hVostt
Calabonga
А как вы считаете?


Вообще, при использовании EF, лепить сверху паттерн Репозиторий совсем не обязательно. В этом есть как и плюсы, так и минусы. Если вы совершенно точно никогда за рамки EF в своих проектах не выйдите, и за рамки одной единственной БД, то не нужен вам этот репозиторий.


В общем-то, идея понятна. Но хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository??? Я-то себе сделал nuget обобщенный репозиторий.
23 июл 18, 10:53    [21593388]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Алексей К
Member

Откуда: Новосибирск
Сообщений: 13631
Calabonga
В общем-то, идея понятна. Но хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository???
Например для Pagination удобно использовать расширяющие методы для IQueryable.
23 июл 18, 11:14    [21593482]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Petro123
Member

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)
Сообщений: 35700
Calabonga
EntityFramework не реализует Pagination

А почему нужно все пихать что даже не всегда нужно?
Пагинация нужна строго для грида с пагинацией.
И правильно сказали делается одной строчкой.
Тоже самое и с остальным.
....
Если бы ты показал as is to be и весь ужас сегодняшнего EF тогда конечно.
23 июл 18, 11:23    [21593519]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
skyANA
Member

Откуда: Зеленоград
Сообщений: 25854
Calabonga
Я-то себе сделал nuget обобщенный

Гы... https://aspnetboilerplate.com/Pages/Documents/EntityFramework-Integration
23 июл 18, 11:50    [21593609]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
hVostt
Member

Откуда:
Сообщений: 14975
Calabonga
В общем-то, идея понятна. Но хочу тогда уточнить, что есть EntityFramework не реализует Pagination, Row Level Security и Model -> ViewModel mapping, то где реализуете данную логику если не используюте Repository??? Я-то себе сделал nuget обобщенный репозиторий.


EF реализует Pagination через интерфейсы IQueryable Take/Skip.

В обязанности репозитория не входит обеспечение Row Level Security, реализовывать его там -- вколачивать кривые ржавые костыли.

Посмотри паттерн Спецификация и его реализации на IQueryable (гуглится на раз). Это то, что тебе нужно для сесурыти )

Model -> ViewModel должен делать контроллер, с помощью AutoMapper, например. Это его задача, а не репозитория, который совершенно точно не должен знать и понятий иметь ни о каких вью моделях.
23 июл 18, 11:53    [21593625]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
hVostt
Member

Откуда:
Сообщений: 14975
skyANA
Calabonga
Я-то себе сделал nuget обобщенный

Гы... https://aspnetboilerplate.com/Pages/Documents/EntityFramework-Integration


В общем, кто во что горазд.

Поэтому, на не больших проектах и даже средненьких можно легко обходится без доп. слоя репо, так как EF им и является. А в больших EF и не нужен вовсе
23 июл 18, 11:55    [21593629]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
skyANA
Member

Откуда: Зеленоград
Сообщений: 25854
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 июл 18, 11:59    [21593646]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
hVostt
Member

Откуда:
Сообщений: 14975
Calabonga,

Посмотрел на нугет, перешёл на сайт, глянул исходники.
Я считаю, что разделение на ReadableRepositoryBase и Writeable это ошибка.
Репозиторий по своей сути, это набор методов:

GetById
Add
Remove
Update

Это идеальный во всех смыслах репозиторий. Отсадить выполнение запросов надо не в репозиторий "только для чтения" -- это немного асбурдно, так как репо хранит объекты, сущности, а результы запроса самые разные, далеко не сущности, а некие проекции, аггрегации, суммы, группировки. Это все не задачи репозитория.

Т.е. надо IRepository и типа IQueryService если уж на то пошло. IQueryService принимает либо Query Object, либо работает со спецификациями и позволяет теребонькать за IQueryable.
23 июл 18, 12:01    [21593657]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
skyANA
Member

Откуда: Зеленоград
Сообщений: 25854
hVostt
Calabonga,

Посмотрел на нугет, перешёл на сайт, глянул исходники.
Я считаю, что разделение на ReadableRepositoryBase и Writeable это ошибка.
Репозиторий по своей сути, это набор методов:

GetById
Add
Remove
Update

Это идеальный во всех смыслах репозиторий. Отсадить выполнение запросов надо не в репозиторий "только для чтения" -- это немного асбурдно, так как репо хранит объекты, сущности, а результы запроса самые разные, далеко не сущности, а некие проекции, аггрегации, суммы, группировки. Это все не задачи репозитория.

Т.е. надо IRepository и типа IQueryService если уж на то пошло. IQueryService принимает либо Query Object, либо работает со спецификациями и позволяет теребонькать за IQueryable.

Мы же это вроде как обсуждали уже в мае
23 июл 18, 12:06    [21593676]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
hVostt
Member

Откуда:
Сообщений: 14975
skyANA
Мы же это вроде как обсуждали уже в мае


Повторение матерь учения :)
23 июл 18, 12:07    [21593681]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Calabonga
Member

Откуда: www.calabonga.net
Сообщений: 213
hVostt,


Выставлять IQueryable наружу это мовитон, ибо с юнит тестами к нему не подлезешь. А то что сборка разбита на две части, это только плюс, потому что в зависимости от ролей пользователя он получит только то, что можно. Очень удобно инжектить. Да реализация проще, например есть только сайты для чтения fate name.ru например. Думаю позже на два пакета разбить, чтобы ставить только то, что нужно.

А что касается "идельного", ты вы же сказали, что модно и без него если есть EF... :)
23 июл 18, 17:50    [21595148]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Calabonga
Member

Откуда: www.calabonga.net
Сообщений: 213
hVostt,


Думаю вам нужно ознакомиться с плюсами и минусами выставления наружу IQueryable. Не все так гладко в этом вопросе.
23 июл 18, 17:52    [21595158]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Shocker.Pro
Member

Откуда: ->|<- :адуктО
Сообщений: 19550
А кто тут предлагал выставлять его наружу? Только в сервисный слой.
Или что такое "ружа" в вашем понимании?
23 июл 18, 18:08    [21595211]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
skyANA
Member

Откуда: Зеленоград
Сообщений: 25854
Calabonga
А что касается "идельного", ты вы же сказали, что модно и без него если есть EF... :)

Можно вообще никакие паттерны не использовать в проекте, где есть EF, не пристрелят
23 июл 18, 18:10    [21595219]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
skyANA
Member

Откуда: Зеленоград
Сообщений: 25854
Calabonga
А то что сборка разбита на две части, это только плюс, потому что в зависимости от ролей пользователя он получит только то, что можно.
В плане? Запретить какой-то роли писать, какой-то читать в рамках одного репозитория? Легко.

Calabonga
Очень удобно инжектить.
Один generic repository тоже очень удобно инжектить.

Calabonga
Да реализация проще, например есть только сайты для чтения fate name.ru например.
Дык реализуйте метод GetById, а в методах Add, Remove, Update киньте исключение: "Ай, ай, ай, у нас сайт только для чтения, какой суке понадобилось дёрнуть Add?!"

А вообще давно уже придумали CQS.
23 июл 18, 18:17    [21595245]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Petro123
Member

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)
Сообщений: 35700
Shocker.Pro
А кто тут предлагал выставлять его наружу? Только в сервисный слой.
Или что такое "ружа" в вашем понимании?
+1 Moжно и в модели, если она чуть толще голых сущностей.
Ну и потом он десктоп забыл.
Все его примеры от MVC web идут.
23 июл 18, 18:31    [21595290]     Ответить | Цитировать Сообщить модератору
 Re: EntityFramework и паттерн Repository  [new]
Petro123
Member

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)
Сообщений: 35700
Calabonga
Не все так гладко в этом вопросе.
увы.
В жизни все не гладко).
Вот MS взяло и сделало реализацию репозитария в виде DBSet<T>
И поди докажи что у тебя репо лучше) и программистам легче).
Дописывать EF это все равно что Мону Лизу дорисовывать).
23 июл 18, 19:05    [21595389]     Ответить | Цитировать Сообщить модератору
Топик располагается на нескольких страницах: [1] 2 3 4 5   вперед  Ctrl      все
Все форумы / ADO.NET, LINQ, Entity Framework, NHibernate, DAL, ORM Ответить