Compartir a través de


CA1868: llamada innecesaria a "Contains" para conjuntos

Propiedad Value
Identificador de la regla CA1868
Título Llamada innecesaria a "Contains" para conjuntos
Categoría Rendimiento
La corrección es problemática o no problemática Poco problemático
Habilitado de forma predeterminada en .NET 8 Como sugerencia

Causa

Una llamada ISet<T>.Add o ICollection<T>.Remove se restringe mediante una llamada Contains. O bien, una llamadaIImmutableSet<T>.Add o IImmutableSet<T>.Remove se restringe mediante una llamada IImmutableSet<T>.Contains.

Descripción de la regla

Tanto ISet<T>.Add(T) como ICollection<T>.Remove(T) realizan una búsqueda, lo que hace que sea redundante llamar a ICollection<T>.Contains(T) de antemano. Es más eficaz llamar a Add(T) o Remove(T) directamente, lo que devuelve un valor booleano que indica si el elemento se ha agregado o quitado.

Esta lógica también se aplica a IImmutableSet<T>.Add(T) y IImmutableSet<T>.Remove(T), salvo que devuelven un nuevo conjunto si se agrega o quita el elemento, o el conjunto original si no lo era.

Cómo corregir infracciones

Reemplace la llamada a ICollection<T>.Contains(T) (o IImmutableSet<T>.Contains(T)) seguida de una llamada a ISet<T>.Add(T) o ICollection<T>.Remove(T) (o IImmutableSet<T>.Add(T) ) IImmutableSet<T>.Remove(T)por una sola llamada al último método.

Ejemplo

El siguiente fragmento de código muestra una infracción 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

El siguiente fragmento de código corrige la infracción:

void Run(ISet<string> set)
{
    set.Add("Hello World");
}
Sub Run(set As ISet(Of String))
    set.Add("Hello World")
End Sub

Cuándo suprimir las advertencias

Es seguro suprimir esta advertencia si el rendimiento no es un problema.

Supresión de una advertencia

Si solo quiere suprimir una única infracción, agregue directivas de preprocesador al archivo de origen para deshabilitar y volver a habilitar la regla.

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

Para deshabilitar la regla de un archivo, una carpeta o un proyecto, establezca su gravedad en none del archivo de configuración.

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

Para obtener más información, consulte Procedimiento para suprimir advertencias de análisis de código.