Utilisation de la variable itératrice de la boucle foreach dans une expression lambda - pourquoi échoue?

Question

Consultez le code suivant:

public class MyClass 
{ 
    public delegate string PrintHelloType(string greeting); 


    public void Execute() 
    { 

     Type[] types = new Type[] { typeof(string), typeof(float), typeof(int)}; 
     List<PrintHelloType> helloMethods = new List<PrintHelloType>(); 

     foreach (var type in types) 
     { 
      var sayHello = 
       new PrintHelloType(greeting => SayGreetingToType(type, greeting)); 
      helloMethods.Add(sayHello); 
     } 

     foreach (var helloMethod in helloMethods) 
     { 
      Console.WriteLine(helloMethod("Hi")); 
     } 

    } 

    public string SayGreetingToType(Type type, string greetingText) 
    { 
     return greetingText + " " + type.Name; 
    } 

... 

} 

Après avoir appelé myClass.Execute(), le code imprime la réponse inattendue suivante:

 
Hi Int32 
Hi Int32 
Hi Int32 

De toute évidence, je me attends "Hi String", "Hi Single", "Hi Int32", mais apparemment ce n'est pas Cas. Pourquoi le dernier élément du tableau itéré est utilisé dans toutes les 3 méthodes au lieu de la appropriée?

Comment réécrire le code pour atteindre l'objectif souhaité?

Answer 1

Bienvenue dans le monde des fermetures et des variables capturées :)

Eric Lippert a une explication approfondie de ce comportement:

• Closing over the loop variable considered harmful
• Closing over the loop variable, part two

essentiellement, c'est la variable de boucle qui est capturée, pas sa valeur. obtenir ce que vous pensez que vous devriez obtenir, faites ceci:

foreach (var type in types) 
{ 
    var newType = type; 
    var sayHello = 
      new PrintHelloType(greeting => SayGreetingToType(newType, greeting)); 
    helloMethods.Add(sayHello); 
} 

Answer 2

Vous pouvez corriger en introduisant variable supplémentaire:

... 
foreach (var type in types) 
     { 
      var t = type; 
      var sayHello = new PrintHelloType(greeting => SayGreetingToType(t, greeting)); 
      helloMethods.Add(sayHello); 
     } 
.... 

Answer 3

Comme une brève explication qui fait allusion aux messages de blog qui SWeko référencés, un lambda capture la variable , pas la valeur . Dans une boucle foreach, la variable n'est pas unique à chaque itération, la même variable est utilisée pour la durée de la boucle (ceci est plus évident lorsque vous voyez l'expansion que le compilateur effectue sur foreach au moment de la compilation). Par conséquent, vous avez capturé la même variable à chaque itération et la variable (à la dernière itération) fait référence au dernier élément de votre ensemble.

Mise à jour: Dans les versions les plus récentes de la langue (à partir de C# 5), la variable de boucle est considérée comme nouvelle à chaque itération, en fermant de manière plus qu'elle ne produit pas le même problème comme dans les anciennes versions (C# 4 et avant).