Partage via

CA1827 : N’utilisez pas Count()/LongCount() quand Any() peut être utilisé

  • Article
Propriété Value
Identificateur de la règle CA1827
Titre Ne pas utiliser Count()/LongCount() quand Any() peut être utilisé
Catégorie Performances
Le correctif est cassant ou non cassant Sans rupture
Activé par défaut dans .NET 8 À titre de suggestion

Cause

La méthode Count() ou LongCount() a été utilisée où la méthode Any() serait plus efficace.

Description de la règle

Cette règle signale les appels de méthode LINQ Count() et LongCount() utilisés pour vérifier si la collection comporte au moins un élément. Ces méthodes énumèrent la totalité de la collection pour calculer le nombre. La même vérification est plus rapide avec la méthode Any(), car aucune énumération n’est alors nécessaire.

Notes

Cette règle est similaire à CA1860 : Évitez d’utiliser la méthode d’extension « Enumerable.Any() ». Toutefois, cette règle suggère d’utiliser la Count propriété, tandis que cette règle s’applique à la méthode d’extension Linq Count() .

Comment corriger les violations

Pour corriger une violation, remplacez l’appel de méthode Count ou LongCount par la méthode Any. Par exemple, les deux extraits de code suivants montrent une violation de la règle et indiquent comment la corriger :

using System.Collections.Generic;
using System.Linq;

class C
{
    public string M1(IEnumerable<string> list)
        => list.Count() != 0 ? "Not empty" : "Empty";

    public string M2(IEnumerable<string> list)
        => list.LongCount() > 0 ? "Not empty" : "Empty";
}

using System.Collections.Generic;
using System.Linq;

class C
{
    public string M1(IEnumerable<string> list)
        => list.Any() ? "Not empty" : "Empty";

    public string M2(IEnumerable<string> list)
        => list.Any() ? "Not empty" : "Empty";
}

Conseil

Un correctif de code est disponible pour cette règle dans Visual Studio. Pour l’utiliser, positionnez le curseur sur la violation et appuyez sur Ctrl+. (point). Choisissez Ne pas utiliser Count() ni LongCount() lorsque Any() peut être employé dans la liste des options qui vous est présentée.

Correctif de code pour CA1827 : Ne pas utiliser Count() ni LongCount() quand Any() peut être employé

Quand supprimer les avertissements

Vous pouvez sans risque supprimer une violation de cette règle si vous ne vous souciez pas de l’impact sur les performances de l’énumération inutile de la collection pour calculer le nombre.

Supprimer un avertissement

Si vous voulez supprimer une seule violation, ajoutez des directives de préprocesseur à votre fichier source pour désactiver et réactiver la règle.

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

Pour désactiver la règle sur un fichier, un dossier ou un projet, définissez sa gravité sur none dans le fichier de configuration.

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

Pour plus d’informations, consultez Comment supprimer les avertissements de l’analyse de code.

Voir aussi