C# Expression Visitor如何否定构建过滤器及IQueryable实现问题

首先得说，你这个基础逻辑其实已经踩对了方向——第三方API这种「OR包裹AND列表」的结构，本质就是把整个过滤条件拆解成若干组“必须同时满足的条件”，然后只要满足其中任意一组即可。结合你提到的现有流程，我帮你梳理下核心逻辑的精细化处理，以及常见坑的解决办法：

核心逻辑的优化点 #

• OR语句的正确展开方式：
  当解析到OR（对应LINQ的||）时，不能简单把当前过滤器“展开”，而是要新建独立的Or分组。比如原来的过滤器是[Or1(AndA, AndB)]，遇到OR AndC后，应该变成[Or1(AndA, AndB), Or2(AndC)]，而不是把AndC塞进Or1里（那会误变成AND逻辑）。如果是嵌套OR的情况，要注意单独为嵌套逻辑创建新分组，避免和顶级OR分支混在一起。

• AND语句的正确追加方式：
  解析到AND（对应LINQ的&&）时，确实要把这个And条件追加到当前所有活跃的Or分组里。比如当前有[Or1(AndA), Or2(AndB)]，遇到AND AndC后就变成[Or1(AndA, AndC), Or2(AndB, AndC)]，完美对应(A AND C) OR (B AND C)的逻辑，这部分你的原有思路是对的。

常见问题的解决办法 #

1. 嵌套逻辑处理（比如(A OR B) AND C） #

这种嵌套逻辑可以直接转化为API支持的结构：把外部的AND条件批量追加到内部每个OR分支里，最终变成(A AND C) OR (B AND C)，正好契合我们的AND处理逻辑。解析时只要先提取内部的OR分支，再把外部AND条件逐一追加就行。

2. 特殊操作符兼容 #

如果API支持Not、In这类操作符，你可以给And类扩展OperatorType枚举（比如Equal、NotEqual、In、NotIn），解析表达式树时把对应逻辑转换成And实例。比如解析Where(x => x.Id != 5)时，就生成And { Field = "Id", Operator = "NotEqual", Value = 5 }。

3. 避免过滤器冗余 #

遇到重复的And条件（同一字段、操作符、值）时，可以做去重处理：给每个And生成唯一签名（比如拼接字段+操作符+值的哈希），维护一个哈希集合记录每个Or分组已有的条件，避免重复添加，减少API请求体积。

简化版代码示例 #

// 表达式树解析核心方法
private void VisitBinaryExpression(BinaryExpression expr)
{
    if (expr.NodeType == ExpressionType.OrElse) // 处理OR逻辑
    {
        // 先处理左侧表达式，保存当前OR分组状态
        var leftOrs = new List<Or>(_filters);
        _filters.Clear();
        // 处理右侧表达式，生成新的OR分组
        Visit(expr.Right);
        // 合并左右两侧的OR分组
        _filters.AddRange(leftOrs);
    }
    else if (expr.NodeType == ExpressionType.AndAlso) // 处理AND逻辑
    {
        // 先处理左侧表达式
        Visit(expr.Left);
        // 把右侧表达式转为And条件，追加到所有OR分组
        var andCondition = ConvertToAnd(expr.Right);
        foreach (var orGroup in _filters)
        {
            orGroup.ands.Add(andCondition);
        }
    }
    // 其他操作符（比如Equal、GreaterThan）的处理...
}

// 将单个表达式转换为And条件
private And ConvertToAnd(Expression expr)
{
    if (expr is BinaryExpression binaryExpr)
    {
        var fieldName = ((MemberExpression)binaryExpr.Left).Member.Name;
        var value = ((ConstantExpression)binaryExpr.Right).Value;
        var operatorType = binaryExpr.NodeType switch
        {
            ExpressionType.Equal => "Equal",
            ExpressionType.NotEqual => "NotEqual",
            ExpressionType.GreaterThan => "GreaterThan",
            _ => throw new NotSupportedException($"不支持的操作符: {binaryExpr.NodeType}")
        };
        return new And { Field = fieldName, Operator = operatorType, Value = value };
    }
    throw new NotSupportedException($"不支持的表达式类型: {expr.GetType().Name}");
}

调试小技巧 #

• 解析完成后，把_filters序列化成JSON，对比预期的过滤器结构是否一致，快速定位逻辑偏差。
• 针对复杂LINQ查询（多层嵌套Where、复合逻辑），可以分步打印每一步的过滤器状态，看问题出在哪个节点的解析上。

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