Чому List <T> .ForEach дозволяє змінювати його список?

Якщо я використовую:

var strings = new List<string> { "sample" };
foreach (string s in strings)
{
  Console.WriteLine(s);
  strings.Add(s + "!");
}

Addв foreachвикидає InvalidOperationException (Колекція була змінена, операція перерахування не може виконати), який я вважаю логічним, так як ми висмикнув килим з - під ніг.

Однак, якщо я використовую:

var strings = new List<string> { "sample" };
strings.ForEach(s =>
  {
    Console.WriteLine(s);
    strings.Add(s + "!");
  });

він негайно вистрілює собі в стопу, циклічно, поки не викине OutOfMemoryException.

Це для мене несподіванка, оскільки я завжди думав, що List.ForEach був просто обгорткою для foreachабо для for.
Хтось має пояснення, як і чому така поведінка?

(Натхненний циклом ForEach для загального списку, що повторюється нескінченно )

Це тому, що ForEachметод не використовує перечислювач, він forциклічно перебирає елементи за допомогою циклу:

public void ForEach(Action<T> action)
{
    if (action == null)
    {
        ThrowHelper.ThrowArgumentNullException(ExceptionArgument.match);
    }
    for (int i = 0; i < this._size; i++)
    {
        action(this._items[i]);
    }
}

(код отримано за допомогою JustDecompile)

Оскільки перечислювач не використовується, він ніколи не перевіряє, чи змінився список, і кінцева умова forциклу ніколи не досягається, оскільки _sizeзбільшується на кожній ітерації.

List<T>.ForEachреалізовано через forinside, тому він не використовує перечислювач і дозволяє змінювати колекцію.

Оскільки ForEach, приєднаний до класу List, внутрішньо використовує цикл for, який безпосередньо приєднаний до його внутрішніх членів, - що ви можете побачити, завантаживши вихідний код для .NET framework.

http://referencesource.microsoft.com/netframework.aspx

Якщо цикл foreach - це, перш за все, оптимізація компілятора, але він також повинен діяти проти колекції як спостерігач - тож, якщо колекція модифікується, вона видає виняток.

Ми знаємо про цю проблему, вона була недоглядом, коли вона спочатку була написана. На жаль, ми не можемо його змінити, оскільки це заважало б виконувати цей раніше працюючий код:

        var list = new List<string>();
        list.Add("Foo");
        list.Add("Bar");

        list.ForEach((item) => 
        { 
            if(item=="Foo") 
                list.Remove(item); 
        });

Корисність цього методу є сумнівною, як зазначив Ерік Ліпперт , тому ми не включили його для .NET для додатків у стилі Metro (тобто додатків Windows 8).

Девід Кін (команда BCL)