在 Asp.Net MVC 中实体作为模型出现“EntityCollection 已初始化”错误？

我在创建一个复杂的对象时遇到了很大的困难。我有一个带有顾问表的 EF 模型，该表与许多其他表具有一对多关系。我想按原样使用 Consultant 对象作为模型，因为它非常简单（正如 NerdDinner 教程中所做的那样），就像我对其他没有一对多关系的对象所做的那样。问题是，当我尝试发布到 Create 方法时，这些关系会导致此错误：“EntityCollection 已初始化”。

有几个人建议我改用 ViewModel，我发布了一个关于此的问题（MVC 中的 ViewModel 和与实体框架的一对多关系？ https://stackoverflow.com/questions/5083880/viewmodels-and-one-to-many-relationships-with-entity-framework-in-mvc）因为我不太明白。问题是代码变得非常荒谬...这与我迄今为止所欣赏的 MVC 的简单性相去甚远。

在那个问题中，我忘记提及除了 Create 方法之外，Edit 方法还使用相同的 CreateConsultant 方法（名称可能会产生误导，它实际上填充 Consultant 对象）。因此，为了不在编辑时添加额外的“程序”，我需要使该方法进一步复杂化。所以现在看起来像这样：

private Consultant CreateConsultant(ConsultantViewModel vm, Consultant consultant) //Parameter Consultant needed because an object may already exist from Edit method.
        {
            consultant.Description = vm.Description;
            consultant.FirstName = vm.FirstName;
            consultant.LastName = vm.LastName;
            consultant.UserName = User.Identity.Name;

            if (vm.Programs != null)
                for (int i = 0; i < vm.Programs.Count; i++)
                {
                    if (consultant.Programs.Count == i)
                        consultant.Programs.Add(vm.Programs[i]);
                    else
                        consultant.Programs.ToList()[i] = vm.Programs[i];
                }
            if (vm.Languages != null)
                for (int i = 0; i < vm.Languages.Count; i++)
                {
                    if (consultant.Languages.Count == i)
                        consultant.Languages.Add(vm.Languages[i]);
                    else
                        consultant.Languages.ToList()[i] = vm.Languages[i];
                }
            if (vm.Educations != null)
                for (int i = 0; i < vm.Educations.Count; i++)
                {
                    if (consultant.Educations.Count == i)
                        consultant.Educations.Add(vm.Educations[i]);
                    else
                        consultant.Educations.ToList()[i] = vm.Educations[i];
                }
            if (vm.WorkExperiences != null)
                for (int i = 0; i < vm.WorkExperiences.Count; i++)
                {
                    if (consultant.WorkExperiences.Count == i)
                        consultant.WorkExperiences.Add(vm.WorkExperiences[i]);
                    else
                        consultant.WorkExperiences.ToList()[i] = vm.WorkExperiences[i];
                }

            if (vm.CompetenceAreas != null)
                for (int i = 0; i < vm.CompetenceAreas.Count; i++)
                {
                    if (consultant.CompetenceAreas.Count == i)
                        consultant.CompetenceAreas.Add(vm.CompetenceAreas[i]);
                    else
                        consultant.CompetenceAreas.ToList()[i] = vm.CompetenceAreas[i];
                }

            string uploadDir = Server.MapPath(Request.ApplicationPath) + "FileArea\\ConsultantImages\\";
            foreach (string f in Request.Files.Keys)
            {
                var filePath = Path.Combine(uploadDir, Path.GetFileName(Request.Files[f].FileName));
                if (Request.Files[f].ContentLength > 0)
                {
                    Request.Files[f].SaveAs(filePath);
                    consultant.Image = filePath;
                }

            }
            return consultant;
        }

这是荒谬的，这可能是由于我的无能，但我需要知道如何正确地做到这一点。仅回答“使用 ViewModel”显然是不够的，因为这就是让我陷入困境的原因。我希望简单的实体对象作为模型的简单性，但没有“EntityCollection 已初始化”错误。我该如何解决这个问题？

当然，如果我只是以错误的方式执行 ViewModel 策略，也欢迎提出建议，但主要是我想知道如果我以简单的“NerdDinner”简单对象方式执行此操作，会导致此错误。另请记住，相关视图仅限于网站的授权用户。我很乐意以“正确”的方式做到这一点，但如果使用 ViewModels 意味着代码很难维护，我会放弃它......

请帮忙！

UPDATE:

事实证明这段代码根本不起作用。我只是在调用编辑更新值后进行了检查，但它没有更新它们。所以只有创建部分有效。

这是不起作用的部分：

consultant.Programs.ToList()[i] = vm.Programs[i];

我有一种预感，我无法使用 ToList 并更新 EntityCollection 中的项目。但这使得事情变得更加困难。所以现在我不知道如何直接对实体进行操作，我更喜欢这样做（见上文）。我不知道如何让这个 ViewModel 的东西工作，更不用说让它干净了......

有任何想法吗？这里一定有什么真正的错误，我希望有人会发现我是如何错过了一些简单的东西，使所有这些代码都发生了转变！

好吧，根据我的经验，EF 的东西在线上使用时不容易很好地发挥作用，句号（也就是说，无论您是在 MVC 还是 WCF 或其他环境中，它都不是传递数据的好格式）。对我来说，这是一个 EF 问题，而不是 MVC 问题，因为您在直接使用 EF 模型时遇到的问题与它们在 EF 对象上下文中的跟踪方式有关。有人告诉我，有一些解决方案涉及重新附加传递的实体，但我发现这比它的价值更麻烦；和其他人一样，这就是为什么大多数人在这种情况下说“使用视图模型”作为输入参数。

我同意你上面的代码有点令人不快，有几种方法可以修复它。首先，检查一下自动映射器 http://automapper.codeplex.com/，这很有帮助，因为您可以跳过定义明显的内容（例如，当两个模型都有一个简单的属性，例如相同类型和名称的“Name”）。

其次，无论如何，你用循环所做的事情有点不对劲。您不应假设从客户端发布的列表与 EF 正在跟踪的列表（而是列表；指的是您的程序等）具有相同的顺序等。相反，请考虑可能发生的情况并考虑这些情况。我将使用程序作为示例。

1) 顾问有一个计划，但该计划的详细信息已更改。 2) 顾问在视图模型中没有程序 2a) 该程序已经存在于数据库中的某个地方，只是不属于该顾问的一部分 2b) 该程序是新的。 3）顾问有一个不在视图模型中的程序（您当前没有考虑到的东西）。

我不知道在每种情况下您希望发生什么（即，视图模型是完整且规范的，还是更新期间的实体模型？），但假设您正在循环查看视图模型的程序，因为您做上面的。对于场景 1（这似乎是您的主要问题），您将需要执行类似的操作（使用 AutoMapper）：

var updated = vm.Programs[i];    
var original = consultant.Programs.SingleOrdefault(p=>p.ID == uppdated.ID);
Mapper.Map(updated,original);
yourEfContext.SaveChanges(); // at some point after this, doesn't have to be inside the loop

编辑： 另一种可能对您有用的想法是，当您从数据库中获取顾问时（不确定您使用什么机制），请确保您也对要更新的集合调用 Include 。否则，您所做的每一次迭代都将是到数据库的另一次往返，如果您只是使用 Include 一次性加载它们，显然您可以避免这种情况。