Entity Framework Core 3.2更新关联Identity User的Medication实体失败

这个错误的核心原因很明确：当你尝试把Medication实体的状态设为Modified时，它的Id属性还是临时值（通常是默认的0或者未赋值）。EF Core需要确认这是一个已经存在于数据库中的实体（拥有永久的、已生成的主键值），才允许设置修改状态——不然它会误以为你要新增一个实体，而不是更新现有实体。

之前没有关联User时没问题，大概率是因为当时的更新逻辑里Id被正确传递了，而新增User关联后，可能在表单提交、模型绑定或者仓储层逻辑里出现了Id丢失的情况。下面是具体的解决步骤：

1. 确保前端表单传递了Medication的Id #

在你的Edit视图表单里，一定要包含Id的隐藏字段，这样提交时Id才会被正确传递到控制器：

<!-- 在Edit.cshtml的表单中添加这个隐藏字段 -->
<input type="hidden" asp-for="Id" />

如果没有这个字段，模型绑定出来的Medication对象Id会是默认值（比如0），EF自然会认为这是个新实体。

2. 修改仓储层的更新逻辑（推荐做法） #

不要直接给传入的Medication对象设置Modified状态，正确的姿势是先从数据库获取已被EF跟踪的原实体，再更新它的属性：

public void UpdateMedication(Medication medication, string userId)
{
    // 先根据Id和UserId找到数据库里的原实体（确保是当前用户的资源）
    var existingMed = _context.Medications.FirstOrDefault(m => m.Id == medication.Id && m.UserId == userId);
    
    if (existingMed == null)
    {
        throw new ArgumentException("找不到要更新的Medication记录");
    }

    // 把传入的新值映射到原实体上
    existingMed.Name = medication.Name;
    existingMed.Dosage = medication.Dosage;
    existingMed.Frequency = medication.Frequency;
    // 其他需要更新的属性...

    // EF会自动跟踪实体的变化，不需要手动设置State
    _context.SaveChanges();
}

这种方式不仅避免了临时Id的问题，还能确保你只更新当前用户有权限修改的记录，同时也更符合EF Core的跟踪机制。

如果你一定要用直接设置状态的方式，必须确保传入的medication对象Id是数据库中已存在的永久值，并且最好先Attach到上下文：

public void UpdateMedication(Medication medication, string userId)
{
    // 先验证Id是否有效
    if (medication.Id == 0)
    {
        throw new InvalidOperationException("Medication的Id不能是临时值");
    }

    // 附加实体并设置状态
    _context.Attach(medication).State = EntityState.Modified;
    // 注意：如果关联了User，可能需要确保UserId也正确，避免越权
    medication.UserId = userId;
    _context.SaveChanges();
}

3. 在控制器中添加参数校验 #

在Edit的POST方法里，先检查传入的Medication对象Id是否有效，提前拦截问题：

[HttpPost]
public IActionResult Edit(Medication medication)
{
    if (!ModelState.IsValid || medication.Id == 0)
    {
        // 如果Id是临时值，返回错误或者重新返回表单
        ModelState.AddModelError("", "无法找到要编辑的药物记录");
        return View(medication);
    }

    try
    {
        _medicationRepository.UpdateMedication(medication, User.Identity.GetUserId());
        return RedirectToAction(nameof(Index));
    }
    catch (Exception ex)
    {
        ModelState.AddModelError("", "更新失败：" + ex.Message);
        return View(medication);
    }
}

4. 验证实体的主键配置 #

最后确认一下Medication实体的Id属性配置是否正确，确保它是数据库生成的主键：

public class Medication
{
    [Key]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public int Id { get; set; }

    // 其他属性...
    public string UserId { get; set; }
    public IdentityUser User { get; set; }
}

如果之前新增Medication时能正常生成Id，这个配置应该没问题，但更新时必须确保使用的是已经生成的Id值。

调试小技巧 #

在UpdateMedication方法开头打个断点，查看传入的medication.Id值——如果是0或者默认值，那肯定是前端没传过来，先检查视图的表单；如果Id是正确的，那再检查上下文的跟踪状态，看是不是实体被重复跟踪了。

内容的提问来源于stack exchange，提问作者Bite-us