Warum erlaubt List <T> .ForEach, dass die Liste geändert wird?

Wenn ich benutze:

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

Das Addin den foreachWürfen eine InvalidOperationException (Sammlung wurde geändert; Aufzählungsoperation wird möglicherweise nicht ausgeführt), die ich für logisch halte, da wir den Teppich unter unseren Füßen hervorziehen.

Wenn ich jedoch benutze:

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

Es schießt sich sofort in den Fuß, indem es eine Schleife ausführt, bis eine OutOfMemoryException ausgelöst wird.

Dies ist eine Überraschung für mich, da ich immer dachte, dass List.ForEach entweder nur ein Wrapper für foreachoder für ist for.
Hat jemand eine Erklärung für das Wie und Warum dieses Verhaltens?

(Inspiriert von der ForEach-Schleife für eine endlos wiederholte generische Liste )

Dies liegt daran, dass die ForEachMethode den Enumerator nicht verwendet, sondern die Elemente mit einer forSchleife durchläuft :

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

(Code mit JustDecompile erhalten)

Da der Enumerator nicht verwendet wird, wird nie überprüft, ob sich die Liste geändert hat, und die Endbedingung der forSchleife wird nie erreicht, da sie _sizebei jeder Iteration erhöht wird.

List<T>.ForEachwird über forinside implementiert , verwendet also keinen Enumerator und ermöglicht das Ändern der Sammlung.

Da das an die List-Klasse angehängte ForEach intern eine for-Schleife verwendet, die direkt an die internen Mitglieder angehängt ist. Dies können Sie sehen, indem Sie den Quellcode für das .NET-Framework herunterladen.

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

Wobei eine foreach-Schleife in erster Linie eine Compileroptimierung ist, aber auch als Beobachter gegen die Sammlung arbeiten muss. Wenn also die Sammlung geändert wird, wird eine Ausnahme ausgelöst.

Wir wissen über dieses Problem Bescheid, es war ein Versehen, als es ursprünglich geschrieben wurde. Leider können wir es nicht ändern, da dies jetzt verhindern würde, dass dieser zuvor funktionierende Code ausgeführt wird:

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

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

Die Nützlichkeit dieser Methode selbst ist fraglich, wie Eric Lippert betonte, weshalb wir sie für .NET für Apps im Metro-Stil (dh Windows 8-Apps) nicht aufgenommen haben.

David Kean (BCL-Team)