【发布时间】:2010-10-27 05:33:16
【问题描述】:
我刚刚编写了几个分页扩展方法,我很想知道是否可以进行任何改进。
我对基本分页方法非常满意,您可以在其中提供页面大小和页码(如下所示)
public static IEnumerable<T> Paginate<T>(this IEnumerable<T> source, int pageSize, int pageNumber)
{
if (pageSize == 0) throw new ArgumentOutOfRangeException("pageSize");
if (pageNumber == 0) throw new ArgumentOutOfRangeException("pageNumber");
return source.Skip(pageSize * (pageNumber - 1)).Take(pageSize);
}
但我想知道是否有更好的方法来执行“自动”分页,它会返回 IEnumerable<IEnumerable<T>>
public static IEnumerable<IEnumerable<T>> Paginate<T>(this IEnumerable<T> source, int pageSize)
{
source.ThrowIfNull("source");
if (pageSize == 0) throw new ArgumentOutOfRangeException("pageSize");
var pageCount = (int)Math.Ceiling(source.Count() / (double)pageSize);
if (pageSize == 1)
pageCount = source.Count();
for (int i = 1; i <= pageCount; i++)
{
yield return source.Paginate(pageSize, i);
}
}
似乎有点怀疑必须迭代两次(一次用于计数,一次用于收益率返回。
有什么明显的方法可以改进这些方法吗?
【问题讨论】:
-
对于收益回报,您只需遍历页面即可。有一个 foreach 解决方法,但我怀疑它会更快,因为 foreach 本身更慢。如果您渴望性能,我宁愿改进 Paginate 方法。像 Skip() 和 Take() 这样的 IEnumerable 扩展方法可以手动编写,在这种特殊情况下它们会更快地工作。 (您必须从 X 循环到 Y 索引,而不是从 X 循环到结束,然后从 X 到 Y 循环,就像跳过和采取那样)