通用實體保存方法不起作用？

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(); 
      }