リポジトリパターン内のIDでフィルタするのは悪い習慣です

Question

私はASP.NET MVC4とエンティティフレームワーク5を使用しています。

本質的にすべてのコントローラーアクション結果は、ログインしたユーザーの企業IDによってdb結果をフィルターに掛けます。私はちょうどコントローラからDbContextを直接フィルタリングするのではなく、モデルを返すためのリポジトリパターンを実装し始めました。 （メソッドの結果をフィルタリングするためにcompanyIDをリポジトリに渡す）

これを行うことは悪い習慣であるが、対象に関する情報を見つけることができないという面白い気持ちがある。私は以下の私の現在のコードの基本バージョンを挿入します、それは悪い習慣であるかどうかについての情報と、そういう理由で感謝します。

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

事前にお手伝いいただきありがとうございます。

Answer 1

マルチテナントアプリケーションです。各企業のデータを分離しておくためには、フィルタリングが必要です。あなたのアプローチは健全なものです。可能であれば、下流のリポジトリメソッドを個別にフィルタリングするのではなく、すでにフィルタリングされているコンテキストを提供してください。