MVC 中的 ViewModel 和与实体框架的一对多关系？

Question

我有一个用于在数据库中存储有关顾问的信息的应用程序。该模型是一个实体框架模型，数据库表与许多其他表（WorkExperiences、Programs、CompetenceAreas 等）具有一对多关系。现在，当我想在视图中创建一个新的 Consultant 对象时，我实际上只想将 Consultant 对象作为模型传递给视图。但首先，有人向我建议（Asp.Net MVC 3 应用程序中复杂子对象的集合？）我不应该这样做，而是使用 ViewModels。其次，也许这就是原因，当我尝试发布顾问对象（如果将其用作视图中的模型）时，我收到一条错误消息“EntityCollection 已被初始化”，并且错误的原因似乎是对象的集合，例如 WorkExperiences。

所以我的第一个问题是为什么我会收到此错误。

但更重要的是，如果我应该使用 ViewModel，我该如何正确地做到这一点？因为我实际上已经尝试过一些东西，并且成功了。但是......代码很糟糕。谁能告诉我我应该做什么才能让这个工作更干净？

让我向您展示我所拥有的内容（这再次有效，但在代码方面是一场噩梦）：

GET Create 方法：

    public ActionResult Create()
    {
        Consultant consultant = new Consultant();
        ConsultantViewModel vm = GetViewModel(consultant);

        return View(vm);
    }

创建“ViewModel”的辅助方法（如果这实际上是 ViewModel 应该的样子）：

    private ConsultantViewModel GetViewModel(Consultant consultant)
    {
        ConsultantViewModel vm = new ConsultantViewModel();
        vm.FirstName = consultant.FirstName;
        vm.LastName = consultant.LastName;
        vm.UserName = consultant.UserName;
        vm.Description = consultant.Description;

        vm.Programs = consultant.Programs.ToList();
        vm.Languages = consultant.Languages.ToList();
        vm.Educations = consultant.Educations.ToList();
        vm.CompetenceAreas = consultant.CompetenceAreas.ToList();
        vm.WorkExperiences = consultant.WorkExperiences.ToList();
        return vm;
    }

POST创建方法：

    [HttpPost]
    [ValidateInput(false)] //To allow HTML in description box
    public ActionResult Create(ConsultantViewModel vm, FormCollection collection)
    {
        try
        {
            Consultant consultant = CreateConsultant(vm);
            _repository.AddConsultant(consultant);
            _repository.Save();
            return RedirectToAction("Index");
        }
        catch
        {
            return View();
        }
    }

创建顾问对象的辅助方法（这个方法特别糟糕，我必须检查集合是否不为空，以防用户决定不在这些列表中添加任何内容...）：

    private Consultant CreateConsultant(ConsultantViewModel vm)
    {
        Consultant consultant = new Consultant();
        consultant.Description = vm.Description;
        consultant.FirstName = vm.FirstName;
        consultant.LastName = vm.LastName;
        consultant.UserName = vm.UserName;

        if (vm.Programs != null)
            foreach (var program in vm.Programs)
                consultant.Programs.Add(program);
        if (vm.Languages != null)
            foreach (var language in vm.Languages)
                consultant.Languages.Add(language);
        if (vm.Educations != null)
            foreach (var education in vm.Educations)
                consultant.Educations.Add(education);
        if (vm.WorkExperiences != null)
            foreach (var workExperience in vm.WorkExperiences)
                consultant.WorkExperiences.Add(workExperience);
        if (vm.CompetenceAreas != null)
            foreach (var competenceArea in vm.CompetenceAreas)
                consultant.CompetenceAreas.Add(competenceArea);

        return consultant;
    }

所以，它再次起作用，但远不像我可以直接使用 Consultant 对象那么干净（如果不是因为“EntityCollection 已初始化”错误”...）。那么我应该怎么做呢？

Answer 1

首先，您不应该使用实体对象作为视图模型，因为（我现在至少可以想到两个原因，但还有更多）：

1. 您不想公开敏感数据，例如“ID”或“密码”。想象一下，您的顾问有一个 Id，而邪恶的用户打开编辑顾问页面并回发不同的 Id。因此，邪恶的用户将成功更新不同的Consultant。

2. 目前，您在视图中显示的任何内容都与您的 Consultant 对象的外观相对应。但如果您想添加不属于 Consultant 对象的额外信息（就像复选框字段一样简单）。在这种情况下，您必须重写大量代码、创建 ViewModel、映射它等。而如果您从一开始就遵循 ViewModel 模式，则可以在需要时进行这个简单的更改。

关于您的代码 - 您可以尝试将 AutoMapper 与 嵌套映射用于此类转换。即使您不这样做，也可以通过使用投影使您的代码变得更加简洁。

private ConsultantViewModel GetViewModel(Consultant consultant)
{
    return new ConsultantViewModel
               {
                   FirstName = consultant.FirstName,
                   LastName = consultant.LastName,
                   ...
                   vm.Programs = consultant.Programs.ToList(),
                   ...
               };
 }

 private Consultant CreateConsultant(ConsultantViewModel vm)
 {
     var consultant = new Consultant
                      {
                          Description = vm.Description,
                          FirstName = vm.FirstName,
                          ...
                       };

     if (vm.Programs != null)
     {
         vm.Programs.ForEach(consultant.Programs.Add);
     }
     ...

     return consultant;
}