ASP.NET MVC部门编辑功能异常：点击更新却新增条目求助

核心问题原因 #

服务层的UpdateAsync方法直接对传入的newDepartments调用_context.Update()，但这个对象是从表单绑定生成的未被EF上下文跟踪的实体，EF无法识别它是已存在的记录，因此会执行插入操作而非更新。

具体修改方案 #

1. 修正服务层的UpdateAsync方法 #

修改DepartmentsService中的UpdateAsync方法，改为先从数据库获取被上下文跟踪的实体，再逐个更新属性：

public async Task<Departments> UpdateAsync(int id, Departments newDepartments)
{
    // 从数据库获取已跟踪的实体
    var existingDepartment = await _context.Departments.FirstOrDefaultAsync(n => n.DepartmentId == id);
    if (existingDepartment == null)
        throw new KeyNotFoundException($"Department with id {id} not found");

    // 更新实体属性
    existingDepartment.Name = newDepartments.Name;
    existingDepartment.PictureURL = newDepartments.PictureURL;
    existingDepartment.Description = newDepartments.Description;

    // 保存更改
    await _context.SaveChangesAsync();
    return existingDepartment;
}

2. 增强控制器Edit方法的校验 #

在控制器的Edit Post方法中，添加id与实体ID的一致性校验，避免非法提交：

[HttpPost]
public async Task<IActionResult> Edit(int id, [Bind("DepartmentId,Name,PictureURL,Description")] Departments departments)
{
    // 校验传入的id与实体ID是否匹配
    if (id != departments.DepartmentId)
    {
        return View("NotFound");
    }

    if (!ModelState.IsValid)
    {
        return View(departments);
    }

    try
    {
        await _service.UpdateAsync(id, departments);
    }
    catch (KeyNotFoundException)
    {
        return View("NotFound");
    }

    return RedirectToAction(nameof(Index));
}

3. 确认Edit视图的正确性（关键） #

你当前贴出的"编辑视图代码"实际是Delete视图（标题为Delete department，表单提交到Delete动作），请确认你的Edit视图包含以下内容：

• 表单的asp-action="Edit"
• 包含DepartmentId的隐藏字段（确保提交时能传递ID）
• 可编辑的表单字段（而非readonly）

正确的Edit视图示例片段：

@model Departments

@{
    ViewData["Title"] = "Edit Department";
}

<form asp-action="Edit">
    <input type="hidden" asp-for="DepartmentId" />
    <div class="form-group">
        <label asp-for="Name" class="control-label"></label>
        <input asp-for="Name" class="form-control" />
        <span asp-validation-for="Name" class="text-danger"></span>
    </div>
    <!-- 其他字段 -->
    <input type="submit" value="Update" class="btn btn-primary" />
</form>

原理说明 #

EF Core的Update方法会根据实体的跟踪状态判断操作：

• 如果是上下文已跟踪的实体（比如通过GetByIdAsync获取的），Update会标记为修改状态，执行更新
• 如果是未跟踪的实体（比如表单绑定生成的），EF会检查主键：如果主键为默认值（比如int类型的0）则插入；如果主键有值，但上下文未跟踪，仍会执行插入（除非你手动附加并标记为修改）
  通过先获取数据库中的跟踪实体再更新属性，能确保EF正确识别为修改操作。

内容的提问来源于stack exchange，提问作者Denisa-Maria