c# - 删除前检查实体状态是否被删除的目的

Question

    public virtual void Delete(T entity)
    {
        DbEntityEntry dbEntityEntry = DbContext.Entry(entity);
        if (dbEntityEntry.State != EntityState.Deleted)
        {
            dbEntityEntry.State = EntityState.Deleted;
        }
        else
        {
            DbSet.Attach(entity);
            DbSet.Remove(entity);
        }
    }

当然这个方法和这个是一样的:

    public virtual void Delete(T entity)
    {
        DbEntityEntry dbEntityEntry = DbContext.Entry(entity);
        if (dbEntityEntry.State == EntityState.Deleted)
        {
            DbSet.Attach(entity); 
            DbSet.Remove(entity);  // Why to remove here??
        }
        else
        {
            // And why to set as Deleted here??
            dbEntityEntry.State = EntityState.Deleted; 
        }
    }

我在 this 中看到了上面的代码文章。恕我直言，此代码的某些部分是多余的。 if 语句 的目的是什么？ else 语句 的目的是什么？正如我所测试的，DbSet.Remove(entity); 和 dbEntityEntry.State = EntityState.Deleted 之间的区别在于，所有引用的行也在第一个代码中被删除，但是在第二个中，EF 将抛出有关对此行具有 FK 的行的异常。

Answer 1

当您调用 DbSet.Remove(entity)在 EF 6.X 中，您实际上使用的是 ObjectContext.DeleteObject(entity) .这是在 InternalSet<TEntity> 中调用的代码类:

public virtual void Remove(object entity)
{
    DebugCheck.NotNull(entity);
    if (!(entity is TEntity))
    {
      throw Error.DbSet_BadTypeForAddAttachRemove("Remove", entity.GetType().Name, typeof(TEntity).Name);
    }
    InternalContext.DetectChanges();
    InternalContext.ObjectContext.DeleteObject(entity);
}

如果依赖实体的外键不可为空，默认情况下，当您配置关系时，Code First 会在关系上设置级联删除，因此，当您调用 Remove 时方法，它还会删除约束关系中的所有子对象(EF 将为父实体和相关实体生成删除语句)。如果依赖实体上的外键可以为空，Code First 不会在关系上设置级联删除，当主体被删除时，外键将设置为 null .

现在，当您使用第二个变体将父实体的状态更改为 Deleted 时( bEntityEntry.State = EntityState.Deleted )，EF 将整个实体图附加到具有指定状态的父实体和子实体的上下文中，根据您拥有的关系类型，可能会发生三件事:

• 如果关系是可选，即数据库中从子项到父项的外键允许NULL值，这个外来的将被设置为空，如果你调用 SaveChanges这个NULL childEntity 的值将写入数据库(即删除两者之间的关系)。 SQL UPDATE 会发生这种情况陈述。没有 DELETE语句发生。

• 如果关系是必需的(FK 不允许 NULL 值)并且关系是未识别(这意味着外键不是 child 的(复合)主键的一部分)您必须将 child 添加到另一个 parent 或您必须明确删除 child (然后使用 DeleteObject)。如果您不执行任何这些操作，则会违反引用约束，并且 EF 将在您调用 SaveChanges 时抛出异常。 - 臭名昭著的“由于一个或多个外键属性不可为空”异常或类似异常无法更改。

• 如果关系是identifying(它必然是required那么因为主键的任何部分都不能是NULL)EF会标记childEntity作为Deleted以及。如果您调用 SaveChanges SQL DELETE语句将被发送到数据库。如果数据库中没有其他引用约束被违反，实体将被删除，否则抛出异常。

更多信息，你可以查看这个post和 this也是。

所以，恕我直言，最好创建一个通用的 Delete方法是这样的:

public virtual void Delete(T entity)
{
   //Attach the entity to your context
   DbSet.Attach(entity);
   // Remove the current entity and the related entities in case of your relationship was configured with cascade delete
   DbSet.Remove(entity); 
}

更新

我看不出作者使用这两种变体的意义。通常当你创建一个 Remove 时通用方法使用这两个变体之一，而不是同时使用。如果您使用 DbSet.Remove我看不出检查状态有什么意义，此方法将为您完成这项工作。如果您使用第二个变体，我会创建一个这样的方法:

public virtual void Delete(T entity)
{
    // An Added entity does not yet exist in the database. If it is then marked as deleted there is
    // nothing to delete because it was not yet inserted, so just make sure it doesn't get inserted.
   dbEntityEntry.State == EntityState.Added
                ? EntityState.Detached
                : EntityState.Deleted); 
}