Pourquoi List <T> .ForEach permet-il de modifier sa liste?

Si j'utilise:

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

le Adddans le foreachjette une InvalidOperationException (la collection a été modifiée; l'opération d'énumération peut ne pas s'exécuter), ce que je considère comme logique, car nous tirons le tapis de sous nos pieds.

Cependant, si j'utilise:

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

il se tire rapidement dans le pied en faisant une boucle jusqu'à ce qu'il lance une OutOfMemoryException.

Cela me surprend, car j'ai toujours pensé que List.ForEach n'était qu'un wrapper pour foreachou pour for.
Quelqu'un at-il une explication sur le comment et le pourquoi de ce comportement?

(Inspiré par la boucle ForEach pour une liste générique répétée à l'infini )

C'est parce que la ForEachméthode n'utilise pas l'énumérateur, elle parcourt les éléments avec une forboucle:

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 obtenu avec JustDecompile)

Puisque l'énumérateur n'est pas utilisé, il ne vérifie jamais si la liste a changé et la condition de fin de la forboucle n'est jamais atteinte car elle _sizeest augmentée à chaque itération.

List<T>.ForEachest implémenté à l' forintérieur, il n'utilise donc pas d'énumérateur et permet de modifier la collection.

Parce que le ForEach attaché à la classe List utilise en interne une boucle for qui est directement attachée à ses membres internes - que vous pouvez voir en téléchargeant le code source du framework .NET.

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

Où en tant que boucle foreach est avant tout une optimisation du compilateur mais doit également opérer contre la collection en tant qu'observateur - donc si la collection est modifiée, elle lève une exception.

Nous connaissons ce problème, c'était un oubli lors de sa rédaction initiale. Malheureusement, nous ne pouvons pas le changer car cela empêcherait maintenant ce code précédemment fonctionnel de s'exécuter:

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

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

L'utilité de cette méthode elle-même est discutable comme l'a souligné Eric Lippert , nous ne l'avons donc pas incluse pour les applications de style .NET pour Metro (c'est-à-dire les applications Windows 8).

David Kean (équipe BCL)