【C#】Listに匿名型のメンバを追加したい

Question

こんにちは。今回もよろしくお願いします。

前提・実現したいこと

２つのListを突合し、1つ目のリストに足りない項目を2つ目のリストからAddして補完する
関数を作成しましたが、実行時エラーによりプログラムが停止してしまいます。
原因はある程度特定できたと思うのですが、解決方法が分かりません。

<Listの構造>
突合したいList(第1引数)は、LINQにて作成しました。下図のような構成になっています。

突合したいList(第2引数)も同じ構成ですが、.result.Choiseの内容が一部異なります。
下図のように、赤丸の部分が第1引数にない項目のため、第1引数の.resultにAddし、戻り値として第1引数を返します。

該当のソースコード

        private List<dynamic> List_Comp(List<dynamic> target ,List<dynamic> reference)
        {

            int list_count = 0;
            bool Contains_flg = false;
            foreach (dynamic dy in reference)
            {
                foreach (var item in dy.result)
                {
                    Contains_flg = false;
                    foreach (var it in target[list_count].result)
                    {
                        if(item.Choise == it.Choise)
                        {
                            Contains_flg = true;
                        }
                    }
                    if(Contains_flg == false)
                    {
                      // 選択肢の追加
                        // 実行時エラー
                        target[list_count].result.Add(new { Choise = item.Choise.ToString(), Count = 0 });  

                        // 試したこと①　正常にListのresultにAddされた
                          // target[list_count].result.Add(new { Choise = "aaa", Count = 0 });
                        // 試したこと②　正常にメッセージ"aaa"が表示された                
                          // MessageBox.Show(item.Choise);                                                    
                    }
                }
                list_count++;
            }

            return (target);
        }

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

Microsoft.CSharp.RuntimeBinder.RuntimeBinderException: ''System.Collections.Generic.List<<>f__AnonymousType0<string,int>>.Add(<>f__AnonymousType0<string,int>)' に最も適しているオーバーロード メソッドには無効な引数がいくつか含まれています'

原因の考察

試したこと①から、そもそもエラーがでた行の全体的な記述は間違ってないことは分かりました。
また、固定値でのAddなら動作したため、Choise = item.Choise.ToString()の部分が問題であることは特定できました。

試したこと②から、item.Choise自体はきちんと取得できていることは分かりました。

エラーメッセージの内容から、itemが匿名型のためitem.Choiseの.Choiseをメソッドとコンパイラが勘違いしているのでは？と思っています。
上記から、itemが匿名型であることが問題で、2回目のforeachのvarを何かしらに変更すればきちんと動作するのではと考えましたが、具体的対策が思いつきません。

完全独学なので、突っ込みどころ満載かと思いますがビシビシ突っ込んでもらえるとありがたいです。

補足情報（FW/ツールのバージョンなど）

Microsoft Visual Studio Community 2017
Version 15.7.3
VisualStudio.15.Release/15.7.3+27703.2026
Microsoft .NET Framework
Version 4.7.03190

サンプルプログラム（2019年10月31日13時43分追記）

再現しないとご指摘があったため、List生成から呼び出しまでのサンプルプログラムを作成しました。
渡しの環境では以下プログラムで同じエラーが発生します。

using System.Collections.Generic;
using System.Linq;

namespace ConsoleApp3
{
    class Program
    {
        static void Main(string[] args)
        {

            List<string[]> data1 = new List<string[]>();
            List<string[]> data2 = new List<string[]>();

            List<dynamic> agg1 = new List<dynamic>();
            List<dynamic> agg2 = new List<dynamic>();

            data1.Add(new string[4] { "1", "2", "3", "4" });
            data1.Add(new string[4] { "2", "2", "3", "4" });
            data1.Add(new string[4] { "3", "2", "3", "4" });

            data2.Add(new string[4] { "1", "2", "3", "4" });
            data2.Add(new string[4] { "2", "2", "3", "4" });
            data2.Add(new string[4] { "3", "2", "3", "4" });
            data2.Add(new string[4] { "aaa", "2", "3", "4" });

            int item_count = 0;
            for (int j = 0; j < 2; j++)
            {

                agg1.Add(new
                {
                    title = j.ToString(),
                    result = data1.GroupBy(x => x[item_count])
                             .Select(x => new { Choise = x.Key, Count = x.Count() })
                             .OrderBy(x => x.Choise).ToList()
                });

                item_count++;

            }

            item_count = 0;
            for (int j = 0; j < 2; j++)
            {

                agg2.Add(new
                {
                    title = j.ToString(),
                    result = data2.GroupBy(x => x[item_count])
                             .Select(x => new { Choise = x.Key, Count = x.Count() })
                             .OrderBy(x => x.Choise).ToList()
                });

                item_count++;

            }


            agg1 = List_Comp(agg1, agg2);




        }
        private static List<dynamic> List_Comp(List<dynamic> target, List<dynamic> reference)
        {

            int list_count = 0;
            bool Contains_flg = false;
            foreach (dynamic dy in reference)
            {
                foreach (var item in dy.result)
                {
                    Contains_flg = false;
                    foreach (var it in target[list_count].result)
                    {
                        if (item.Choise == it.Choise)
                        {
                            Contains_flg = true;
                        }
                    }
                    if (Contains_flg == false)
                    {
                        // 選択肢の追加
                        // 実行時エラー
                        target[list_count].result.Add(new { Choise = item.Choise, Count = 0 });  
                    }
                }
                list_count++;
            }

            return (target);
        }



    }


}

Answer 1

完全独学なので、突っ込みどころ満載かと思いますがビシビシ突っ込んでもらえるとありがたいです。

とのことなので、突っ込みます。

dynamic は便利ですが、そのような用途で使ってはいけません。
そのような用途ではValueTuple を使うか クラスを定義して使ってください。

C# は、厳密に型指定された静的型付け言語ですが、dynamic を多用すると
静的型付け言語のメリットを失い、動的型付け言語のデメリットが発生します。
結果、静的型付け言語と動的型付け言語両方のデメリットがあり、メリットがない酷いプログラムになってしまいます。

Comments

[inari_ken]

ご回答ありがとうございます。
内容咀嚼します。取り急ぎ御礼まで。

[yuto_club]

ミス投稿

[yuto_club]

↑すいません。回答をつけるスレッドを間違えてしまいました。

正しくは、下記への回答です。

https://teratail.com/questions/220438#reply-323245

[inari_ken]

今回は動的型付けによるデメリットをもろに受けた事例ということが理解できました。
C#の型については様々な種類があるので苦手意識を持っており、dynamicに頼っていましたが今回の件でドツボにはまったので、脱却する良い経験になりました。もちろん、dynamicも使える時は使ったらいいと思いますが今回のことが起こらないよう、吟味するようにします。ありがとうございました。

Answer 2

サンプルプログラムの下記に示す行で、コンパイラは Choise メンバの型をdynamic 型として静的に解決します。
しかし、List<T> の動的型は、ここで推論された匿名型と異なるので Add メソッドの呼び出しは失敗します。

target[list_count].result.Add(new { Choise = item.Choise, Count = 0 });

これを回避するには、 item.Choiseを string 型に明示的キャストをするか、 agg1 の生成時に下記のように dynamic 型のメンバにする必要があります。

.Select(x => new { Choise = (dynamic)x.Key, Count = x.Count() })

~~ 該当のソースコード では ToString メソッドが呼ばれているので、サンプルプログラムとは状況が異なるかと思います。~~ ← 誤りだったため訂正です。 ToString メソッドを呼んだとしても静的に解決される型は、 dynamic なので同じことです。

Comments

[inari_ken]

サンプルプログラムの方にToStringメソッドをつけ忘れてしまっていました。失礼しました。
stringに明示的キャストしてAddする方向でプログラムを修正し、正常に動作することを確認しました。

ご説明によりエラーとなった理由が理解できました。
そもそもこのListをクラス化することも念頭にいれつつ、作業を進めていきます。ありがとうございました。

Answer 3

２つのListを突合し、1つ目のリストに足りない項目を2つ目のリストからAddして補完する

それは和集合なので、求めるにはLINQのUnionを使ってください。

- agg1 = List_Comp(agg1, agg2);
+ agg1 = agg1.Union(agg2).ToList();

下記回答はZuishinさんに指摘される前の回答で、同じことが実現できますが、冗長的な実装です。

---

二つの配列の差分はLINQのExceptを使うことで求めることができます。その差分をAddRangeで追加すれば解決です。

var src = new List<Hoge>();
var src2 = new List<Hoge>();

//TODO:src,src2の初期化をする

src.AddRange(src2.Except(src));

---

余談

hihijijiさんの回答には概ね同意で、本来はdynamicな型ではなくValueTupleを使うかクラスを定義するべきと思いますが、このような書き方をすることで匿名型の空リストを定義できるようです。

var agg1 = Enumerable.Range(0,0)
     .Select(x => new
     {
         title = "",
         result = new []
         {
             new
             {
                 Choise = "",
                 Count = 0
             }
         }.ToList()
     })
     .ToList();
var agg2 = agg1.ToList();

Comments

[Zuishin]

それは Union ではないでしょうか。

[BluOxy]

やっていることが和集合であることに気づいていませんでした。
ありがとうございます。

何か特別なメソッドを作ることも必要なさそうですね。

[inari_ken]

ご回答ありがとうございます。
本件では、親Listの"agg1"をUnionするのではなく、子リストの"agg1.result"内の和集合を取る認識です。
agg1 = agg1[0].result.Union(agg2l[0].result).ToList();
（エラーになってしまいますがイメージとしてはこんな感じです。）

また、Addしたagg1の.result.Countを0にするような仕様としたいので、今回はUnion1行で済ませることは難しいと判断しています。

余談については、非常に勉強になりました。どこかで使えそうです。ありがとうございました。

Answer 4

item.Choise.ToString()の戻り値の型がdynamicになってるはずなので、

Choise = item.Choise.ToString()

を

Choise = (string)item.Choise.ToString()

にしてみてください。

Comments

[inari_ken]

ご回答ありがとうございます。
上記のとおり書き換えたところ、正常にaddできました。
Tostring()しなのに、戻り値はdynamicなことに驚いておりますが、取り急ぎ御礼まで。

Answer 5

試しに下記のような呼び出しをしてみましたが、エラーが再現しませんね。

List_Comp(
	new List<dynamic>() {
		new { 
			result = new List<dynamic>()
			{
				new { Choise ="a", Count = 1}
			},
			title = "1"
		}
	},
	new List<dynamic>() {
		new { 
			result = new List<dynamic>()
			{
				new { Choise ="b", Count = 1}
			}
			,title = "2"
		}
	}
);

エラーが再現する最小の呼び出しコードを書いてもらうことはできますか？

Comments

[inari_ken]

ご回答ありがとうございます。
本文にサンプルプログラムを追記しましたので、ご確認をお願いします。