CA1827: Any() が使用できる場合は、 Count() および LongCount() は使用しません
プロパティ | 値 |
---|---|
ルール ID | CA1827 |
Title | Any() が使用できる場合は Count()/LongCount() を使用しません |
[カテゴリ] | パフォーマンス |
修正が中断ありか中断なしか | なし |
.NET 9 では既定で有効 | 提案として |
原因
Count()または LongCount() method は、Any() メソッドの方が効率的である場合に使用されました。
規則の説明
この規則は、コレクションに少なくとも 1 つの要素があるかどうかを確認するために使用される、LINQ の Count() および LongCount() メソッド呼び出しにフラグを付けます。 これらのメソッドでは、数を計算するためにコレクション全体を列挙します。 Any() メソッドを使用すると、コレクションの列挙が行なわれないので、同じチェックが高速になります。
Note
この規則は、CA1860: Enumerable.Any()' 拡張メソッドの使用は避けてください、と類似です。 ただし、この規則は Count
property の使用を提案しますが、この規則は Linq Count()
extension メソッドに適用されます。
違反の修正方法
違反を修正するには、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";
}
ヒント
Visual Studio では、この規則に対するコード修正を使用できます。 これを使用するには、違反にカーソルを置き、Ctrl+. (ピリオド) を押します。 表示されるオプションの一覧から [Any() を使用できる場合は、Count() または LongCount() を使用しません] を選択します。
どのようなときに警告を抑制するか
数を計算するための不要なコレクションの列挙によるパフォーマンスへの影響が懸念されない場合は、この規則違反を抑制しても問題ありません。
警告を抑制する
単一の違反を抑制するだけの場合は、ソース ファイルにプリプロセッサ ディレクティブを追加して無効にしてから、規則をもう一度有効にします。
#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()' 拡張メソッドを使用しない
関連項目
.NET