CA1868:针对集的对“Contains”不必要调用
属性 | 值 |
---|---|
规则 ID | CA1868 |
标题 | 不必要调用集合的 Contains |
类别 | “性能” |
修复是中断修复还是非中断修复 | 非中断 |
在 .NET 9 中默认启用 | 作为建议 |
原因
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
有关详细信息,请参阅如何禁止显示代码分析警告。