动态排序中OrderByNull扩展方法泛型参数兼容异常排查

问题场景 #

我需要通过反射实现一套动态排序逻辑：

1. 先按指定的sortColumn是否为空排序
2. 再按该列的值排序
3. 最后按硬编码的列（summary或description）排序

已经实现了可以正常运行的OrderBy扩展方法，但自己编写的OrderByNull扩展方法在执行时抛出了异常：

System.InvalidOperationException : 'No generic method 'OrderByDescending' on type 'System.Linq.Queryable' is compatible with the supplied type arguments and arguments. No type arguments should be provided if the method is non-generic.'

错误原因解析 #

原来的OrderByNull实现有两个核心问题：

1. 排序键类型不匹配：调用Queryable.OrderBy/OrderByDescending时，你传入的泛型参数第二个类型是property.PropertyType，但实际上排序的依据是「属性是否为null」的布尔值（bool类型），这导致Queryable找不到匹配的泛型方法，直接抛出异常。
2. null判断方式不够稳妥：使用Expression.Call调用Equals(null)的方式，对于值类型属性会出现问题，而且部分LINQ查询提供者可能无法正确解析这种调用，换成直接的相等判断更可靠。

修复后的OrderByNull扩展方法 #

以下是修复后的完整代码，关键修改点我已经标注出来：

public static IOrderedQueryable<TEntity> OrderByNull<TEntity>(this IQueryable<TEntity> source, string orderByProperty, bool desc, bool thenBy = false) {
    string command = desc ? "OrderByDescending" : "OrderBy";
    if (thenBy) command = desc ? "ThenByDescending" : "ThenBy";
    
    var type = typeof(TEntity);
    var property = type.GetProperty(orderByProperty);
    var parameter = Expression.Parameter(type, "p");
    
    // 修改点1：用Expression.Equal直接判断属性与null相等，兼容引用/值类型
    var equalsNullExpression = Expression.Equal(
        Expression.Property(parameter, orderByProperty), 
        Expression.Constant(null, typeof(object))
    );
    
    var orderByExpression = Expression.Lambda(equalsNullExpression, parameter);
    
    // 修改点2：泛型参数的第二个类型改为typeof(bool)，因为排序键是布尔值
    var resultExpression = Expression.Call(
        typeof(Queryable),
        command,
        new Type[] { type, typeof(bool) },
        source.Expression,
        Expression.Quote(orderByExpression));
    
    return (IOrderedQueryable<TEntity>)source.Provider.CreateQuery<TEntity>(resultExpression);
}

验证后的排序逻辑 #

修复后，你的排序调用代码可以正常执行：

if (!String.IsNullOrEmpty(sortColumn)) {
    var descending = sortDirection == "desc";
    views = views.AsQueryable()
        .OrderByNull<ToDoView>(sortColumn, true)
        .OrderBy<ToDoView>(sortColumn, descending, true)
        .ThenBy(v => v.summary ?? v.description).ToList();
}

内容的提问来源于stack exchange，提问作者Justin