在存储库模式中通过ID进行筛选是不好的做法

我正在使用实体框架5的 ASP.NET MVC4 。

基本上，每个控制器操作结果都通过登录的用户公司ID来过滤数据库结果 。 我刚刚开始实现一个存储库模式来返回模型，而不是直接从控制器过滤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());
    }
}

Thankyou提前为您提供协助:)

---

这是一个多租户应用程序。 需要过滤以保持每个公司的数据不同。 你的方法是合理的; 如果可能，请提供已经过滤的上下文，而不是单独在下游存储库方法中进行过滤。

链接地址: http://www.djcxy.com/p/71999.html

上一篇: Is it bad practice to filter by ID within the repository pattern

下一篇: How to handle timeouts best?