foreachループを使用せずにC＃のリスト内にリストを挿入します。もっといい方法？

Question

私は、そのオブジェクトの1つとしてネストされたリスト（List-ChildClass）を持つオブジェクトのリスト（List-ParentClass）内のオブジェクトのリストを持っています。 List-ChildClassを設定するには、以下のようにforeachループを使用しました。私はまた、以下のようにlinqクエリをネストしました。

この時点で私はパフォーマンス上の問題を抱えており、これを行うためのよりよい方法があると感じています。

質問：私はこれをより速く/より速く行うことができますか？

注 - これはC＃で書かれたWebベースの.net MVCアプリケーションです。私はEFをSQLデータベースに戻します。

public class ParentClass 
{ 
    public int pcid { get; set; } 
    public List<ChildClass> ChildClassList { get; set; } 
} 

public class ChildClass 
{ 
    public int pcid { get; set; } 
    public int ccid { get; set; } 
} 

public class DoWork 
{ 
    public void ExampleMethodForEach() 
    { 
     List<ParentClass> ParentClassList = new List<ParentClass>(); 

     foreach(ParentClass a in ParentClassList) 
     { 
      a.ChildClassList = EFDatabase2.where(b => b.pcid == a.pcid).select(b => b.ccid).ToList(); 
     } 
    } 

    public void ExampleMethodLinq() 
    { 
     var ParentClassList = (from a in EFDatabase 
           select new ParentClass 
           { 
            ccid = a.ccid, 
            pcid = (from b in EFDatabase2 
              where b.pcid == a.pcid 
              select b.ccid).ToList() 
            //something like this were I nest a query 
           }).ToList(); 
    } 
} 

Answer 1

リレーショナルデータベースとLINQを使用する場合、を使用してデータを関連付けるときに最も良い方法は、です。あなたのケースでは、最も適切でgroup join次のとおりです。あなたに素敵な高速な単一のデータベースクエリを与える必要があります

var ParentClassList = 
    (from p in EFDatabase 
    join c in EFDatabase2 on p.pcid equals c.pcid into children 
    select new ParentClass 
    { 
     pcid = p.pcid, 
     ChildClassList = 
      (from c in children 
       select new ChildClass 
       { 
        pcid = c.pcid, 
        ccid = c.ccid 
       }).ToList() 
    }).ToList(); 

。

P.S.あなたのEFDatabaseとEFDatabase2変数が、同じデータベース内の2つのテーブルを参照することを願っています。

Answer 2

データベースを複数回使用しています。 N + 1の問題があります。

私が示唆しているのは、すべての親を最初に照会することですが、子供のデータは除外します。次に、取得したすべての親のIDを取得し、配列内に配置します。この配列を使用してSQLにIN句を作成します。

親IDの配列を使用してすべての子をロードした後、親IDをキーとしてToLookupを使用してLookupにマップし、foreachを使用して子のリストを親に割り当てます。

var parents = EFDatabase2.Parents.Where(...).Select(p => new ParentClass { pcid = p.pcid }).ToList(); 
var ids = parents.Select(p => p.pcid).ToArray(); 
var children = EFDatabase2.Children.Where(c => ids.Contains(c.ccid)).Select(c => new ChildClass { pcid = c.pcid, ccid = c.ccid }).ToLookup(c => c.pcid); 

foreach (var parent in parents) 
{ 
    parent.Children = children[parent.pcid]; 
} 

この場合、データベースに対して2つのクエリしか実行しません。