SortedDictionaryのCountプロパティをforループで使用の際、ループが最後まで回ってくれない

Question

C# SortedDictionaryのItemをRemoveする簡単なメソッドを作っているのですが、For loopを回す回数のところにvar.Count（Itemは5つです）で設定するとループが3回で止まってしまいます。

###問題が発生しているコード

       public static void RemoveDictionaryItem(SortedDictionary<int, Employee> var)
        {
            for (int i = 0; i <var.Count; i++)
            {
                var.Remove(i);
            }
        }

試しに

Console.WriteLine(var.Count);

で実行すると、ちゃんと「5」と表示されます。

###試したこと
Cast、Convert、var.Keys.Countすべて試しましたが結果は全て同じループが3回しか回りません。

for (int i = 0；i <Convert.ToInt32(var.Count); i++)
もしくは
for (int i = 0；i <(int)var.Count; i++)
もしくは
for (int i = 0；i <Convert.ToInt32(var.Keys.Count); i++)

ちなみにループに直接「5」とハードコードを入れると、ちゃんと最後まで回って削除してくれます。

        public static void RemoveDictionaryItem(SortedDictionary<int, Employee> var)
        {
            for (int i = 0; i < 5; i++)
            {
                var.Remove(i);
            }
        }

Answer 1

上記のChironianさんの解答を踏まえて、foreachで解決しました。

 public static void RemoveDictionaryItem(SortedDictionary<int, Employee> var)
        {
            foreach (var key in var.Keys.ToList())
            {
                var.Remove(key);
            }
        }

Answer 2

こんにちは。

削除しながらvar.Countを評価しているのでそうなりますよね。
Keyを全て取得して、その分削除してみてはどうですか。

foreach (var key in var.Keys.ToList())
{
    var.Remove(key);
}

もしほんとに例のとおり要素の削除だけしたいのであれば、var.Clear();で済みますが。

Comments

[mercredi]

すみません、今投稿に気が付きました。
ありがとうございます。

Answer 3

こんにちは。

1回Remove()する度に、var.Countが１つずつ減るからです。
2回目のRemove()の次のループでは、i=2、var.Count=3なのでもう一回だけループを回るということです。

Comments

[mercredi]

ありがとうございます。そうですね！
なんで気が付かなかったんだろう、、、信じられない。