Linq: 集合が空でないことを判定するためにCount() > 0を使わないでください

クズども · 掲載地 2017/10/12 13:25:01

Linq以前は、集合が非空であるかどうか、すなわち集合が要素を含むかどうかを判定するために通常、以下の方法を用いました。

LengthやCount属性を使えば、上記の書き方に問題はありません。

しかしLinq時代には、Enumerable.Count拡張法がLengthとCountの性質を「統一」し、非零性を判断するための次の書き方を生み出しました。

この書き方は問題なく動作しますが、非常に深刻なパフォーマンスの問題を引き起こすことがあります。

ただし、これは可能であり必ずしもそうとは限らず、上記のメソッドがArray<T>、List、またはCollectionとして渡されれば<T>問題はありません。

では、いつ何かがうまくいかないのでしょうか? 以下の方法を見てみましょう。

ログインが見えます。

呼ばれた時:

ログインが見えます。

実行速度はかなり遅く、私のコンピュータがソースを実行するのに約70秒かかりました。 Count() > 0。

分析すると、GetNumsコードの5行目のyield returniはintを実行します。 MaxValue、それは必要ですか?

実際、1つの要素を返す限り、その集合は空でないこと、すべての要素を返す必要はないと結論づけられます。

では、どう判断すればいいのでしょうか? Enumerable.any 拡張メソッドを用いることができます:

SomeActionメソッドを次のように修正します:

ログインが見えます。

もう一度呼べば、実行時間はほとんど無視できるほどです。

まとめると、Count() > 0はイールドリターンに遭遇した際に必然的にパフォーマンスの問題を抱えます。

[出典] Linq: 集合が空でないことを判定するためにCount() > 0を使わないでください

関連記事

閲覧したセクション