LINQ selectのBig O等価

代わりにLINQ選択を使用すると、ネストされたループのBig O等価に変更があるかどうかを確認しようとしています。

public void myFunc(List<Foo> fooList, List<Bar> barList)
{
    foreach(Foo foo in fooList)
    {
        foreach(Bar bar in barList)
        {
            if(foo.PropA == bar.PropA && bar.PropZ.HasValue)
                foo.PropC = foo.PropB * bar.PropZ;
        }
    }
}

このネストされたループの例は、複雑さのためにO（n ^ 2）であると思います。

ネストされたループを次のようなLINQ選択に置き換えました。

public void myFunc(List<Foo> fooList, List<Bar> barList)
{
    foreach(Foo foo in fooList)
    {
        Bar bar = (from b in barList
                    where foo.PropA == b.PropA
                    select b).FirstOrDefault();

        if(bar.PropZ.HasValue)
            foo.PropC = foo.PropB * bar.PropZ;
    }
}

他に何もない場合、コードは、「これを処理するために特定のものBarを探している」と明示的に述べているため、少し読みやすくなっています。

私の質問はこれです：LINQ selectを使用するとBig Oの複雑さが軽減されますか？

両方のコードが（最悪の場合）両方のリストの項目の各組み合わせについて比較を実行する必要があるため、Big Oは同じです。たとえば、リスト間に一致がない場合、Linq実装の最適化では、すべてのアイテムをチェックする必要がなくなります。

しかし、ねえ、あなたは本当にbig-Oについて知りたくないのですか？2番目の方が速いかどうかを知りたい。答えは「はい」です。linqベースのソリューションは、リスト内の最初の一致が見つかるまでチェックを実行するだけなので、linqベースのソリューションの方が高速です（十分な大きさがある限り）。 。このWhereメソッドはリスト全体をすぐにはスキャンしませんが、評価者は遅延イテレータを作成します。次に、FirstOrDefaultメソッドは最初の一致が見つかるまでイテレータを実行します。つまり、一般的なケースでは、リスト全体を走査する必要はありません。もちろん、Linqソリューションにはある程度のオーバーヘッドがあるため、nが十分に低い場合は、nested-forループの方が高速になります。

自分でソースをチェックアウトできます：https : //github.com/dotnet/corefx/blob/master/src/System.Linq/src/System/Linq/Enumerable.cs

（ただしbreak、最初のサンプルでを追加した場合、2つのコードは意味的に同等になり、オーバーヘッドが少ないため、最初のコードはより高速になります。）

複雑さに違いはありません。LinqのWhereはO（n）であるため、どちらもO（n²）です。
FirstOrDefaultの使用は、これを行うことと同等です。

if(foo.PropA == bar.PropA && bar.PropZ.HasValue) 
{
    foo.PropC = foo.PropB * bar.PropZ;
    break;
}

これは計算時間の改善ですが、複雑さは改善されません。

2つのリストのいずれかにHashmapを使用することで、より適切に実行できます。LINQのの参加 2つの一致する要素を検索するときにパフォーマンスに得ることができますので、オペレータは、2つのリストのいずれかのHashMapを作成します。

    public void myFunc(List<Foo> fooList, List<Bar> barList)
    {
        var rows = fooList.Join(
                        barList, 
                        f => f.PropA, 
                        b => b.PropA, 
                        (f, b) => new { Foo = f, Bar = b }
                   );

        foreach (var row in rows)
        {
            if (row.Bar.PropZ.HasValue)
                row.Foo.PropC = row.Foo.PropB * row.Bar.PropZ.Value;
        }
    }

これはO（n log（n））で実行する必要があります