C#で独自クラスのListを階層構造にしたい。

前提・実現したいこと

getObjectメソッドで作成したsampleObjectListから、userIdとparentIdが紐づく箇所を階層構造にして
Accountクラスに格納したいです。

下記のコードだとs3のparentIdとs1のuserIdが一緒なので、
foreachの中でs1の場合childrenにs3が入るような形に直して、Accountクラスを作成したいです。
プログラムを始めたばかりで、拙い文章ですがよろしくお願いします。

発生している問題・エラーメッセージ

該当のソースコード

メインのクラス

  public void Main(string[] args)
        {
            var accountList = new List<Account>();
            var sampleObjectList = GetObject();

            foreach (var item in sampleObjectList)
            {
                var account = new Account();
                account.name = item.name;
                account.userId = item.userId;
                //ここでuserIdとParentIdが紐づくものをchildrenに格納したい。

                accountList.Add(account);
            }
        }

        private List<sampleObject> GetObject()
        {
            var so = new List<sampleObject>();
            var s1 = new sampleObject()
            {
                name = "aaa",
                userId = "aaa",
                parentId = null
            };
            var s2 = new sampleObject()
            {
                name = "bbb",
                userId = "bbb",
                parentId = null
            };
            var s3 = new sampleObject()
            {
                name = "ccc",
                userId = "ccc",
                parentId = "aaa"
            };
            so.Add(s1);
            so.Add(s2);
            so.Add(s3);

            return so;
        }

モデルクラス

    public class Account
    {
        public string userId { get; set; }
        public string name { get; set; }
        public List<Account> children { get; set; }
    }

    public class sampleObject
    {
        public string userId { get; set; }
        public string name { get; set; }
        public string parentId { get; set; }
    }

退会済みユーザー

2019/07/16 22:02

パット見思っただけですが、Account クラスの構造 (List<Account> children のところ) は正しいですか？

tanaka_sanada

2019/07/16 22:54

はい、今のコードですとlistにする意味はないかもしれませんが親に対して複数子要素がある場合も想定してList<account>としております。

退会済みユーザー

2019/07/17 01:12 編集

> 親に対して複数子要素がある場合も想定してList<account>としております。それはちょっと自分的には違和感ありありという感じです。モデルクラスの構造を見直した方がよさそうな気がします。親があって、それに複数の子があるという場合、例えば以下の記事の Blog (親), Post (子) のようにしてはいかがですか？新しいデータベースの code First https://docs.microsoft.com/ja-jp/ef/ef6/modeling/code-first/workflows/new-database

行動規範の内容に同意します

回答2件

他の方の回答の通り、間違いなく全部作ってからやったほうがいいのですが、一応できますよのサンプル。
テンポラリを使ったりして汚いので、お勧めはしません。そもそもまともに動作確認してないので使えないかもしれません。

    foreach (var item in sampleObjectList)
    {
        var account = new Account();
        account.name = item.name;
        account.userId = item.userId;

        AddChild(accountList, item.parentId, account);

        accountList.Add(account);
    }

//...

private Dictionary<string, List<Account>> temp = new Dictionary<string, List<Account>>();
private void AddChild(List<Account> accounts, string parentId, Account child)
{
    if (temp.ContainsKey(child.userId))
    {
        child.children = temp[child.userId];
        temp.Remove(child.userId);
    }

    if (parentId == null) return;

    var parent = accounts.FirstOrDefault(e => e.userId == parentId);
    if (parent != null)
    {
        parent.children.Add(child);
    }
    else
    {
        if (!temp.ContainsKey(parentId)) temp[parentId] = new List<Account>();
        temp[parentId].Add(child);
    }
}

private List<sampleObject> GetObject()
{
    var so = new List<sampleObject>()
    {
        new sampleObject()
        {
            name = "ddd",
            userId = "ddd",
            parentId = "aaa"
        },
        new sampleObject()
        {
            name = "aaa",
            userId = "aaa",
            parentId = null
        },
        new sampleObject()
        {
            name = "bbb",
            userId = "bbb",
            parentId = null
        },
        new sampleObject()
        {
            name = "ccc",
            userId = "ccc",
            parentId = "aaa"
        },
    };
    return so;
}

投稿2019/07/17 01:04