sysauxКакие данные нужно хранить во ViewModel-и, а какие лучше в ViewBag-е?
ViewModel — 1 к 1 данные, которые нужно отображать во View и принимать обратно методом POST, т.е. не больше и не меньше. Лишние поля — плохо, потому что: 1) вводят в заблуждение программистов, которые будут сопровождать и поддерживать ПО; 2) мешают при юнит-тестировании и портят карту покрытия тестами (например, dotCover будет показывать, что эти поля не используются, а если используются, то.. где? почему?). Недостаток полей приводит к кривым костылям для извлечения данных ч/з ViewBag или бог знает ещё как.
ViewBag — удобно и вполне легально использовать для:
1) данных для выпадающих списков (DropDown), например по соглашению: ViewBag.ИмяПоля_Data
2) данных для настройки вложенных PartialViews и Display/EditorTeplates, например, CSS-стили и атрибуты
3) обобщённых данных, использующихся в несвязанных View, например, сообщений, алертов, т.е. частей, которые могут использоваться на любых страницах и глупо эти данные пихать во все ViewModel.
4) другие крайне специфичные вещи, которые выходят за рамки Одно Действие → Одно или несколько Представлений.
Использование ViewBag не даёт контроль типов, и делает невозможным проверку на этапе компиляции, отсутствие IntelliSense усложняет сопровождение и разработку, поэтому их надо использовать совместно с жёстко заданными соглашениями на уровне компонентов (контролов) и атрибутов, т.е. желательно за рамками прикладной логики.
В целом же всегда надо отдавать предпочтение строго типизированным ViewModel-ям, при чём не желательно использовать одну ViewModel в разных Views, т.е. грубо говоря одному View должна соответствовать одна ViewModel, что даёт следующее соглашение по наименованию ViewModel: КонтроллерПредставлениеViewModel.
1.
2.
3.
4.
5.
6.
7.
8.
public class EmployeeController
{
public ActionResult Index()
{
var model = new EmployeeIndexViewModel { ... };
return View(model);
}
}
