このLINQ式を最適化する方法は？

Question

私はこのコードを持っている：料金表はより多くの10の値を持つ場合

var query = from deal in db.Deals 
      where deal.EndDate >= DateTime.UtcNow 
      select deal; 

var priceList = filters.Price.GetPriceRangeList(); 
foreach(var price in priceList) 
{ 
    var startPrice = price.StartPrice; 
    var endPrice = price.EndPrice; 
    var priceResult = from deal in query 
      where (deal.DiscountPrice >= startPrice && deal.DiscountPrice <= endPrice) 
      select deal; 
    if(priceResult.Count() != 0) 
     priceResults = (priceResults == null) ? priceResult : priceResult.Union(priceResults); 
} 
query = priceResults != null ? query.Intersect(priceResults) : Enumerable.Empty<Deal>().AsQueryable(); 

私のクエリが遅いです。

私はフィルターとしてIntersectを使用しています。

これらのクエリを最適化するにはどうすればよいですか？最適化のための

Answer 1

あなたはいくつかの問題を持っています。最初は、クエリがforeachループの繰り返しごとに実行されるということです。 ToListまたはToArrayを呼び出すと、一度だけ実行されることが保証されます 第2に、ユニオンは高価です。 foreachループの繰り返しごとにpriceResultを繰り返します。 第3に、カウントも繰り返しますpriceResultです。任意の要素があるかどうかを知りたい場合は.Anyを代わりに使用してください。しかし、私はあなたがそれを避けることができると思います。私が正しくあなたのコードを読んでいれば、私は、異なるが、一度だけ

Answer 2

ひとつのアイデアは、StartPrice昇順でStartPriceがDiscountPrice財産よりも高くなると、あなたの内側のクエリがちょうどトラバーサルを停止することができ、そのようにクエリをソートするために、次のようになります。

var query = from deal in db.Deals 
      where deal.EndDate >= DateTime.UtcNow 
      orderby deal.DiscountPrice ascending 
      select deal; 

.. 
foreach(..) 
{ 
    var startPrice = price.StartPrice; 
    var endPrice = price.EndPrice; 

    var queryLocal = query.SkipWhile(deal => deal.DiscountPrice < startPrice); 
    var priceResult = queryLocal.TakeWhile(deal => deal.DiscountPrice >= startPrice 
             && deal.DiscountPrice <= endPrice); 

    .. 
}