关于c＃：在存储库接口中使用可选参数会被视为糟糕的设计吗？

Would using optional parameters in a repository interface be considered bad design?

我创建了两个类来外部控制存储库上的页码(Page和排序(Order)。存储库公开了IEnumerable而不是IQueryable，因为我希望存储库具有强大的可测试性。

例如，如果我有以下存储库：

1
2
3
4

interface IUserRepository
{
public IEnumerable<User> GetAll();
}

如果我希望调用者控制响应的子集，他们将需要按页面/顺序传递。所以首先考虑的是添加重载：

1
2
3
4
5

interface IUserRepository
{
IEnumerable<User> GetAll();
IEnumerable<User> GetAllOrderedAndPaged(Order order, Page page);
}

我发现这个问题是，我必须创建多个重载，这是我发现非常繁琐的。

我想(但不确定)做的是让Order和Page作为可选参数：

1
2
3
4

interface IUserRepository
{
IEnumerable<User> GetAll(Order order = null, Page page = null);
}

这样，调用代码可以很容易地进行排序/分页，但不能处理主查询(这是我试图通过公开IEnumerable而不是IQueryable来避免的)。

这个设计看起来是好是坏？我知道这可能有点主观，但我想看看这在功能上是否有问题。主要的关键是，我的存储库是可测试的，调用者不能过多地使用/更改查询。我认为，由于排序/排序对于返回的数据集来说是非常常见的任务，为什么不将它们合并到存储库接口的设计中呢？同样，在测试时，我只关心返回正确的数据集，但是调用方可以说"现在，给我这个页面/数据的有序子集"。