CA1868: Chamada desnecessária para 'Contém' para conjuntos

Artigo
12/16/2023

Property	valor
ID da regra	CA1868
Título	Chamada desnecessária para 'Contém' para conjuntos
Categoria	Desempenho
A correção está quebrando ou não quebrando	Sem quebra
Habilitado por padrão no .NET 8	Como sugestão

Motivo

Uma ISet<T>.Add chamada ou ICollection<T>.Remove é protegida por uma chamada para Contains. Ou, uma chamada ou IImmutableSet<T>.Remove é protegida por uma IImmutableSet<T>.Add chamada para IImmutableSet<T>.Contains.

Descrição da regra

Ambos ISet<T>.Add(T) e ICollection<T>.Remove(T) realizar uma pesquisa, o que torna redundante ligar ICollection<T>.Contains(T) antecipadamente. É mais eficiente chamar Add(T) ou diretamente, o que retorna um valor booleano indicando se o item foi adicionado ou Remove(T) removido.

Essa lógica também se aplica a IImmutableSet<T>.Add(T) e IImmutableSet<T>.Remove(T), exceto que eles retornam um novo conjunto se o item for adicionado ou removido, ou o conjunto original se não foi.

Como corrigir violações

Substitua a chamada para (ou ) seguida por uma chamada para ou (ou ou IImmutableSet<T>.Add(T)ICollection<T>.Remove(T)IImmutableSet<T>.Contains(T)IImmutableSet<T>.Remove(T)) por uma única chamada para ICollection<T>.Contains(T)ISet<T>.Add(T) o último método.

Exemplo

O seguinte trecho de código mostra uma violação de 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

O trecho de código a seguir corrige a violação:

void Run(ISet<string> set)
{
    set.Add("Hello World");
}

Sub Run(set As ISet(Of String))
    set.Add("Hello World")
End Sub

Quando suprimir avisos

É seguro suprimir esse aviso se o desempenho não for uma preocupação.

Suprimir um aviso

Se você quiser apenas suprimir uma única violação, adicione diretivas de pré-processador ao seu arquivo de origem para desativar e, em seguida, reativar a regra.

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

Para desabilitar a regra para um arquivo, pasta ou projeto, defina sua severidade como none no arquivo de configuração.

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

Para obter mais informações, consulte Como suprimir avisos de análise de código.

Partilhar via