我的 Entity Framework 存储库和服务层方法应返回哪些类型：List、IEnumerable、IQueryable？

Question

我有一个具体的存储库实现，它返回实体的 IQueryable：

public class Repository
{
    private AppDbContext context;

    public Repository()
    {
        context = new AppDbContext();
    }


    public IQueryable<Post> GetPosts()
    {
        return context.Posts;
    }
}

然后，我的服务层可以根据其他方法（位置、分页等）的需要执行 LINQ

现在我的服务层设置为返回 IEnumerable：

public IEnumerable<Post> GetPageOfPosts(int pageNumber, int pageSize)
{ 
    Repository postRepo = new Repository();

    var posts = (from p in postRepo.GetPosts()  //this is IQueryable
                orderby p.PostDate descending
                select p)
                .Skip((pageNumber - 1) * pageSize)
                .Take(pageSize);

    return posts;
}

这意味着如果我想绑定到中继器或其他控件，我必须在我的代码隐藏中执行 ToList()。

这是处理返回类型的最佳方式，还是我需要在从服务层方法返回之前转换为列表？

Answer 1

您的返回类型应始终在继承层次结构中尽可能高（或者我应该将其写为低，如果基础朝向底部） .如果你所有的方法都需要IQueryable<T>，那么所有的返回值都应该放弃那个类型。

也就是说，IEnumerable<T> 有一个方法 (AsQueryable())，您可以调用它来实现（我认为是）所需的结果。

Answer 2

这两种方法都是可能的，只是选择问题。

一旦你使用IQueryable，你就有了一个简单的存储库，它在大多数情况下都可以工作，但它的可测试性更差，因为在IQueryable 上定义的查询是 linq-to-entities。如果你模拟存储库，它们是单元测试中的 linq-to-objects = 你没有测试你的真实实现。您需要集成测试来测试您的查询逻辑。

一旦您使用IEnumerable，您将拥有非常复杂的存储库公共接口——您需要为每个需要在存储库中公开特殊查询的实体提供特殊的存储库类型。这种存储库在存储过程中更为常见——存储库中的每个方法都映射到单个存储过程。这种类型的存储库提供了更好的关注点分离和更少的抽象泄漏，但同时它消除了很多 ORM 和 Linq 的灵活性。

最后，您可以采用组合方法，其中方法为最常见的场景（更经常使用的查询）返回 IEnumerable，另一种方法为罕见或复杂的动态构建查询公开 IQueryable。

编辑：

如 cmets 中所述，使用 IQueryable 有一些副作用。当您公开 IQueryable 时，您必须保持上下文活动，直到您执行查询 - IQueryable 以与 IEnumerable 相同的方式使用延迟执行，因此除非您调用 ToList、First 或其他执行您的查询的函数仍然需要您的上下文。

实现这一点的最简单方法是在存储库中使用一次性模式 - 在其构造函数中创建上下文并在存储库释放时释放它。然后您可以使用using 块并在其中执行查询。这种方法适用于非常简单的场景，您对每个存储库的单个上下文感到满意。更复杂（和常见）的场景需要在多个存储库之间共享上下文。在这种情况下，您可以使用上下文提供者/工厂（一次性）之类的东西并将工厂注入存储库构造函数（或允许提供者创建存储库）。这导致了 DAL 层工厂和自定义工作单元。

Answer 3

您的问题的另一个词似乎需要确定AppDbContext 的处置时间或位置。

如果你不dispose，也就是应用退出的时候就dispose，返回IEnumerable/IQueryable没问题，没有实际数据。但是，在处置 AppDbContext 之前，您需要将类型作为 IList 返回，并包含实际数据。

更新： 尽管您已经知道，但我认为您需要了解以下代码含义。

//outside of this code is refered to your code.

//Returning IEnumerable could be used outside this scope if AppDbContext is ensured no disposing
public IEnumerable<Post> GetIEnumerableWithoutActualData()
{
    return context.Posts;
}

//Even if AppDbContext is disposed, IEnumerable could be used.
public IEnumerable<Post> GetIEnumerableWithActualData()
{
    return context.Posts.ToList();
}