动态OrderBy多列排序函数报错求助：变量未定义异常

错误原因是每次循环都创建了新的ParameterExpression（名为p的变量），后续ThenBy调用依赖的是第一次循环生成的表达式树中的旧p，但最终Lambda绑定的是新循环里的p，导致变量跨作用域引用，触发未定义错误。

修复后的代码 #

internal static Func<IQueryable<TEntity>, IOrderedQueryable<TEntity>> GetOrderBy(List<(string, bool)> orderColumnsAndIsDesc)
{
    bool isFirst = true;
    MethodCallExpression resultExp = null;
    string methodName;
    
    // 关键：只创建一个全局的参数表达式，所有表达式树共用同一个变量p
    ParameterExpression argQueryable = Expression.Parameter(typeof(IQueryable<TEntity>), "p");
    Type entityType = typeof(TEntity);

    foreach (var item in orderColumnsAndIsDesc)
    {
        string prop = item.Item1;
        bool isDescending = item.Item2;
        
        // 构建字段访问的Lambda表达式
        ParameterExpression arg = Expression.Parameter(entityType, "x");
        Expression expr = arg;
        PropertyInfo pi = entityType.GetProperty(prop, BindingFlags.IgnoreCase | BindingFlags.Public | BindingFlags.Instance);
        expr = Expression.Property(expr, pi);
        LambdaExpression lambda = Expression.Lambda(expr, arg);

        if (isFirst)
        {
            methodName = isDescending ? "OrderByDescending" : "OrderBy";
            resultExp = Expression.Call(
                typeof(Queryable), 
                methodName, 
                new Type[] { entityType, pi.PropertyType }, 
                argQueryable, // 使用全局的p变量
                Expression.Quote(lambda));
        }
        else
        {
            methodName = isDescending ? "ThenByDescending" : "ThenBy";
            resultExp = Expression.Call(
                typeof(Queryable), 
                methodName, 
                new Type[] { entityType, pi.PropertyType }, 
                resultExp, // 复用之前的排序表达式
                Expression.Quote(lambda));
        }

        isFirst = false;
    }

    // 用全局的p变量构建最终Lambda并编译
    var finalLambda = Expression.Lambda<Func<IQueryable<TEntity>, IOrderedQueryable<TEntity>>>(resultExp, argQueryable);
    return finalLambda.Compile();
}

核心修改点 #

1. 统一参数变量：将argQueryable（即p）的创建移到循环外，确保整个表达式树的所有节点都引用同一个变量，避免作用域冲突。
2. 移除冗余代码：删除循环内未使用的IQueryable<TEntity> query实例，以及重复创建的outerExpression，直接使用全局参数作为OrderBy的第一个参数。
3. 简化逻辑：将orderType字符串判断改为直接使用布尔值isDescending，减少不必要的字符串操作。

验证说明 #

修复后，多字段排序时，ThenBy调用会基于前一次排序的结果表达式（resultExp）继续构建，所有节点都绑定同一个p变量，不会再出现作用域未定义的错误。

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