CA1827: Any() が使用できる場合は、 Count() および LongCount() は使用しません

原因

Count()または LongCount() method は、Any() メソッドの方が効率的である場合に使用されました。

規則の説明

この規則は、コレクションに少なくとも 1 つの要素があるかどうかを確認するために使用される、LINQ の Count() および LongCount() メソッド呼び出しにフラグを付けます。 これらのメソッドでは、数を計算するためにコレクション全体を列挙します。 Any() メソッドを使用すると、コレクションの列挙が行なわれないので、同じチェックが高速になります。

違反の修正方法

違反を修正するには、Count または LongCount メソッド呼び出しを Any メソッドに置き換えます。 たとえば、次の 2 つのコード スニペットは、規則違反とその修正方法を示しています。

using System.Collections.Generic;
using System.Linq;

class C
{
    public string M1(IEnumerable<string> list)
        => list.Count() != 0 ? "Not empty" : "Empty";

    public string M2(IEnumerable<string> list)
        => list.LongCount() > 0 ? "Not empty" : "Empty";
}

using System.Collections.Generic;
using System.Linq;

class C
{
    public string M1(IEnumerable<string> list)
        => list.Any() ? "Not empty" : "Empty";

    public string M2(IEnumerable<string> list)
        => list.Any() ? "Not empty" : "Empty";
}

どのようなときに警告を抑制するか

数を計算するための不要なコレクションの列挙によるパフォーマンスへの影響が懸念されない場合は、この規則違反を抑制しても問題ありません。

警告を抑制する

単一の違反を抑制するだけの場合は、ソース ファイルにプリプロセッサ ディレクティブを追加して無効にしてから、規則をもう一度有効にします。

#pragma warning disable CA1827
// The code that's violating the rule is on this line.
#pragma warning restore CA1827

ファイル、フォルダー、またはプロジェクトの規則を無効にするには、構成ファイルでその重要度を none に設定します。

[*.{cs,vb}]
dotnet_diagnostic.CA1827.severity = none

詳細については、「コード分析の警告を抑制する方法」を参照してください。

関連規則

• CA1826:Linq の列挙可能なメソッドの代わりにプロパティを使用します
• CA1828:AnyAsync が使用できる場合は CountAsync/LongCountAsync を使用しません
• CA1829: Enumerable.Count() メソッドではなく Length/Count プロパティを使用します
• CA1860: 'Enumerable.Any()' 拡張メソッドを使用しない

関連項目

• パフォーマンス ルール