这是一个 EF Core 错误还是我在更新一个实体时做错了？

Question

我将 Entity Framework Core 与存储库模式一起使用。为了帮助我，我用基本的 CRUD 方法编写了一个基础存储库。更新方法如下：

public void Update(TEntity entity)
{
    var contextEntry = _context.Entry<TEntity>(entity);
    if (contextEntry.State == EntityState.Dettached)
    {
        _context.Attach(entity);
    }
    contextEntry.State = EntityState.Modified;
    _context.SaveChanges();
}

鉴于包含此方法的 BaseRepository 类，我创建了一个继承自此的 User 存储库

public class UserRepository : BaseRepository<User>, IUserRepository
{
}

我已经在一个使用 ASP.NET Core 编码的 Web API 的 PUT 方法中使用了它

[HttpPut("~/api/users/{id}")]
public IActionResult Put(int id, [FromBody] User user)
{
    if (user == null || user.UserId != id)
    {
        return BadRequest();
    }

    userRepository.Update(user);
    return new NoContentResult();
}

现在，在发出请求时，_context.Attach(entity) 行出现一个错误。异常表示它无法添加要跟踪的实体，因为已经有另一个具有相同键的实体正在跟踪。

调试时我看到contextEntry.State 被设置为Unchanged。因此，它显然不等于EntityState.Dettached。尽管如此，执行还是进入了if 语句并尝试附加实体。

这里有些不对劲。这是一个错误吗？还是我做错了什么？我相信我是这个更新策略做错了什么的人，但我不确定。那么，我的方法有什么问题？

编辑：我将Update 方法更新为仅使用_context.Update(entity) 和_context.SaveChanges() 之后。尽管如此，_context.Update(entity) 还是会抛出一个 InvalidOperationException 并带有此消息：

附加信息：无法跟踪实体类型“用户”的实例，因为已经在跟踪具有相同键的该类型的另一个实例。添加新实体时，对于大多数键类型，如果没有设置键（即，如果键属性为其类型分配了默认值），则会创建一个唯一的临时键值。如果您为新实体显式设置键值，请确保它们不会与现有实体或为其他新实体生成的临时值发生冲突。附加现有实体时，请确保只有一个具有给定键值的实体实例附加到上下文。

Answer 1

您在项目中的某个位置从数据库中获取相同的实体，这就是它给您错误的原因。

在Update 方法中，您只需在上下文中添加实体，这就是您得到contextEntry.State Unchanged 的​​原因。

您可以通过两种方式解决此问题。

1. 从数据库中获取同一实体时，需要在同一实体上调用Detach方法。

2. 将您在Update 方法中收到的实体中的值复制到现有的上下文实体中，并将​​该实体保存在数据库中。

Answer 2

所有信息都在异常消息中...您已经拥有附加了该主键的实体的另一个副本。

我会推荐以下之一（首选）：

1. 为每个操作使用新的上下文，没有长期存在的存储库/上下文
2. 使用主键在您的上下文中使用.Set<TEntity>.Find(object[] key)，以便检索您已有的任何实体。
3. 在您当前的更新方法中，使用Set<TEntity>.Local.Find(..) 检查它是否已经存在