71

使用带有通用存储库模式的 EF5 和 ninject 进行依赖注入，并在尝试使用我的 edmx 存储的过程将实体更新到数据库时遇到问题。

我在 DbContextRepository.cs 中的更新是：

public override void Update(T entity)
{
    if (entity == null)
        throw new ArgumentException("Cannot add a null entity.");

    var entry = _context.Entry<T>(entity);

    if (entry.State == EntityState.Detached)
    {
        _context.Set<T>().Attach(entity);
        entry.State = EntityState.Modified;
    }
}

从返回到我的存储库的 AddressService.cs 中，我有：

 public int Save(vw_address address)
{
    if (address.address_pk == 0)
    {
        _repo.Insert(address);
    }
    else
    {
        _repo.Update(address);
    }

    _repo.SaveChanges();

    return address.address_pk;
}

当它遇到 Attach 和 EntityState.Modified 时，它会出现错误：

ObjectStateManager 中已存在具有相同键的对象。ObjectStateManager 无法跟踪具有相同键的多个对象。

我已经查看了堆栈和互联网上的许多建议，但没有提出任何解决它的方法。任何变通方法将不胜感激。

谢谢！

4

10 回答 10

125

编辑：使用原始答案Find而不是Local.SingleOrDefault. 它与@Juan 的Save方法结合使用，但可能会导致对数据库的不必要查询，并且else部分可能永远不会执行（执行 else 部分会导致异常，因为 Find 已经查询了数据库并且没有找到实体，因此无法更新） . 感谢@BenSwayne 发现问题。

您必须检查具有相同键的实体是否已被上下文跟踪并修改该实体而不是附加当前实体：

public override void Update(T entity) where T : IEntity {
    if (entity == null) {
        throw new ArgumentException("Cannot add a null entity.");
    }

    var entry = _context.Entry<T>(entity);

    if (entry.State == EntityState.Detached) {
        var set = _context.Set<T>();
        T attachedEntity = set.Local.SingleOrDefault(e => e.Id == entity.Id);  // You need to have access to key

        if (attachedEntity != null) {
            var attachedEntry = _context.Entry(attachedEntity);
            attachedEntry.CurrentValues.SetValues(entity);
        } else {
            entry.State = EntityState.Modified; // This should attach entity
        }
    }
}

如您所见，主要问题是该SingleOrDefault方法需要知道找到实体的密钥。您可以创建简单的接口来公开密钥（IEntity在我的示例中）并在您想要以这种方式处理的所有实体中实现它。

于 2012-09-25T17:03:15.763 回答
8

我不想通过添加接口或属性来污染我自动生成的 EF 类。所以这真的是上述一些答案的一点点（所以归功于Ladislav Mrnka）。这为我提供了一个简单的解决方案。

我在更新方法中添加了一个函数来找到实体的整数键。

public void Update(TEntity entity, Func<TEntity, int> getKey)
{
    if (entity == null) {
        throw new ArgumentException("Cannot add a null entity.");
    }

    var entry = _context.Entry<T>(entity);

    if (entry.State == EntityState.Detached) {
        var set = _context.Set<T>();
        T attachedEntity = set.Find.(getKey(entity)); 

        if (attachedEntity != null) {
            var attachedEntry = _context.Entry(attachedEntity);
            attachedEntry.CurrentValues.SetValues(entity);
        } else {
            entry.State = EntityState.Modified; // This should attach entity
        }
    }
}

然后当你调用你的代码时，你可以使用..

repository.Update(entity, key => key.myId);
于 2014-02-28T12:22:30.717 回答
7

您实际上可以通过反射检索 Id，请参见下面的示例：

        var entry = _dbContext.Entry<T>(entity);

        // Retreive the Id through reflection
        var pkey = _dbset.Create().GetType().GetProperty("Id").GetValue(entity);

        if (entry.State == EntityState.Detached)
        {
            var set = _dbContext.Set<T>();
            T attachedEntity = set.Find(pkey);  // access the key
            if (attachedEntity != null)
            {
                var attachedEntry = _dbContext.Entry(attachedEntity);
                attachedEntry.CurrentValues.SetValues(entity);
            }
            else
            {
                entry.State = EntityState.Modified; // attach the entity
            }
        }
于 2013-02-20T07:18:01.517 回答
2

@ serj-sagan你应该这样做：

**请注意，YourDb 应该是从 DbContext 派生的类。

public abstract class YourRepoBase<T> where T : class
{
    private YourDb _dbContext;
    private readonly DbSet<T> _dbset;

    public virtual void Update(T entity)
    {
        var entry = _dbContext.Entry<T>(entity);

        // Retreive the Id through reflection
        var pkey = _dbset.Create().GetType().GetProperty("Id").GetValue(entity);

        if (entry.State == EntityState.Detached)
        {
           var set = _dbContext.Set<T>();
           T attachedEntity = set.Find(pkey);  // access the key
           if (attachedEntity != null)
           {
               var attachedEntry = _dbContext.Entry(attachedEntity);
               attachedEntry.CurrentValues.SetValues(entity);
           }
           else
           {
              entry.State = EntityState.Modified; // attach the entity
           }
       }
    }

}

于 2013-10-31T14:46:29.660 回答
1

如果您将上下文设置为 AsNoTracking() 这将停止 aspmvc 跟踪内存中实体的更改（无论如何这都是您在网络上想要的）。

_dbContext.Products.AsNoTracking().Find(id);

我建议您在http://www.asp.net/mvc/tutorials/getting-started-with-ef-using-mvc/advanced-entity-framework-scenarios-for-an-mvc-web上阅读更多相关信息-应用

于 2014-04-23T17:52:44.403 回答
1

另一个解决方案（基于@Sergey 的回答）可能是：

private void Update<T>(T entity, Func<T, bool> predicate) where T : class
{
    var entry = Context.Entry(entity);
    if (entry.State == EntityState.Detached)
    {
        var set = Context.Set<T>();
        T attachedEntity = set.Local.SingleOrDefault(predicate); 
        if (attachedEntity != null)
        {
            var attachedEntry = Context.Entry(attachedEntity);
            attachedEntry.CurrentValues.SetValues(entity);
        }
        else
        {
            entry.State = EntityState.Modified; // This should attach entity
        }
    }
}

然后你会这样称呼它：

Update(EntitytoUpdate, key => key.Id == id)
于 2014-04-24T19:04:52.190 回答
0

如果没有反射并且不想使用接口，可以使用功能委托在数据库中查找实体。这是上面的更新示例。

private void Update<T>(T entity, Func<ObservableCollection<T>, T> locatorMap) where T : class
{
    var entry = Context.Entry(entity);
    if (entry.State == EntityState.Detached)
    {
        var set = Context.Set<T>();
        T attachedEntity = locatorMap(set.Local); 

        if (attachedEntity != null)
        {
            var attachedEntry = Context.Entry(attachedEntity);
            attachedEntry.CurrentValues.SetValues(entity);
        }
        else
        {
            entry.State = EntityState.Modified; // This should attach entity
        }
    }
}

你可以这样称呼它：

Update(EntitytoUpdate, p => p.SingleOrDefault(a => a.Id == id))
于 2013-12-15T03:46:16.463 回答
0

分离找到的实体（参见attachedEntityLadislav的解决方案）并重新附加修改后的实体对我来说效果很好。

这背后的原因很简单：如果某些东西是不可变的，那么将它（作为一个整体，实体）从它所属的位置替换为所需的那个。

以下是如何执行此操作的示例：

var set = this.Set<T>();
if (this.Entry(entity).State == EntityState.Detached)
{
    var attached = set.Find(id);
    if (attached != null) { this.Entry(attached).State = EntityState.Detached; }
    this.Attach(entity);
}

set.Update(entity);

当然，人们可能很容易发现这个片段是泛型方法的一部分，因此使用了T作为模板参数的 和Set<T>()。

于 2017-07-13T06:38:27.210 回答
-2

上面的答案可能是 EF 4.1+。对于那些使用 4.0 的人，试试这个简单的方法……没有经过真正的测试，但确实附加并保存了我的更改。

    public void UpdateRiskInsight(RiskInsight item)
    {
        if (item == null)
        {
            throw new ArgumentException("Cannot add a null entity.");
        }

        if (item.RiskInsightID == Guid.Empty)
        {
            _db.RiskInsights.AddObject(item);
        }
        else
        {
            item.EntityKey = new System.Data.EntityKey("GRC9Entities.RiskInsights", "RiskInsightID", item.RiskInsightID);
            var entry = _db.GetObjectByKey(item.EntityKey) as RiskInsight;
            if (entry != null)
            {
                _db.ApplyCurrentValues<RiskInsight>("GRC9Entities.RiskInsights", item);
            }

        }

        _db.SaveChanges();

    }
于 2012-12-28T03:46:05.557 回答
-3

你可能忘记fBLL = new FornecedorBLL();在 algun 地方instaçia 对象

于 2016-08-24T23:00:39.660 回答