如何将LINQ的OrderBy方向指定为布尔值？

Question

我有一个具有此签名的简单数据类:

internal interface IMyClass {
    string Letter { get; }
    int Number { get; }
}

我希望能够根据字段(指定为string sortField)和方向(指定为bool isAscending)对此数据进行排序

目前我使用的是switch(在每种情况下都有升序逻辑if)

IEnumerable<IMyClass> lst = new IMyClass[];//provided as paramater
switch (sortField)
{
    case "letter":
        if( isAscending ) {
            lst = lst.OrderBy( s => s.Letter );
        } else {
            lst = lst.OrderByDescending( s => s.Letter );
        }
        break;
    case "number":
        if( isAscending ) {
            lst = lst.OrderBy( s => s.Number );
        } else {
            lst = lst.OrderByDescending( s => s.Number );
        }
        break;
}

对于2个属性来说,这非常难看,但是当排序逻辑不同时,它就成了一个问题(我们也看到s => s.Number在代码中重复了两次)

问题 传递布尔值以选择排序方向的最佳方法是什么？

我试过的 我已经拆开了System.Core.dll并找到了OrderBy Extension方法实现:

排序依据:

public static IOrderedEnumerable<TSource> OrderBy<TSource, TKey>(
        this IEnumerable<TSource> source, 
        Func<TSource, TKey> keySelector
    ){

    return new OrderedEnumerable<TSource, TKey>(
        source, 
        keySelector, 
        null, 
        false
    );
}

OrderByDescending:

public static IOrderedEnumerable<TSource> OrderByDescending<TSource, TKey>(
        this IEnumerable<TSource> source, 
        Func<TSource, TKey> keySelector
    ){
        return new OrderedEnumerable<TSource, TKey>(
            source, 
            keySelector, 
            null, 
            true
        );
}

似乎有两个命名方法的目的是抽象这个布尔值.我无法轻松创建自己的扩展,因为OrderedEnumberableSystem.Core是内部的,并且写一个图层来自bool - > methodName - > bool对我来说似乎不对.

Answer 1

我想说你自己的扩展方法:

public static IEnumerable<T> Order<T, TKey>(this IEnumerable<T> source, Func<T, TKey> selector, bool ascending)
{
    if (ascending)
    {
        return source.OrderBy(selector);
    }
    else
    {
        return source.OrderByDescending(selector);
    }
}

然后你可以写:

lst = lst.Order( s => s.Letter, isAscending );

至于指定方法名称:我希望这不作为一个cop-out答案,但我认为你应该坚持使用选择器函数而不是传入一个字符串.去字符串路由并没有真正省去任何打字或提高清晰度("letter"真的比它更快或更清晰s => s.Letter？)并且只会使你的代码变得更胖(你需要保持从字符串到选择器函数的某种映射或写入自定义解析逻辑在它们之间进行转换)并且可能更脆弱(如果你采用后一种方式,那么错误的可能性非常高).

如果您的目的是从用户输入中取一个字符串来自定义排序,当然,您别无选择,所以请随意忽略我令人沮丧的言论!

编辑:既然你是接受用户输入,这里就是我的意思是通过映射:

class CustomSorter
{
    static Dictionary<string, Func<IMyClass, object>> Selectors;

    static CustomSorter()
    {
        Selectors = new Dictionary<string, Func<IMyClass, object>>
        {
            { "letter", new Func<IMyClass, object>(x => x.Letter) },
            { "number", new Func<IMyClass, object>(x => x.Number) }
        };
    }

    public void Sort(IEnumerable<IMyClass> list, string sortField, bool isAscending)
    {
        Func<IMyClass, object> selector;
        if (!Selectors.TryGetValue(sortField, out selector))
        {
            throw new ArgumentException(string.Format("'{0}' is not a valid sort field.", sortField));
        }

        // Using extension method defined above.
        return list.Order(selector, isAscending);
    }
}

上面显然不像从字符串动态生成表达式并调用它们那样聪明; 根据您的偏好以及您所参与的团队和文化,这可以被视为优势或劣势.在这种特殊情况下,我认为我会投票支持手动绘图,因为动态表达路径感觉过度设计.