ループ中にリストの要素を削除する場合の注意点

リストをループで回している中で、そのリストの要素を削除する場合は、ループを逆順で回しましょう。

NG

        for (int i = 0; i < enemyList.Count; i++) {
            if (!enemyList[i].IsAlive) {
                enemyList.RemoveAt(i);
            }
        }

OK

        for (int i = enemyList.Count - 1; i >= 0; i--) {
            if (!enemyList[i].IsAlive) {
                enemyList.RemoveAt(i);
            }
        }

なぜ逆順にしなければならないのか？を示したものが以下の図になります。NG側では、要素Cを削除した際に後ろにある要素が一つ前に詰められる影響で、要素Dが正しく処理されない問題が発生しています。

ちなみに、リストの要素を削除したいけどループ中に処理する必要はない時にはRemoveAllが便利です。

        enemyList.RemoveAll(enemy => !enemy.IsAlive);

参考リンク

• リスト（List）の列挙中にリスト要素を削除するには？