实体框架代码首先按 ID 删除而不获取（通用样式）

Question

如果我有 ID，请告诉我这是否是一种在不获取实体的情况下删除实体的不错方法。

我有一个具有以下界面的通用商店（我只会显示删除）：

public interface IStore : IReadOnlyStore
{
    void Delete<TEntity>(TEntity entity) where TEntity : class, IEntity, new();
    void SaveChanges();
}

在该接口的具体 Store 类中，这是我的删除方法：

public void Delete<TEntity>(TEntity entity) where TEntity : class, IEntity, new()
{
     var obj = Ctx.Entry(entity);
     if (obj.State ==  System.Data.EntityState.Detached)
     {
         Ctx.Set(typeof(TEntity)).Attach(obj.Entity);
     }
     Ctx.Set(typeof(TEntity)).Remove(obj.Entity);
}

我已经测试了两个新实体：

Store.Delete(new Foo() { Id = request.Entity.Id });

以及获取实体然后调用删除。

通过调试，我对这两种情况都有预期的效果。

我只是想确保这是一个好的设计并且这种方法没有副作用。

作为参考，Ctx 只是 DbContext 本身。

谢谢。

Answer 1

这是很好的设计，没有副作用:)（恕我直言）

两个备注：

• 我想知道您是否可以通过以下方式简化您的 Delete 方法：

  public void Delete<TEntity>(TEntity entity) 
      where TEntity : class, IEntity, new()
  {
      Ctx.Entry(entity).State = EntityState.Deleted;
  }
  

  如果实体尚未附加，我希望将状态设置为Deleted 会自动附加。但我不确定它是否有效。 （让我知道它是否适用于附加和分离的场景（如果你应该对此进行测试）。

• 如果您考虑到性能优化（避免加载实体），请不要忘记，如果上下文中有多个实体要删除，SaveChanges 仍将向每个实体发送一个单独的 DELETE 语句数据库。使用 EF 进行批量删除在性能上非常糟糕，并且在这种情况下，返回 SQL 语句（DELETE ... WHERE ... IN ... 许多 ID ....）有时是有意义的（如果性能很重要）。