¿Por qué List <T> .ForEach permite modificar su lista?

Si uso:

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

el Adden el foreacharroja una InvalidOperationException (la colección fue modificada; la operación de enumeración puede no ejecutarse), lo cual considero lógico, ya que estamos tirando de la alfombra debajo de nuestros pies.

Sin embargo, si uso:

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

rápidamente se dispara en el pie haciendo un bucle hasta que arroja una OutOfMemoryException.

Esto me sorprende, ya que siempre pensé que List.ForEach era solo un envoltorio para foreacho para for.
¿Alguien tiene una explicación del cómo y el por qué de este comportamiento?

(Inspirado por el bucle ForEach para una lista genérica repetida sin cesar )

Es porque el ForEachmétodo no usa el enumerador, recorre los elementos con un forbucle:

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]);
    }
}

(código obtenido con JustDecompile)

Dado que el enumerador no se utiliza, nunca comprueba si la lista ha cambiado y la condición final del forciclo nunca se alcanza porque _sizeaumenta en cada iteración.

List<T>.ForEachse implementa por fordentro, por lo que no utiliza enumerador y permite modificar la colección.

Porque el ForEach adjunto a la clase List utiliza internamente un bucle for que está directamente adjunto a sus miembros internos, que puede ver descargando el código fuente para el marco .NET.

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

Mientras que un bucle foreach es, ante todo, una optimización del compilador, pero también debe operar contra la colección como observador, por lo que si la colección se modifica, lanza una excepción.

Sabemos de este problema, fue un descuido cuando se escribió originalmente. Desafortunadamente, no podemos cambiarlo porque ahora evitaría que se ejecute este código que anteriormente funcionaba:

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

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

La utilidad de este método en sí es cuestionable, como señaló Eric Lippert , por lo que no lo incluimos para .NET para aplicaciones de estilo Metro (es decir, aplicaciones de Windows 8).

David Kean (equipo BCL)