实体框架存储库模式为什么不返回Iqueryable？

Question

有几个很好的博客关于如何使用泛型类实现存储库模式和工作单元模式。

使用实体框架6.1实现数据访问层

实现存储库和工作单元模式

这个想法是，定义一个通用接口IRepository和一个隐藏数据实际访问方式的类存储库。可以使用实体框架DbContext来访问它，或者存储库可能是用于单元测试的内存集合。

public interface public interface IRepository<T> where T : class
{
    T GetById(int Id);
    void DeleteById(int Id);

    void Add(T entity);
    void Update(T entity);

    etc.
}

我经常看到添加了几个与Queryable和/或Enumerable函数类似的Query函数。

例如，在实现数据访问层中，我看到：

/// Returns an IEnumerable based on the query, order clause and the properties included
/// <param name="query">Link query for filtering.</param>
/// <param name="orderBy">Link query for sorting.</param>
/// <param name="includeProperties">Navigation properties seperated by comma for eager loading.</param>
/// <returns>IEnumerable containing the resulting entity set.</returns>
IEnumerable<T> GetByQuery(Expression<Func<T, bool>> query = null, Func<IQueryable<T>, IOrderedQueryable<T>> orderBy = null, string includeProperties = "");

/// <summary>
/// Returns the first matching entity based on the query.
/// </summary>
/// <param name="predicate"></param>
/// <returns></returns>
T GetFirst(Expression<Func<T, bool>> predicate);

如果该接口有一个函数IQueryable GetQuery()，那么我就不必创建像GetFirst()和GetByQuery()这样的函数。

问题:为什么不推荐这样做？人们会以不可取的方式更改数据吗？

Answer 1

我们使用存储库模式的原因之一是封装胖查询。这些查询使得很难在 ASP.NET MVC 控制器中读取、理解和测试操作。此外，随着应用程序的增长，在多个位置重复胖查询的机会也会增加。使用存储库模式，我们将这些查询封装在存储库类中。结果是更纤薄、更清洁、更易于维护且更易于测试的操作。请考虑以下示例：

var orders = context.Orders
    .Include(o => o.Details)
        .ThenInclude(d => d.Product)
    .Where(o => o.CustomerId == 1234);

在这里，我们直接使用没有存储库模式的 DbContext。当你的存储库方法返回IQueryable时，其他人将获取IQueryable并在其上编写查询。结果如下：

var orders = repository.GetOrders()
    .Include(o => o.Details)
        .ThenInclude(d => d.Product)
    .Where(o => o.CustomerId == 1234);

你能看出这两段代码的区别吗？唯一的区别在第一行。在第一个例子中，我们使用上下文。订单，在第二个我们使用仓库。GetOrders()。那么，这个存储库在解决什么问题呢？没什么！

您的存储库应该返回域对象。因此，GetOrders（）方法应该返回一个IENumable。有了这个，第二个示例可以重写为：

var orders = repository.GetOrders(1234);

看到区别了吗？检索自Hamedani先生的博客

Answer 2

不建议这样做，因为它会使存储库模式无效。此模式的目的是通过抽象使您的DAL实现与您的其他项目分开。

本质上，返回IQueryable将返回TSQL语句，而不是结果，这意味着引用DAL的任何项目都需要额外的EF引用才能执行查询。这种“数据泄漏”会使您的项目更加紧密，从而与关注点分离原则相矛盾。

您可以在此处阅读有关存储库模式及其好处的更多信息：http://www.codeproject.com/Articles/526874/Repositorypluspattern-cplusdoneplusright