为什么使用 PredicateBuilder 的这段代码不起作用？

Question

为什么我的列表没有返回任何内容？

 class Program
    {
        static void Main(string[] args)
        {
            var list = new List<string>{ "apple" , "mango" , "chickoo", "kiwi" };
            var searchWord = new List<string>{ "a" };

            var predicate = PredicateBuilder.False<string>();

            foreach(var word in searchWord)
            {
                predicate.Or(p => p.Contains(word));
            }

           var qry = list.Where(predicate.Compile());

           foreach (var item in qry)
           {
               Console.WriteLine(item);
           }

           Console.Read();
        }
    }

我正在使用 Joseph Albahari 的 PredicateBuilder。

Answer 1

您需要将结果分配给您的 predicate 变量：

predicate = predicate.Or(p => p.Contains(word));

PredicateBuilder 页面还强调了一个重要问题：

循环中的临时变量是
需要避免外部变量
陷阱，其中相同的变量是
为每次迭代捕获
foreach 循环。

因此，您的代码应类似于以下内容：

        foreach(var word in searchWord)
        {
            var temp = word;
            predicate = predicate.Or(p => p.Contains(temp));
        }

Answer 2

这里有两个问题：

1. Or 扩展方法不会改变现有的表达式树 - 它返回一个新的表达式树。表达式树通常是不可变的。
2. 您正在关闭循环变量。要了解此问题的原因，请参阅 关闭被认为有害的循环变量。

尝试：

foreach(var word in searchWord)
{
    var wordCopy = word;
    predicate = predicate.Or(p => p.Contains(wordCopy));
}

或者更好：

var predicate = searchWord.Aggregate(PredicateBuilder.False<string>(),
                   (predSoFar, word) => predSoFar.Or(p => p.Contains(word)));

Answer 3

这对我来说看起来不正确：

foreach(var word in searchWord)
{
    predicate.Or(p => p.Contains(word));
}

我怀疑它应该是：

foreach(var word in searchWord)
{
    predicate = predicate.Or(p => p.Contains(word));
}

换句话说，就像 LINQ 的其余部分一样，PredicateBuilder 不允许您更改当前谓词- 它允许您基于现有谓词和新条件构建新谓词。

无论如何，这肯定是 示例代码 所建议的......

当您查看完整的源代码时PredicateBuilder 的代码（在同一页上），证实了这一点 - 谓词实际上只是一个 Expression> - 即表达式树。您没有创建 PredicateBuilder 类或类似内容的实例。表达式树是不可变的，因此 Or 唯一可以做的就是返回一个新的表达式树，如下所示：

public static Expression<Func<T, bool>> Or<T>
    (this Expression<Func<T, bool>> expr1,
     Expression<Func<T, bool>> expr2)
{
    var invokedExpr = Expression.Invoke(expr2,
                                        expr1.Parameters.Cast<Expression>());
    return Expression.Lambda<Func<T, bool>>
          (Expression.OrElse(expr1.Body, invokedExpr), expr1.Parameters);
}