`.Select`メソッドを使用して追加のループをスキップする

Question

エンティティに添付されたナビゲーションプロパティをロードする必要がある次のコードを仮定します（私はEF CoreとEF 6でこの問題に直面しました。ファイン）

ViewBag.DisplayTags = new String(query.ArticlesTags.SelectMany(x => { 
          db.Attach<EFContext.Database.ArticlesTags>(x).Navigation("Tag").Load(); 
          return x.Tag.Name + ", "; 
         }).ToArray()); 

私は選択文の中のEFトラッカーにコレクションを添付して、ナビゲーションプロパティをロードします。私はこれを行って、forループをもう一度スキップしました。

質問は、別のループをスキップすることは利点です（selectは内部的にループを実行するため）。このアプローチは私の見解から隠されている問題を引き起こしますか？何か欠点はありますか？それはお勧めですか？

Answer 1

これを行わないでください。データベースのすべてのArticlesTagに対して1つのクエリを実行します。したがって、ArticlesTagsテーブルに100行がある場合、上記のコードは、データベースサーバーに対して101のクエリを順次実行します（ArticlesTagsの直接クエリの場合は1つ追加します）。

次のコードは、ArticlesTagsのリストに項目ごとに実行します：明示的にデータベースを照会することによって、その単一ArticlesTag上のTagプロパティをロード

Navigation("Tag").Load(); 

を。

単一のSelect文を使用して、データベースからすべての名前をリストとして取得し、メモリ内のstring.Joinを使用してそれらを結合することができます。何かが好きです

var names = db.ArticlesTags.Select(x => x.Tag.Name).ToArray(); 
var displayName = string.Join(", ", names);