CA1868: セットに対する 'Contains' の不要な呼び出し
| プロパティ | 値 |
|---|---|
| ルール ID | CA1868 |
| Title | セットに対する 'Contains' の不要な呼び出し |
| [カテゴリ] | パフォーマンス |
| 修正が中断ありか中断なしか | なし |
| .NET 8 では既定で有効 | 提案として |
原因
ISet<T>.Add または ICollection<T>.Remove の呼び出しは、Contains の呼び出しによってガードされます。 または、IImmutableSet<T>.Add または IImmutableSet<T>.Remove の呼び出しは、IImmutableSet<T>.Contains の呼び出しによってガードされます。
規則の説明
ISet<T>.Add(T) と ICollection<T>.Remove(T) は両方とも検索を実行するため、ICollection<T>.Contains(T) の事前の呼び出しは冗長になります。 Add(T) または Remove(T) を直接呼び出すと、項目が追加されたか削除されたかを示すブール値が返されるため、より効率的です。
このロジックは、項目が追加または削除された場合に新しいセットを返すか、そうでない場合は元のセットを返す場合を除き、IImmutableSet<T>.Add(T) と IImmutableSet<T>.Remove(T) にも適用されます。
違反の修正方法
ISet<T>.Add(T) または ICollection<T>.Remove(T) (または IImmutableSet<T>.Add(T) または IImmutableSet<T>.Remove(T)) の呼び出しが続く ICollection<T>.Contains(T) (または IImmutableSet<T>.Contains(T)) の呼び出しを、後のメソッドの 1 回の呼び出しに置き換えます。
例
次のコード スニペットは、CA1868 の違反を示しています。
void Run(ISet<string> set)
{
if (!set.Contains("Hello World"))
{
set.Add("Hello World");
}
}
Sub Run(set As ISet(Of String))
If Not set.Contains("Hello World") Then
set.Add("Hello World")
End If
End Sub
次のコード スニペットでは違反を修正しています。
void Run(ISet<string> set)
{
set.Add("Hello World");
}
Sub Run(set As ISet(Of String))
set.Add("Hello World")
End Sub
どのようなときに警告を抑制するか
パフォーマンスが問題でない場合は、この警告を抑制しても問題ありません。
警告を抑制する
単一の違反を抑制するだけの場合は、ソース ファイルにプリプロセッサ ディレクティブを追加して無効にしてから、規則をもう一度有効にします。
#pragma warning disable CA1868
// The code that's violating the rule is on this line.
#pragma warning restore CA1868
ファイル、フォルダー、またはプロジェクトの規則を無効にするには、構成ファイルでその重要度を none に設定します。
[*.{cs,vb}]
dotnet_diagnostic.CA1868.severity = none
詳細については、「コード分析の警告を抑制する方法」を参照してください。
.NET