Mã khuôn khổ thực thể đầu tiên xóa theo ID mà không tìm nạp (Kiểu chung)

Question

Hãy cho tôi biết đây có phải là cách tiếp cận tốt để xóa một thực thể mà không tìm nạp nó khi có ID.

tôi có một cửa hàng chung với giao diện như sau (tôi sẽ chỉ hiển thị Xoá):

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

Và trong lớp cửa hàng cụ thể của giao diện đó, đây là phương pháp xóa của tôi:

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

Tôi đã thử nghiệm cả việc tạo mới một Thực thể:

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

cũng như tìm nạp thực thể và sau đó gọi ng xóa.

Thông qua gỡ lỗi, tôi có ảnh hưởng mong muốn trên cả hai trường hợp.

Tôi chỉ muốn đảm bảo đây là thiết kế tốt và không có tác dụng phụ nào đối với phương pháp này.

Để tham chiếu, Ctx chỉ là chính DbContext.

Cảm ơn.

Answer 1

Đó là thiết kế tốt và không có tác dụng phụ :) (IMHO)

Hai nhận xét:

• Tôi đang tự hỏi nếu bạn có thể đơn giản hóa phương pháp Delete của bạn bằng cách:

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

  Tôi hy vọng rằng việc đặt trạng thái thành Deleted sẽ tự động đính kèm nếu đối tượng chưa được đính kèm. Nhưng tôi không chắc chắn nếu nó hoạt động. (Hãy cho tôi biết liệu nó có hoạt động cho các kịch bản đính kèm và tách rời (nếu bạn nên kiểm tra điều này).)

• Nếu bạn có ý tưởng tối ưu hóa hiệu suất (tránh tải các thực thể) đừng quên rằng, nếu có nhiều các đối tượng cần xóa trong ngữ cảnh, SaveChanges sẽ vẫn gửi một câu lệnh DELETE cho mỗi thực thể đến cơ sở dữ liệu. Xóa hàng loạt với EF là khá khủng khiếp trong hoạt động và nó là một địa hình nơi sẽ trở lại một câu lệnh SQL (DELETE ... WHERE ... IN ... nhiều ID ....) đôi khi có ý nghĩa (nếu hiệu suất vấn đề).