Question

以下代码段工作正常。

switch (sortOrder)
{
    case "Group":
        list = list.OrderBy(o => o.Group).ToList();
        break;
    case "Company":
        list = list.OrderBy(o => o.Company).ToList();
        break;
    case "CurrencyId":
        list = list.OrderBy(o => o.CurrencyId).ToList();
        break;
    case "Field":
        list = list.OrderBy(o => o.Field).ToList();
        break; 
}

但我想写一个更灵活的代码，根本不使用开关。类似的东西：

list = list.OrderBy(o => o.sortOrder).ToList();

我该怎么做？

Answer 1

使用反射是另一种选择：

var sortOrder = "Group";

list = list.OrderBy(o => o.GetType().GetProperty(sortOrder).GetValue(o)).ToList();

请注意，如果您将“sortOrder”设置为该类中实际不存在的属性名称，您将获得NullReferenceException。但是如果你控制了价值，而不只是接受用户输入的内容（例如），那就没关系。

Answer 2

你必须使用反射。问题是反射是缓慢而丑陋的。

list.OrderBy(o => o.GetType().GetProperty(sortOrder).GetValue(o)).ToList()

你对表现的影响不大。为了使它看起来更好，你可以将它放入扩展方法：

public static IEnumerable<T> OrderByPropertyName<T>(this IEnumerable<T> source, string propertyName)
{
    var property = typeof(T).GetProperty(propertyName);

    if (property == null) return source; // or throw exception
    else return source.OrderBy(x => property.GetValue(x));
}

然后将其用作

list.OrderByPropertyName(sortOrder)

它比第一次反射方法稍微快一些，因为GetProperty只被调用一次。如果属性不存在，您还可以控制行为。

在我看来，在这种情况下switch优于反思。

Answer 3

@Jakub Lortz的解决方案很好，但是有更通用的解决方案。

public static IQueryable<T> OrderByPropertyName<T>(
    this IQueryable<T>source,
    string propertyName)
where T: YourClass
{
    // Here better to throw some exception
    if (propertyName == null) return source; 
    var property = typeof(T).GetProperty(propertyName);

    return source.OrderBy(x => property.GetValue(x));
}

在C＃中替换OrderBy Switch

3 个答案: