如何优化此 LINQ 扩展方法

Question

我编写了一个自定义排序 LINQ 扩展方法，如下所示，但我认为它可以针对大型结果进行优化。

这是代码：

    public static IEnumerable<T> OrderByAncesty<T>(this IEnumerable<T> source, Func<T, DateTime> dateSelector, Func<T, float> scoreSelector)
    {
        var original = source.ToList();
        var maxDate = source.Max(dateSelector);
        var list = from p in original
                   let date = dateSelector(p)
                   let score = scoreSelector(p)
                   let date1 = date.ToOADate()
                   let date2 = maxDate.ToOADate()
                   let ancesty = (1 - (float)date1 / (float)date2) * score
                   select new
                   {
                       TObject = p,
                       Ancesty = ancesty
                   };
        return list.OrderBy(p => p.Ancesty).Select(p => p.TObject);
    }

Answer 1

每个“let”子句都增加了额外的授权级别。您可以通过删除它们来稍微改进一些东西。您也不需要匿名类型 - 或者很可能不需要 ToList() 调用。此外，每次都在 maxDate 上调用 ToOADate() 是没有意义的。

public static IEnumerable<T> OrderByAncesty<T>(this IEnumerable<T> source, 
    Func<T, DateTime> dateSelector, Func<T, float> scoreSelector)
{
    var maxDate = (float) source.Max(dateSelector).ToOADate();
    return original.OrderBy(p =>  
              (1 - (float)dateSelector(p).ToOADate() / maxDate))
              * scoreSelector(p));
}

请注意，如果没有“let”子句，它就很清楚了。