Repository模式中，Update总是失败及其解析（转）

出处：http://www.cnblogs.com/scy251147/p/3688844.html

关于Entity Framework中的Attached报错的完美解决方案终极版

前发表过一篇文章题为《关于Entity Framework中的Attached报错的完美解决方案》，那篇文章确实能解决单个实体在进行更新、删除时Attached的报错，注意我这里说的单个实体，指的是要更新或删除的实体不包含其它实体（比如导航属性就包含其它实体），也就是简单POCO对象；但如果不是呢？那么那篇文章里的方法在一定程度上不起作用了，仍会报错，我开始也想不明白，明明通过IsAttached函数判断要更新的实体并未Attached,但进行Attaching时但仍然报错说有相同Key，开始还以为是MS的BUG，后经过多次反复调试发现，报错是对的，因为他报的错并不是我当前要更新的实体，而是该实体中关联的实体，代码与演示报错如下：(仅是演示代码)

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27

public class A {    public string a{get;set;}    public string b{get;set;}    public string c{get;set;}    public virtual B b{get;set;} }   public class B {    public string x{get;set;}    public string y{get;set;}    public string z{get;set;} }   var a1= dbContext.Set<A>().Single(); a1.a="test1"; dbContext.SaveChanges();   dbContext.Detach(a1);//从缓存中移除a1实体；     var a2= dbContext.Set<A>().AsNoTracking().Single(); a2.a="test2";  dbContext.Set<A>().Attach(a2); //报错，说B相同的KEY已经有Attached dbContext.Entry(entity).State = EntityState.Modified; dbContext.SaveChanges();

针对这个报错，我在想，为何查询实体A的时候能同时关联查询实体B并都同时Attached到内存中，而当我执行Detach实体A时，却没能关联Detach实体B，问题根源就在这里，知道这个原因了，现在就是要解决这个问题，如何解决呢？既然知道是Detach实体不全面造成的，那么我只需要获取到当前DbContext上下文对象中现有的所有已Attached实体，在执行完相应的CRUD时，再全部依次Detach掉即可，解决方案代码如下：

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

/// <summary> /// 清空DB上下文中所有缓存的实体对象 /// </summary> private void DetachedAllEntities() {     var objectContext = ((IObjectContextAdapter)this.baseContext).ObjectContext;     List<ObjectStateEntry> entries = new List<ObjectStateEntry>();     var states = new[] { EntityState.Added, EntityState.Deleted, EntityState.Modified, EntityState.Unchanged };     foreach (var state in states)     {         entries.AddRange(objectContext.ObjectStateManager.GetObjectStateEntries(state));     }       foreach (var item in entries)     {         objectContext.Detach(item.Entity);     } }     public void Commit()  //封装的统一提交方法 {     this.baseContext.SaveChanges();     this.DetachedAllEntities();//执行清除 }

在使用的时候配合之前那篇文章的IsAttached函数就能完美解决所有的Attached报错问题了！

=======================================================================================

在Repository模式中，我的Update方法总是无法更新实体，这个非常郁闷，Update方法如下：

看上去是没有任何问题的代码，一旦有实体更新的时候，总会出现如下的错误提示：

Attaching an entity of type 'TinyFrame.Data.DomainModel.t_user_application' failed because another entity of the same type already has the same primary keyvalue. This can happen when using the 'Attach' method or setting the state of an entity to 'Unchanged' or 'Modified' if any entities in the graph have conflicting key values. This may be because some entities are new and have not yet received database-generated key values. In this case use the 'Add' method or the 'Added' entity state to track the graph and then set the state of non-new entities to 'Unchanged' or 'Modified' as appropriate.

看字面意思，好像是我的EntityState设置不正确导致的，虽然我尝试过重新设置几次EntityState，但是仍旧无法解决我的问题。

然后实在找不出原因，就利用关键字 “ EF Repository Update ”在Google上面搜集，果然找到一篇文章：Advanced Entity Framework 6 Scenarios for an MVC 5 Web Application (12 of 12)，其中有一段话，提出了问题的所在：

This happened because of the following sequence of events:

    The Edit method calls the ValidateOneAdministratorAssignmentPerInstructor method, which retrieves all departments that have Kim Abercrombie as their administrator. That causes the English department to be read. As a result of this read operation, the English department entity that was read from the database is now being tracked by the database context.

    The Edit method tries to set the Modified flag on the English department entity created by the MVC model binder, which implicitly causes the context to try to attach that entity. But the context can't attach the entry created by the model binder because the context is already tracking an entity for the English department.

One solution to this problem is to keep the context from tracking in-memory department entities retrieved by the validation query. There's no disadvantage to doing this, because you won't be updating this entity or reading it again in a way that would benefit from it being cached in memory.

问题的原因如下：

在Context对象中，已经hold住了一个需要操作的对象，当我们把EntityState修改成modified的时候，Context会再次去加载那个操作对象，但是这样加载是无法成功的，因为当前已经存在一个对象了，再加载会导致重复，然后抛出失败的错误。

解决方法很简单，就是在展示列表的时候，利用AsNoTracking将Hold住的对象释放掉即可。我们修改代码如下：

然后提交，OK，问题解决。

=============================Update 2014.09.19======================

看到评论中有朋友虽然按照上述方法，但是仍然无法解决这一问题。原因是在Context中还保留有当前实体的副本所致，这里只要我们将实体副本从内存中完全移除，就可以了。

   //用于监测Context中的Entity是否存在，如果存在，将其Detach，防止出现问题。
        private Boolean RemoveHoldingEntityInContext(T entity)
        {
            var objContext = ((IObjectContextAdapter)_context).ObjectContext;
            var objSet = objContext.CreateObjectSet<T>();
            var entityKey = objContext.CreateEntityKey(objSet.EntitySet.Name, entity);

            Object foundEntity;
            var exists = objContext.TryGetObjectByKey(entityKey, out foundEntity);
          
            if (exists)
            {
                objContext.Detach(foundEntity);
            }

            return (exists);
        }

然后在Repository中，在进行更新和删除之前，运行一下即可：

  public T Remove(T entity)
        {
            try
            {
                RemoveHoldingEntityInContext(entity);

                _context.DbSet<T>().Attach(entity);
                return _context.DbSet<T>().Remove(entity);
            }
            catch (DbEntityValidationException dbex)
            {
                var msg = string.Empty;
                foreach (var validationErrors in dbex.EntityValidationErrors)
                    foreach (var validateionError in validationErrors.ValidationErrors)
                        msg += string.Format("属性:{0} 错误:{1}", validateionError.PropertyName, validateionError.ErrorMessage);
                var fail = new Exception(msg, dbex);
                throw fail;
            }
        }

        public T Update(T entity)
        {
            try
            {
                RemoveHoldingEntityInContext(entity);

                var updated = _context.DbSet<T>().Attach(entity);
                _context.DbContext.Entry(entity).State = EntityState.Modified;
                return updated;
            }
            catch (DbEntityValidationException dbex)
            {
                var msg = string.Empty;
                foreach (var validationErrors in dbex.EntityValidationErrors)
                    foreach (var validateionError in validationErrors.ValidationErrors)
                        msg += string.Format("属性:{0} 错误:{1}", validateionError.PropertyName, validateionError.ErrorMessage);
                var fail = new Exception(msg, dbex);
                throw fail;
            }
        }