C# と VB.NET の質問掲示板

■No100721 (しーや さん) に返信

No100725 で、

> InvalidOperationException がスローされるはずです。

と書いた件、質問者さんのコードを .NET Framework 4.8 のコンソールアプリにコピペして
確認しました。

foreach (KeyValuePair <string ,int>kvp in item) ループで、2 回目に、

item["bag"] = item["bag"] - item["wawawa"];

でコレクションを変更してますが、そうするとその次の foreach のところで、

System.InvalidOperationException: コレクションが変更されました。列挙操作は実行されない可能性があります。

という例外がスローされます。例外がスローされるのでそこでアプリは中断されます。

質問者さんのケースで、

> 以降ずーっとwawawaを買い続けてしまいます（-3 → 書き出し　のループ）。

となったのは Unity の特殊事情ではないかと思います。

どうすべきかというと、以下のように foreach ループ内で item["bag"] を書き換えたり
しないことです。

internal class Program
    {
        static void Main(string[] args)
        {
            Dictionary<string, int> item = new Dictionary<string, int>()
            {
                {"bag",25},
                {"wawawa",3},
                {"dadada",10},
            };

            int bag = item["bag"];
            foreach (KeyValuePair<string, int> kvp in item)
            {
                string buy = kvp.Key;                
                switch (buy)
                {
                    case "wawawa":
                        bag = bag - item["wawawa"];
                        break;
                    case "dadada":
                        bag = bag - item["dadada"];
                        break;
                    default:
                        break;
                }

                Console.WriteLine($"{bag}");
            }
        }
    }
}

結果は:

25
22
12