重新附加分离的实体会引发异常

Question

我正在为我的项目使用 Entity Framework 6（使用生成的模型和 DbContext）。我在我的程序中使用的架构是数据访问层是从应用程序中抽象出来的。这意味着我的实体将大部分处于分离状态。

似乎重新连接任何以前连接的实体似乎是不可能的（或者我没有正确执行）。

我已经单独测试了一个案例，并且能够缩小问题范围。步骤如下。

1. 从上下文中获取实体并释放上下文。这会分离实体。

   public async Task<ArchiveEntry> GetEntry(string api, string id)
   {
       // Omitted some code for simplicity
       ArchiveEntry entry;
   
       using (var db = new TwinTailDb())
       {
           entry = await db.ArchiveEntries.Where(a => a.Id == id).SingleOrDefaultAsync();
       }
   
       return entry;
   }
   

2. 无论是否更改，都保存实体。

   public async Task Save()
   {
       // Omitted some code for simplicity
       using (var db = new TwinTailDb())
       {
           db.ArchiveEntries.AddOrUpdate(a => a.Id, this);
           await db.SaveChangesAsync();
       }
   }
   

3. 最后，重新附加分离的实体。

   public async Task RevertChanges()
   {
       using (var db = new TwinTailDb())
       {
           if (db.Entry(this).State == EntityState.Detached && Id != 0)
           {
               db.ArchiveEntries.Attach(this);
               //await db.Entry(this).ReloadAsync(); // Commented since this is irrelevant
           }
       }
   }
   

4. 然后我运行这个函数来使用上面的代码。

   public async Task Test()
   {
       ArchiveEntry entry = await ArchiveService.GetEntry(null, "7");
       await entry.Save();
       await entry.RevertChanges();
   }
   

然后它抛出这个错误：

附加类型为“TwinTail.Entities.ArchiveEntry”的实体失败，因为同一类型的另一个实体已经具有相同的主键值。如果图中的任何实体具有冲突的键值，则在使用“附加”方法或将实体的状态设置为“未更改”或“已修改”时，可能会发生这种情况。这可能是因为某些实体是新实体，尚未收到数据库生成的键值。在这种情况下，使用“添加”方法或“已添加”实体状态来跟踪图形，然后将非新实体的状态设置为“未更改”或“已修改”。

这里有一点很重要。如果我跳过第 2 步，它不会引发异常。

我的猜测是实体被修改并保存在不同的上下文中。如果步骤 2 被跳过，实体保持不变，因此重新附加它不会造成问题（只是猜测）。然而，这个实体已经处于分离状态，所以这应该是无关紧要的。

另外一点，ChangeTracker 在这些步骤中不包含任何内容。此外，如果我对分离的实体执行任何上下文操作，它会抛出一个异常，说明它应该首先附加。我还注意到内部的 _entitywrapper 仍然有对旧上下文的引用。

所以，最后，问题来了。如何正确重新附加实体以及为什么会发生此异常。

我在另一个问题 (How to revert changes on detached entities) 中提出了类似的问题，但我觉得我需要发布一个新问题，因为这更笼统。

Answer 1

我在我的程序中使用的架构是数据访问层是从应用程序中抽象出来的。

看起来您正在ArchiveEntry 类本身上实现这些方法。这不是一个抽象的数据访问层，将实体传递给许多像这样的短期上下文会给你带来麻烦。

您应该将该代码放入与实体分开的类中，而不是为实体类提供自己的方法来管理持久性问题，并确保一旦实体附加到上下文，您将继续使用相同的上下文，直到它被处理掉了。释放上下文后，如果您想对实体执行其他操作，则应先从新上下文中检索它，然后再尝试对其执行任何（附加）操作。

Answer 2

我使用的是 AsNoTracking()，所以这个异常对我来说很奇怪。下面的代码解决了我的代码中的问题。我想我可以在 try 块中使用代码并且只使用 catch 块，但我从未测试过这个概念。

具有 PrimaryKey 的实体基类

public class EntityBase<K>
{
    [NotMapped]
    public virtual K PrimaryKey {get;}
}

更新功能

public static void Update<T,K>(DbContext context, T t) where T:EntityBase<K>
{
    DbSet<T> set = context.Set<T>();
    if(set==null) return;
    if(context.Entry<T>(t).State == EntityState.Detached)
    {
         try
         {
             T attached = set.Attached(t);
             context.Entry<T>(attached).State = EntityState.Modified;
         }catch(Exception ex)
         {
             T found = set.Find(t.PrimaryKey);
             context.Entry(found).CurrentValues.SetValues(t);
         }
     }
     context.SaveChanges();
}

扩展功能

public static void Update<T,K>(this DbContext context, T t) where T:EntityBase<K> => Update<T,K>(context, t);