C# for文をLinqで書きかえるときに途中returnをどう表現するか

Question

c#始めたばかりの初心者です。
現在、for文で書いてあるところを出来るだけLinqで書きかえようとしているところです。

もともと

public bool test(int[] array){
    for (int i = 0; i < array.Length; i++)
    {
        if (array[i] > 0) return false;
    }
    return true;
}

このような関数がありました。
引数には要素数40弱の0～4のintが入っている配列が渡されます。
これを

return !array.Where(i=>i>0).ToList().Any();

このように書きかえました。

一応期待している真偽値が返ってくるのですが、引数の配列は最初のほうに0じゃない数が入っていることも多く、このLinqの書き方だと最後まで必ずチェックしてから空かどうか判定しているので、元のfor文より処理速度が遅いんじゃないかと思っています。

そこで、

• 気にするほど処理速度に差はないのか
• 途中returnと同じ処理をLinqで書こうと思ったらどう書けばいいのか

お教え頂ければ幸いです。

Answer 1

ToList()が無駄なので、この方がいいと思います。

return !array.Any(i => i > 0);

Comments

[michael-ilcsy]

回答ありがとうございます！
まだLinqのことが分かっておらずAnyがListにしないと使えないと思っていたので勉強になりました。

Answer 2

ANYの実装
https://referencesource.microsoft.com/#System.Core/System/Linq/Enumerable.cs,1305

        public static bool Any<TSource>(this IEnumerable<TSource> source, Func<TSource, bool> predicate) {
            if (source == null) throw Error.ArgumentNull("source");
            if (predicate == null) throw Error.ArgumentNull("predicate");
            foreach (TSource element in source) {
                if (predicate(element)) return true;
            }
            return false;
        }

ほぼ同じことをしています。

Comments

[michael-ilcsy]

回答ありがとうございます！
他の回答とあわせて考えると　!array.Any(i => i > 0);　こう書けば元の関数と同じことをしている(条件外の要素があれば即return)しているという認識で大丈夫ですか？

[kiichi54321]

そうです。Any()は、あれば、即終わります。
一方、Count()は、すべてを見るので、非効率です。
存在確認だけなら、Any()を使います。

[kiichi54321]

https://referencesource.microsoft.com/#System.Core/System/Linq/Enumerable.cs　
ここに、LINQの実コードがあるので、読むといいと思います。

[michael-ilcsy]

ありがとうございます！
少し難しいですが、ちょこちょこソースも読んでいきたいと思います。