Question

我正在对一个算法进行一些优化，该算法在给定的数组中找到大于X的最小数字，但随后我偶然发现了奇怪的差异。在下面的代码中，“ForeachUpper”在625ms结束，“ForUpper”结束，我相信，几个小时（疯狂地慢）。为什么这样？

  class Teste
{
    public double Valor { get; set; }

    public Teste(double d)
    {
        Valor = d;
    }

    public override string ToString()
    {
        return "Teste: " + Valor;
    }
}

  private static IEnumerable<Teste> GetTeste(double total)
    {
        for (int i = 1; i <= total; i++)
        {
            yield return new Teste(i);
        }
    }
    static void Main(string[] args)
    {
        int total = 1000 * 1000*30 ;
        double test = total/2+.7;

        var ieTeste = GetTeste(total).ToList();


        Console.WriteLine("------------");

        ForeachUpper(ieTeste.Select(d=>d.Valor), test);
        Console.WriteLine("------------");
        ForUpper(ieTeste.Select(d => d.Valor), test);
        Console.Read();
    }

    private static void ForUpper(IEnumerable<double> bigList, double find)
    {
        var start1 = DateTime.Now;

        double uppper = 0;
        for (int i = 0; i < bigList.Count(); i++)
        {
            var toMatch = bigList.ElementAt(i);
            if (toMatch >= find)
            {
                uppper = toMatch;
                break;
            }
        }

        var end1 = (DateTime.Now - start1).TotalMilliseconds;

        Console.WriteLine(end1 + " = " + uppper);
    }

    private static void ForeachUpper(IEnumerable<double> bigList, double find)
    {
        var start1 = DateTime.Now;

        double upper = 0;
        foreach (var toMatch in bigList)
        {
            if (toMatch >= find)
            {
                upper = toMatch;
                break;
            }
        }

        var end1 = (DateTime.Now - start1).TotalMilliseconds;

        Console.WriteLine(end1 + " = " + upper);
    }

由于

Answer 1

IEnumerable<T>不可编入索引。

在Count()循环的每次迭代中调用的ElementAt()和for扩展方法都是O（n）;他们需要循环遍历集合以查找计数或第n个元素。

道德：了解你的收藏类型。

Answer 2

这种差异的原因是您的for循环将在每次迭代时执行bigList.Count()。在您的情况下，这非常昂贵，因为它将执行Select并迭代完整的结果集。

此外，您正在使用ElementAt再次执行选择并将其迭代到您提供的索引。

C＃ - For vs Foreach - 巨大的性能差异

2 个答案: