通用实体保存方法不起作用？

Question

在试图减少代码和班级我用我一直在考虑这个方法量：

public TEntity Save(TEntity entity) 
      { 
       var validatationErrors = Entities.GetValidationErrors().ToList(); 
       if (validatationErrors.Count() > 0) 
        throw new ValidateException(validatationErrors); 

       if (Entities.Entry(entity).State == EntityState.Added) 
        Entities.Set<TEntity>().Add(entity); 
       else 
        Entities.Set<TEntity>().Attach(entity); 

       Entities.SaveChanges(); 
       Entities.Entry(entity).Reload(); 
       return entity; 
      } 

当我创建一个新的产品但它跳过了条件，将它添加到数据库中。因此没有得到保存。同样，如果我尝试修改它不会更改的内容。

为了解决这个问题我要救我的实体在控制器中，像这样：

using (var db = new EntitiesDbContext()) 
       { 
        db.Products.Attach(product); 
        db.Entry(product).State = EntityState.Modified; 
        db.SaveChanges(); 
       } 

这似乎是相当低效。 所以我想知道真的是什么问题的通用保存，使其无法正常工作？

Answer 1

如果你创建一个新的实体，你应该使用.Add方法。 。如果你有一个你知道它存在于数据库中的对象，并且你想避免访问数据库来获取它，应该使用附件。

而且一般的保存方法似乎有几个问题：

• 除非您禁用验证明确你会不止一次验证实体，因为在默认情况下.SaveChanges（）如果实体将验证实体
• 处于Added状态，它已被上下文跟踪，因此不需要再次添加它
• .SaveChanges（）将尝试保存上下文跟踪的所有实体不处于未更改状态。该方法使一个印象是，它只会保存您通过实体，但实际上它可能保存超过该
• 我想调用DbEntityEntry.Reload（）才有用连接实体从数据库值更新属性值 - 如果你总是这样做，那么我想知道你为什么附加实体而不是从数据库中获取它。

Answer 2

• 你的代码是错误的。在你的情况下，你不需要Attach你的实体。你只需要你的实体Add您DbSet：

   public TEntity Save(TEntity entity) 
       { 
        Entities.Set<TEntity>().Add(entity);   
        Entities.SaveChanges(); 
       } 
  

你应该记住：

Entities.Set<TEntity>().Add(entity);

等于

db.Entry(entity).State = EntityState.Added;

和

entity.Property = newProperty;

集实体状态Modified（db.Entry(entity).State为EntityState.Modified）

而且你也不需要验证和重新加载实体。 ObjectContext为你做。

• 中来控制你的实体尝试这个办法：

  public TEntity Save(TEntity entity) 
      { 
  
       var dbEntity = Entities.Set<TEntity>().Find(entity.Id); 
  
       if (dbEntity != null)    
        dbEntity = entity; 
       else 
        Entities.Set<TEntity>().Add(entity); 
  
       Entities.SaveChanges(); 
      }