15

与えられた配列でXより大きい最小の数を見つけるアルゴリズムにいくつかの最適化を行っていましたが、その後aiは奇妙な違いに遭遇しました。以下のコードでは、「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);
    }

ありがとう

4

2 に答える 2

52

IEnumerable<T>インデックス付けできません。

ループのすべての反復で呼び出す拡張メソッドCount()と拡張メソッドはO（n）です。コレクションをループして、カウントまたはn番目の要素を見つける必要があります。ElementAt()for

道徳：あなたのコレクションの種類を知っています。

于 2013-03-04T15:26:39.643 に答える
14

この違いの理由は、forループがbigList.Count()すべての反復で実行されるためです。Selectこれは、を実行して完全な結果セットを反復するため、実際にはコストがかかります。

さらに、これを使用しElementAtて、selectを再度実行し、指定したインデックスまで繰り返します。

于 2013-03-04T15:26:31.690 に答える