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 で 2 回割り当てられたようです。なぜかわからない？

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ます。この点に関するの動作はforeachC# 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 句を完全に作成してから、すべての AND を一度に作成する必要があります。さらに、ローカル 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 に答える