.NET Core MVC编辑时ViewModel关联实体为空致更新错误的最佳实践

这个问题在.NET Core MVC处理关联实体编辑时挺常见的，核心原因是HTTP Post请求只会回传表单里实际存在的字段——你视图里只展示了Counselor和Student的名称，没提交它们的ID或者实体数据，所以ViewModel里这两个关联对象自然是空的，EF Core更新时误以为要把外键置为null，但你的外键是必填项，就抛出了错误。

你现在用Post时重新查询数据库的方法是可行的，但还有更优雅的最佳实践，分几种场景给你参考：

1. 推荐：在ViewModel中保留外键字段，通过隐藏表单回传 #

这是最简洁高效的方案，不需要额外查询数据库：

• 给你的ScheduleViewModel添加CounselorId和StudentId属性（和Schedule实体的外键对应）；
• 在编辑视图中，把这两个外键字段设为隐藏输入，比如：

  @Html.HiddenFor(m => m.CounselorId)
  @Html.HiddenFor(m => m.StudentId)
  <!-- 或者用Tag Helper -->
  <input asp-for="CounselorId" type="hidden" />
  <input asp-for="StudentId" type="hidden" />
  

• Post动作中，直接用ViewModel里的CounselorId和StudentId更新Schedule实体：

  [HttpPost]
  public async Task<IActionResult> Edit(ScheduleViewModel vm)
  {
      if (!ModelState.IsValid)
      {
          // 重新加载关联实体用于错误页面展示
          vm.Counselor = await _context.Counselors.FindAsync(vm.CounselorId);
          vm.Student = await _context.Students.FindAsync(vm.StudentId);
          return View(vm);
      }
  
      var schedule = await _context.Schedules.FindAsync(vm.Schedule.Id);
      if (schedule == null) return NotFound();
  
      // 映射Schedule的属性，直接用回传的外键
      schedule.CounselorId = vm.CounselorId;
      schedule.StudentId = vm.StudentId;
      // 其他属性赋值，比如schedule.Date = vm.Schedule.Date;
  
      await _context.SaveChangesAsync();
      return RedirectToAction(nameof(Index));
  }
  

优点：不需要额外查询数据库，性能好，逻辑简单；注意：隐藏字段可能被前端篡改，Post时要验证外键对应的实体是否存在，以及用户是否有权限操作该实体。

2. 用AutoMapper简化实体映射（优化你当前的方案） #

如果你不想暴露外键到ViewModel，或者需要更复杂的字段映射，可以用AutoMapper来替代手动赋值，减少重复代码：

• 先安装AutoMapper NuGet包，配置映射规则（把ViewModel的Schedule属性映射到数据库Schedule实体，同时保留原有的关联实体）；
• Post动作中查询数据库的Schedule实体，用AutoMapper把ViewModel的字段映射过去，然后保存：

  [HttpPost]
  public async Task<IActionResult> Edit(ScheduleViewModel vm)
  {
      if (!ModelState.IsValid)
      {
          vm.Counselor = await _context.Counselors.FindAsync(vm.Schedule.CounselorId);
          vm.Student = await _context.Students.FindAsync(vm.Schedule.StudentId);
          return View(vm);
      }
  
      var schedule = await _context.Schedules.Include(s => s.Counselor).Include(s => s.Student)
                                             .FirstOrDefaultAsync(s => s.Id == vm.Schedule.Id);
      if (schedule == null) return NotFound();
  
      // 用AutoMapper映射ViewModel的字段到数据库实体
      _mapper.Map(vm.Schedule, schedule);
  
      await _context.SaveChangesAsync();
      return RedirectToAction(nameof(Index));
  }
  

优点：代码更简洁，适合多字段的复杂实体；缺点：需要查询数据库加载关联实体，性能略逊于方案1。

3. 直接操作EF Core实体状态（进阶） #

如果你不想查询数据库，可以直接创建Schedule实体，设置外键和需要更新的属性，然后附加到上下文并标记为修改：

[HttpPost]
public async Task<IActionResult> Edit(ScheduleViewModel vm)
{
    if (!ModelState.IsValid)
    {
        vm.Counselor = await _context.Counselors.FindAsync(vm.CounselorId);
        vm.Student = await _context.Students.FindAsync(vm.StudentId);
        return View(vm);
    }

    var schedule = new Schedule
    {
        Id = vm.Schedule.Id,
        CounselorId = vm.CounselorId,
        StudentId = vm.StudentId,
        // 其他需要更新的属性
    };

    _context.Attach(schedule);
    // 标记所有属性为修改，或者只标记需要更新的属性
    _context.Entry(schedule).State = EntityState.Modified;
    // 比如只更新Date：_context.Entry(schedule).Property(s => s.Date).IsModified = true;

    await _context.SaveChangesAsync();
    return RedirectToAction(nameof(Index));
}

优点：不需要查询数据库，性能最好；缺点：需要手动管理实体状态，容易遗漏字段，适合对EF Core熟悉的开发者。

总结下来，方案1是最推荐的常规做法，既简单又高效；如果有复杂映射需求，方案2更合适；方案3适合追求极致性能的场景。

内容的提问来源于stack exchange，提问作者user1778158