LINQ to Entities orderbyとnullのコレクションに関する質問

Question

私は現在フォーラムを開発中です。私はLINQとEFが初めてです。私のフォーラムでは、最新のトピックを最初に含むトピックのリストを表示するディスプレイがあります。

問題は、「最新」はトピックの返信に関連していることです。だから私はトピックの投稿日付でリストを注文するのではなく、トピックの最後の返信の投稿日付でリストを注文したいと思います。新しい返答のあるトピックがリストの先頭にポップアップするようにします。すべてのトピックに少なくとも1つの返信があることが分かっていれば、これは簡単です。私はちょうどこれを行うでしょう：

var topicsQuery = from x in board.Topics 
        orderby x.Replies.Last().PostedDate descending 
        select x; 

しかし、多くの場合、トピックには返信がありません。その場合は、トピックの投稿日付を代わりに使用したいと考えています。トピックに応答がない場合、x.PostedDateで注文するようにlinqクエリ内に方法がありますか？私はこれで混乱していると助けていただければ幸いです。上記のクエリでは、返信があると仮定してx.Replies.Last()があるため、返信なしのトピックが壊れます。 LastOrDefault()は、返信があることを前提とするPostedDateプロパティにアクセスする必要があるため、機能しません。

ありがとうございました。

Answer 1

var topicsQuery = from x in board.Topics 
        let lastActivityDate = x.Replies.Any() 
         ? x.Replies.Last().PostedDate 
         : x.PostedDate 
        orderby lastActivityDate descending 
        select x; 

編集：

あなたのコメントに答えるために、明示的にLINQの表現の構文は 'せ' ははありません。

var topicsQuery = board.Topics.Select(x => new { 
                Topic = x, 
                LastActivityDate = x.Replies.Any() 
                 ? x.Replies.Last().PostedDate 
                 : x.PostedDate 
            }) 
          .OrderByDescending(p => p.LastActivityDate) 
          .Select(r => r.Topic) 

EDIT2：これをさらに簡略化することができ

ニコラスが示唆されたとして、我々は中間select文などを削除することができ、次のようにしかし、あなたは（中間の選択式を使用して）同じことを達成することができますよくこれは異なる実行のためでない場合は可能ではないことに注意してください。

var topicsQuery = board.Topics 
     .OrderByDescending(x => x.Replies.Any() 
             ? x.Replies.Last().PostedDate 
             : x.PostedDate); 

Answer 2

トピックデータに新しいフィールドを追加することについて考えましたか？ LastActivityDateのようなもの。トピックをPostedDateに設定し、返信が追加されるたびに更新します。

私はSQL Serverのエキスパートではありませんが、クエリで求めていることを実行しようとすると、インデックスが無視され、完全なテーブルスキャンが実行されるクエリが発生する可能性があります。

Answer 3

私はコメントできません。 FWIW、これをコメントとして扱う。

var topicsQuery = from x in board.Topics 
       let lastActivityDate = x.Replies.Any() 
        ? x.Replies.Last().PostedDate 
        : x.PostedDate 
       orderby lastActivityDate descending 
       select x; 

あなたは典型的なフォーラムは特に、この特定のケースでは、多くの読み取りと書き込みの数いる

x.Replies.Last().PostedDate 

この

x.Replies.OrderByDesc(r => r.PostedDate).First().PostedDate 

Answer 4

変更する場合があります。したがって、余分なフィールドにLastPostDateをキャッシュすることを選択します。それはいくつかの冗長性と価値を最新に保つためにもう少しプログラミングを与えますが、私はほとんどの場合、それが最善の方法だと思います。