CA1868：针对集的对“Contains”不必要调用

  • 项目
属性 值
规则 ID CA1868
标题 不必要调用集合的 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)，不同之处在于，如果添加或删除了该项，则它们会返回一个新集合，如果没有添加或删除该项，则返回原始集合。

如何解决冲突

将对 ICollection<T>.Contains(T)（或 IImmutableSet<T>.Contains(T)）的调用（后跟对 ISet<T>.Add(T) 或 ICollection<T>.Remove(T)（或者 IImmutableSet<T>.Add(T) 或 IImmutableSet<T>.Remove(T)）的单次调用）替换为对后一种方法的单次调用。

示例

以下代码片段演示了 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

有关详细信息，请参阅如何禁止显示代码分析警告。