ASP MVC — Правильно ли использовать сущности EntityFramework в качестве модели для вывода данных на View?

Question

[mindgrow]

Добрый день.

Есть база данных, с которой я работаю через EF. Есть сущность EF - ApplicationUser. Есть страница View, на котором выводится список пользователей (ApplicationUser).

Правильно ли, если я буду использовать в качестве модели View тип ApplicationUser, которые я буду выгружать из EntitytFramework и загружать на страницу?

Answer 1

[Андрей]

Обычно EF добавляет в свои сущности контекст, и в итоге (теоретически) это будет работать медленнее и кушать больше памяти.
Да и как правило все поля сущности не нужны, но бывают нужны поля которых в сущностях нет.
Плюс, если вам понадобится добавить во View какое-то поле, то проще будет изменить модель а не сущность ef.

Короче говоря, ничто не мешает использовать сущности во view, но это не правильно, лучше создавать отдельные модели для вывода.

Comments

[mindgrow]

а если использовать отдельные модели для вывода, существуют ли какие-то best practice по преобразованию сущности в модель и обратно?

[Евгений]

Viktor_Ilukhin, можно ручками конвертировать в запросе linq, можно использовать мапперы такие как automapper

[Андрей]

Viktor_Ilukhin, по моему опыту - это довольно редкий случай, когда нужно только замапить одни поля к другим. Обычно какие то еще действия делаются.
Я предпочитаю делать это в отдельных классах, чтобы контроллеры были тонкими и простыми.

[mindgrow]

Евгений, automapper это именно то что я искал, спасибо!

Answer 2

[Akaero]

Для потренироваться - да.
Однако есть ньюанс, предположим, что пользователей у вас 100. Выводить такой список без постраничного вывода не очень классно, поэтому лучше сразу это предусмотреть и сделать класс (к примеру, AppUserViewModel) в котором будут следующие свойства:
1. PageNumber - номер запрашиваемой страницы
2. Count - количество пользователей, которые хотим взять из базы
3.ICollection<ApplicationUser> User {get;set;}

Так вот, эту модельку можно уже получать в методе контроллера и отдавать ее во вью, так у Вас будет более структурированный код.

Можно пойти еще дальше. Вам вряд ли все поля в ApplicationUser нужны (я полагаю, что речь идет о классе из Identity). Поэтому, чтобы во вью не передавать всякие ненужные для представления поля (хеш пароля, дата создания и тд) можно в нашей
AppUserViewModel
вместо
ICollection<ApplicationUser> Users {get; set;}
сделать

ICollection<ApplicationUserViewModel>  Users {get; set;}

- то есть создать отдельный класс, который служит для отображения информации об пользователе, но если в классе ApplicationUser у вас было, к примеру, 10 полей, то в классе ApplicationUserViewModel сделайте только те поля, которые вам нужны для отображения. И перед передачей модели в представление просто создавайте коллекцию этих ApplicationUserViewModel, копируя туда нужную инфу.

Comments

[mindgrow]

а есть какие-то популярные приемы по копированию данных между объектами? чтобы например передать объект-источник и объект-приемник в метод, и он все существующие поля перекопировал?

[Akaero]

https://github.com/AutoMapper/AutoMapper

Погуглите туториалы, там абсолютно прозрачно и просто. Viktor_Ilukhin,