Thực tiễn không tốt để lọc theo ID trong mẫu kho lưu trữ

Question

Tôi đang sử dụng ASP.NET MVC4 với Entity Framework 5.

Về cơ bản mọi kết quả hành động của bộ điều khiển lọc kết quả db bằng ID công ty của người dùng đã đăng nhập. Tôi vừa bắt đầu triển khai một mẫu kho lưu trữ để trả về các mô hình thay vì lọc trực tiếp DbContext từ bộ điều khiển. (Chuyển companyID vào kho lưu trữ để lọc kết quả của phương thức)

Tôi có cảm giác buồn cười là thực hành không tốt, nhưng không thể tìm thấy bất kỳ thông tin nào về chủ đề này. Tôi sẽ chèn một phiên bản cơ bản của mã hiện tại của tôi dưới đây, tôi sẽ đánh giá cao bất kỳ thông tin về việc có hay không nó là thực hành xấu, và tại sao như vậy.

IBookingSystemRepository.cs

public interface IBookingSystemRepository : IDisposable 
{ 
    IEnumerable<Appointment> GetAppointments(); 
    IEnumerable<Appointment> GetAppointments(bool includeDeleted); 
    IEnumerable<Client> GetClients(); 
    IEnumerable<Client> GetClients(bool includeDeleted); 
    void Save(); 
} 

BookingSystemRepository.cs

public class BookingSystemRepository : IBookingSystemRepository 
{ 
    private BookingSystemEntities db; 
    int CompanyID; 

    public BookingSystemRepository(BookingSystemEntities context, int companyID) 
    { 
     this.db = context; 
     this.CompanyID = companyID; 
    } 

    public IEnumerable<Appointment> GetAppointments() 
    { return GetAppointments(false); } 

    public IEnumerable<Appointment> GetAppointments(bool includeDeleted) 
    { 
     return includeDeleted 
      ? db.Appointments.Where(a => a.User.CompanyID == CompanyID) 
      : db.Appointments.Where(a => a.User.CompanyID == CompanyID && a.Deleted.HasValue); 
    } 

    public IEnumerable<Client> GetClients() 
    { return GetClients(false); } 

    public IEnumerable<Client> GetClients(bool includeDeleted) 
    { 
     return includeDeleted 
      ? db.Clients.Where(c => c.CompanyID == CompanyID) 
      : db.Clients.Where(c => c.CompanyID == CompanyID && c.Deleted.HasValue); 
    } 

    public void Save() 
    { 
     db.SaveChanges(); 
    } 

    public void Dispose() 
    { 
     if (db != null) 
      db.Dispose(); 
    } 
} 

TestController.cs

public class TestController : Controller 
{ 
    private BookingSystemEntities db = new BookingSystemEntities(); 

    public ActionResult AppointmentsList() 
    { 
     var user = db.Users.Single(u => u.Email == User.Identity.Name); 
     IBookingSystemRepository rep = new BookingSystemRepository(db, user.CompanyID); 
     return View(rep.GetAppointments()); 
    } 
} 

Thankyou trước để được hỗ trợ của bạn :)

Answer 1

Đây là ứng dụng nhiều người thuê. Việc lọc là cần thiết để giữ riêng từng dữ liệu của công ty. Cách tiếp cận của bạn là một cách âm thanh; nếu có thể, hãy cung cấp ngữ cảnh đã được lọc, thay vì lọc theo các phương thức kho lưu trữ hạ lưu riêng lẻ.