TreeView ノードの階層表示について

Question

実現したいこと

ASP.NET Web Formsで以下を実装したいです。

・DBから取得したデータを画面上のTreeViewに表示
・取得したデータは３種類で「親-子-孫」の関係性で表示する。

なお現状はこの要件を満たしたTreeViewを表示できているのですが、
foreach文を多用しておりスマートではない。（現在のソース）
BindingInfo.DataSourceを使用することによって記述量が減ることは調べたことによりわかっているが、
「親-子」までの関係性までしか表示させることができなかった。（BindingInfo.DataSource等を使用したソース）

サンプルコードでは記述量が多く見えないが、実際の開発となると非常に可読性が悪くなっている状態であるため
foreach文をできるだけ減らし、コードの可読性を向上させたいです。

繰り返し処理の簡略化等々の
何かよい解決方法がありましたら、どなたかご教授願います。

※実装イメージ
![

環境

• .NET Core 3.1 SDK
• VisualStudio 2019 Professional
  • EntityFrameworkVisualEditor
• ComponentOne Enterprise 2020V2

現在のソースコード（サンプル）

 //すべて同一クラス内で作成しているものだと思ってください。
 //親ノードクラス
 private class Parent
        {
            public long Id { get; set; }
            public string Name { get; set; }

            //コンストラクタ
            public Parent(long Id, string Name)
            {
                this.Id = Id;
                this.Name = Name;
            }
        }
 //子ノードクラス
        private class Child
        {
            public long Id { get; set; }
            public string Name { get; set; }
            public long Parent_Id { get; set; }

            //コンストラクタ
            public Child(long Id, string Name, long Parent_Id)
            {
                this.Id = Id;
                this.Name = Name;
                this.Parent_Id = Parent_Id;
            }
        }
 //孫ノードクラス        
        private class Grands
        {
            public long Id { get; set; }
            public string Name { get; set; }

            public long Child_Id { get; set; }

            //コンストラクタ
            public Grands(long Id, string Name, long Child_Id)
            {
                this.Id = Id;
                this.Name = Name;
                this.Child_Id = Child_Id;
            }
        }

        //---------ツリービュー作成------------------
        private void CreateTreeView()
        {
            //DBから取得してきた各情報-------------------
            List<Parent> parents = new List<Parent>();
            List<Child> children = new List<Child>();
            List<Grands> grands = new List<Grands>();

            parents.Add(new Parent(1, "AAA"));
            parents.Add(new Parent(2, "BBB"));
            parents.Add(new Parent(3, "CCC"));

            children.Add(new Child(1, "AAA_A", 1));
            children.Add(new Child(2, "BBB_A", 2));
            children.Add(new Child(3, "CCC_A", 3));
            children.Add(new Child(4, "AAA_B", 1));
            children.Add(new Child(5, "BBB_B", 2));
            children.Add(new Child(6, "CCC_B", 3));

            grands.Add(new Grands(1, "AAA_A_A", 1));
            grands.Add(new Grands(2, "BBB_A_A", 2));
            grands.Add(new Grands(3, "CCC_A_A", 3));
            grands.Add(new Grands(4, "AAA_A_B", 1));
            grands.Add(new Grands(5, "BBB_A_B", 2));
            grands.Add(new Grands(6, "CCC_A_B", 3));
            grands.Add(new Grands(7, "AAA_B_A", 4));
            grands.Add(new Grands(8, "BBB_B_A", 5));
            grands.Add(new Grands(9, "CCC_B_A", 6));
            //DBから取得してきた各情報 end -------------------

            foreach (Parent parent in parents)
            {
                //親ノードのインスタンス、親ノードに値を設定
                C1TreeNode parentNode = new C1TreeNode();
                parentNode.SetValue(parent.Name);

                //子ノードの設定
                foreach (Child child in children.Where(x => x.Parent_Id == parent.Id))
                {
                    //子ノードのインスタンス、子ノードの値を設定
                    C1TreeNode childNode = new C1TreeNode();
                    childNode.SetValue(child.Name);
                    
                    //孫ノードの設定
                    foreach (Grands grand in grands.Where(x => x.Child_Id == child.Id))
                    {

                        //孫ノードのインスタンス、孫ノードの値を設定
                        C1TreeNode grandNode = new C1TreeNode();
                        grandNode.SetValue(grand.Name);

                        //子ノードに孫ノードを設定
                        childNode.Nodes.Add(grandNode);
                    }
                    //親ノードに子ノードを設定
                    parentNode.Nodes.Add(childNode);
                }
                //ツリービューに親ノードを設定
                c1TreeView.Nodes.Add(parentNode);
            }

（BindingInfo.DataSource等を使用したソース）

 private class NodeData
        {
            public Parent parentNodeData { get; set; } //親ノード
            public List<Child> childNodeData { get; set; } //子ノードリスト

            //コンストラクタ
            public NodeData(Parent parentNodeData, List<Child> childNodeData)
            {
                this.parentNodeData = parentNodeData;
                this.childNodeData = childNodeData;
            }
        }
       //---------ツリービュー作成------------------
        private void CreateTreeView()
        {
            //DBから取得してきた各情報-------------------
           　//同上のため省略
            //DBから取得してきた各情報 end -------------------

            //ノード設定用ノードリスト
            List<NodeData> nodeData = new List<NodeData>();

            foreach(Parent parent in parents)
            {
                //親ノードと紐づいている子ノードのリストを取得し、ノードリストを設定する
                List<Child> child1 = children.Where(x => x.Parent_Id == parent.Id).ToList();
                nodeData.Add(new NodeData(parent, child1));
            }

            //データソースの設定
            this.associationTreeView.BindingInfo.DataSource = nodeData;

            // データメンバ（子ノードリスト）の設定
            this.associationTreeView.BindingInfo.DataMember = "\childNodeData";

            // ノート表示名に利用するメンバを設定
            c1TreeView.Columns[0].DisplayFieldName = "parentNodeData\Nam

ボールドテキスト

Comments

[Zuishin]

何を目標にしているかよくわかりませんが、SelectMany を使えばネストは減ります。また各ノードに同じインターフェースを持たせれば親・子・孫をある程度シームレスに扱えるでしょうし、探索部分のみ別クラスに追い出すという手もあります。あるいは再帰を使って深さ優先探索するのも手だと思いますし、Interactive Extensions の Expand メソッドで幅優先探索をする手もあります。foreach を減らしたいということですが、このコードを見る限りではループ回数自体は減ると思えません。親・子・孫の間の共通部分を抽象化してフラットに扱えば多重ループが一重ループあるいは再帰に落とし込めると思います。

[NapoleonClaf2]

Zuishin様
ご回答ありがとうございます。
foreachを減らしたいというのが最終的な目標でした。
が、ループ回数が減るとは思いませんとのことですので、
現状はこのソースを維持してみようと思います。
再帰などはじめて聞く単語が出てきましたのでそちらの方も一度調べてみて
今のソースに実装できるなら実装する…といった形にしたいと思います。
的確なアドバイスありがとうございました。

Answer 1

皆様ご閲覧ありがとうございました。
Zuishin様のアドバイスの元
再帰等を使用してスマートなソースを作成できるように頑張ってみます。