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.