1

我正在创建一个函数，该函数将采用一些 IEnumerable、进行分组、排序、采用一些前 N 个元素并返回这些元素的列表。以后它可能会做更多，这就是为什么我想把它变成一个函数而不是直接使用 LINQ。

我依靠匿名委托来指定 T 类型的哪些成员将用于对集合进行分组和排序。

    public IEnumerable<T> GetList(IEnumerable<T> collection, Func<T, object> groupBy, Func<T, object> orderBy, int howMany)
    {
        var group = collection
            .GroupBy(groupBy)
            .Select(x => x.OrderBy(orderBy).Take(howMany))
            .Aggregate((l1, l2) => l1.Concat(l2));

        return group.ToList();
    }

并像这样使用：

        new CollectionGroupPicker<NumericDomainObject>().GetList(list, x => x.GroupableField, x => x.OrderableField, 2).ToList();

我的问题是 - 有没有更好的方法来传递 TI 类型的哪个成员将用于分组和排序？我在这里使用对象，但有更好的方法吗？

4

2 回答 2

2

我同意 Samuel 使用通用参数类型。另外，为什么不使用SelectMany扁平化结果，或者使其成为扩展方法？

static class GetListExtensionMethods
{
    public static IEnumerable<T> GetList<T, TGroupingField, TOrderingField>(this IEnumerable<T> collection, Func<T, TGroupingField> groupBy, Func<T, TOrderingField> orderBy, int howMany)
    {
        var group = collection
            .GroupBy(groupBy)
            .SelectMany(x => x.OrderBy(orderBy).Take(howMany));
        return group.ToList();
    }
}
于 2013-08-15T20:56:22.873 回答
2

object您应该指定组并选择键作为通用参数，而不是指定。它们的类型将根据使用情况自动推断，调用者可以指定具有任何返回类型的 lambda。

public IEnumerable<T> GetList<TGroupKey, TOrderKey>(IEnumerable<T> collection, 
                                                    Func<T, TGroupKey> groupBy, 
                                                    Func<T, TOrderKey> orderBy, 
                                                    int howMany)
{
    var group = collection
        .GroupBy(groupBy)
        .Select(x => x.OrderBy(orderBy).Take(howMany))
        .Aggregate((l1, l2) => l1.Concat(l2));
    return group.ToList();
}
于 2013-08-15T20:08:54.403 回答