Y a-t-il vraiment plusieurs itérations d'un IEnumerable dans ce code?

Question

public static IEnumerable<T> ForEach<T>(this IEnumerable<T> source, Action<T> action) 
    { 
     foreach (var element in source) action(element); 
     return source; 
    } 

Le code ci-dessus me donne un avertissement que la source est potentiellement itérée plusieurs fois, mais est-ce vraiment? Il est répété une fois dans l'instruction foreach, mais comment l'instruction return l'a-t-elle réitérée?

Answer 1

Le retour ne réitère pas l'énumérable. Mais vous le retournez, et puisque la seule façon dont vous pourriez faire quelque chose avec ce résultat est de l'itérer ailleurs, cet avertissement est donné. Pourquoi le retourner si vous n'allez pas l'itérer plus tard?

Answer 2

Dans le code actuel, vous montrez qu'il n'y a qu'une seule itération. Il est fort possible que vous l'énumériez encore ailleurs, puisque le même énumérable est renvoyé. C'est ce que l'avertissement vous dit.

Ma préférence personnelle serait de changer le type de retour du ForEach en void, et d'ajouter un second qui renvoie le résultat d'un Func<T, R>.

Alors:

public static void ForEach<T>(this IEnumerable<T> source, Action<T> action) 
{ 
    foreach (var element in source) 
     action(element); 
} 

Et:

public static IEnumerable<R> ForEach<T, R>(this IEnumerable<T> source, Func<T, R> action) 
{ 
    foreach (var element in source) 
     yield return action(element); 
} 

De cette façon, vous ne réutilisez accidentellement le même dénombrable et vous faites ce paresseux: il ne sera pas exécuté avant d'appeler réellement la dénombrable .