.Net - 什么时候 List.ForEach 优于标准 foreach 循环？

Question

通用列表类具有 .ForEach(Actionaction) 方法。 现在我已经对它们的执行方式做了一些简单计时，看起来通用的 ForEach 的性能较差。 （代码段编译器友好）代码如下 -

public static class timer{
    public static long foreachloop = 0;
    public static long Gforeachloop = 0;}

public class something{
    public List<string> myStrings = new List<string>();

    public something()
    {
        for(int i = 1; i<=5000000;i++)
        {
            myStrings.Add(i.ToString());
        }
    }}

public class cls1{
    private static List<string> Strings = new List<string>();
    private static List<string> OtherStrings = new List<string>();

    public static void RunSnippet()
    {
        something s = new something();

        Stopwatch watch = new Stopwatch();
        watch.Start();
        foreach(string x in s.myStrings)
        {
            Strings.Add(x);
        }
        watch.Stop();
        timer.foreachloop = watch.ElapsedMilliseconds;

        watch.Reset();
        watch.Start();

        s.myStrings.ForEach(delegate(string n){OtherStrings.Add(n);});

        s.myStrings.Clear();

        watch.Stop();
        timer.Gforeachloop = watch.ElapsedMilliseconds;

        WL("FOREACH-"+timer.foreachloop + ",Count = " + Strings.Count);
        WL("GFOREACH-"+timer.Gforeachloop + ",Count = " + OtherStrings.Count);
    }

    #region Helper methods

    public static void Main()
    {
        try
        {
            RunSnippet();
        }
        catch (Exception e)
        {
            string error = string.Format("---\nThe following error occurred while executing the snippet:\n{0}\n---", e.ToString());
            Console.WriteLine(error);
        }
        finally
        {
            Console.Write("Press any key to continue...");
            Console.ReadKey();
        }
    }

    private static void WL(object text, params object[] args)
    {
        Console.WriteLine(text.ToString(), args);   
    }

    private static void RL()
    {
        Console.ReadLine(); 
    }

    private static void Break() 
    {
        System.Diagnostics.Debugger.Break();
    }

    #endregion
}

FOREACH 的运行时间为 177 毫秒，GFOREACH 的运行时间为 707 毫秒。

现在我猜想使用它有一个很好的理由，但我就是想不出一个。 显然性能不是原因，所以问题是什么时候它是最佳选择？

提前致谢。

Answer 1

Eric Lippert 的这篇博客文章提供了背景信息：

http://blogs.msdn.com/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx

他正在谈论扩展方法的常见建议，以对 执行相同的操作IEnumerable，但哲学上的反对也适用于 List.ForEach。

这表明也许这种方法从来都不是一个好主意，尽管它看起来“很酷”。 使用 foreach 会更清楚。

我建议可以将此类方法视为 修复了经典的循环变量闭包错误。

但在实践中，我越来越善于发现此类错误。

Answer 2

当它看起来更整洁时。

根本不是开玩笑。 真的，我是说真的。 根据您的情况选择更具可读性的样式。 例如，如果您只想对每个项目调用一个方法，例如：

list.ForEach(Console.WriteLine);

这种风格更适合。 但是，如果您有一百行作为循环体，或者您有嵌套循环和控制流结构，则旧样式看起来更好。