C#循环中为表达式添加可变数量条件的实现问题

你的思路方向是对的：通过动态构建表达式来处理可变数量的过滤条件，这种方式比硬编码多个条件或者拼接字符串查询要灵活得多。但你当前代码的问题出在表达式合并的细节上，下面一步步给你拆解：

为什么会出现编译错误？ #

你用Expression.And(expression, newCondition.Body)得到的是一个BinaryExpression——这只是表达式的"主体逻辑部分"，但你需要赋值的变量expression是Expression<Func<DataRow, bool>>类型，它不仅包含逻辑主体，还包含参数定义（也就是那个r）。直接把BinaryExpression赋值给Expression<Func<...>>，类型自然不匹配，编译器就报错了。

正确的实现方式 #

要合并多个Expression<Func<DataRow, bool>>，核心要注意两点：

1. 所有表达式要复用同一个参数实例（避免出现多个不同的r参数导致执行错误）
2. 合并后要重新用Expression.Lambda包装成完整的Expression<Func<DataRow, bool>>

这里推荐用Expression.AndAlso（短路逻辑与，和C#里的&&行为一致，比Expression.And的按位与更符合业务判断逻辑），修正后的代码如下：

var row = parser.ReadFields();
Expression<Func<DataRow, bool>> expression = null;
// 提前定义好统一的参数，所有条件都用这个"r"
ParameterExpression param = Expression.Parameter(typeof(DataRow), "r");

foreach (var pairToCheck in mappingDictionary)
{
    // 构建当前条件的逻辑主体：r[pairToCheck.Value].ToString() == row[pairToCheck.Key]
    var columnAccess = Expression.PropertyOrField(param, pairToCheck.Value);
    var toStringCall = Expression.Call(columnAccess, typeof(object).GetMethod("ToString", Type.EmptyTypes));
    var targetValue = Expression.Constant(row[pairToCheck.Key]);
    var equalsCondition = Expression.Equal(toStringCall, targetValue);

    if (expression == null)
    {
        // 第一个条件，直接包装成完整表达式
        expression = Expression.Lambda<Func<DataRow, bool>>(equalsCondition, param);
    }
    else
    {
        // 合并现有表达式和新条件的主体，再重新包装成完整表达式
        var combinedBody = Expression.AndAlso(expression.Body, equalsCondition);
        expression = Expression.Lambda<Func<DataRow, bool>>(combinedBody, param);
    }
}

// 处理mappingDictionary为空的情况，避免空引用
var filter = expression?.Compile() ?? (r => false);
var recordFound = dt.AsEnumerable().Where(filter).Any();

额外优化点 #

• 最后用Any()代替Count() > 0：Any()找到第一个符合条件的记录就会停止遍历，比Count()遍历整个表效率更高
• 提前处理空表达式的情况：如果mappingDictionary是空的，直接返回false（或者根据你的业务需求调整默认逻辑）

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