2

我正在使用谓词生成器编写以下代码：

IEnumerable<int> ids= new List<int> { 47, 48 };

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

predicate = predicate.And(x => x.CreatedAt >= fromDate && x.CreatedAt <= toDate);

foreach (var id in ids)
{
    predicate = predicate.Or(x => x.Source.Id == id);
}

var result = Database.Set<Customer>().AsExpandable()
                                     .Where(predicate)
                                     .ToList();

生成的 SQL 看起来像（只是 WHERE 子句）：

WHERE ([Filter6].[SourceId] IN (@p__linq__0,@p__linq__1))
AND ([Filter6].[CreatedAt] >= @p__linq__2)
AND ([Filter6].[CreatedAt] <= @p__linq__3)',
N'@p__linq__0 int,
@p__linq__1 int,
@p__linq__2 datetime2(7),
@p__linq__3 datetime2(7)',
@p__linq__0=48,
@p__linq__1=48,
@p__linq__2='2012-02-07 21:59:55.0437985',
@p__linq__3='2012-02-07 22:04:55.5748288'

看起来 id 48 在 SQL 中被分配了两次。不知道为什么？

4

2 回答 2

6 
foreach (var id in ids)
{
    predicate = predicate.Or(x => x.Source.Id == id);
}

您正在关闭循环变量。id改为制作变量的本地副本：

foreach (var id in ids)
{
    int localId = id;
    predicate = predicate.Or(x => x.Source.Id == localId);
}

由于 Linq 是惰性的Or，因此您的谓词id只会在您执行查询时进行评估，此时的值是集合id中的最后一项。ids在这方面的行为foreach将在 C# 5 中改变，这将不再是问题。有关更多信息，请阅读“关闭被认为有害的循环变量”

于 2012-02-07T22:26:52.340 回答
4

如果这就是您所做的全部并且您的列表不长，那么您根本不需要谓词构建器。

 var result = Database.Set<Customer>().AsExpandable() 
                                      .Where( x => x.CreatedAt >= fromDate
                                                   && x.CreatedAt <= toDate
                                                   && ids.Contains( x.Source.Id ) ) 
                                      .ToList();

如果您要使用谓词构建器，那么您需要完全构建 OR 子句，然后一次性完成所有操作。此外，您需要使用本地 id 变量，否则它将关闭迭代变量并仅获取其最后一个值绑定。

// This should translate to True AND (u AND x) AND (FALSE OR y OR z)
var predicate = PredicateBuilder.True<Customer>();

predicate = predicate.And(x => x.CreatedAt >= fromDate && x.CreatedAt <= toDate);      

var idPredicate = PredicateBuilder.False<Customer>();      
foreach (var id in ids)      
{   
    var localId = id;   
    idPredicate = idPredicate.Or(x => x.Source.Id == localId);      
}

predicate = predicate.And( idPredicate );
于 2012-02-07T22:41:17.617 回答