Expand to ``use-implicit-booleaness-not-len`` to catch `len(iterable) == 0` and `>0` · pylint-dev/pylint#10281

(24 commentaires) (0 réactions) (0 assignés)Python (1 059 forks)batch import

False Negative 🦋Good first issueHacktoberfestNeeds decision :lock:

Métriques du dépôt

Stars: (4 978 stars)
Métriques de merge PR: (Merge moyen 58j 21h) (63 PRs mergées en 30 j)

Description

Current problem

C1802 currently catches this

Examples 1&2 (currently caught by rule C1802)

fruits = ["orange", "apple"]
vegetables = []

if len(fruits):
    print(fruits)

if not len(vegetables):
    print(vegetables)

I come across the following sometimes as well (potentially in botched attempt to fix the the flagged violation above), which I think should be equally flagged as a violation.

Examples 3&4 (not caught by PLC1802)

fruits = ["orange", "apple"]
vegetables = []

if len(fruits) > 0:
    print(fruits)

if len(vegetables) == 0:
    print(vegetables)

Examples 5&6 (recommended formulation)

fruits = ["orange", "apple"]
vegetables = []

if fruits:
    print(fruits)

if not vegetables:
    print(vegetables)

Desired solution

Examples 3&4 to be flagged as well

Additional context

See our previous discussion on this issue.

Guide contributeur

Direction de recherche: La demande consiste à étendre la règle existante C1802 (use implicit booleaness not len) pour qu'elle signale également les cas comme `len(iterable) > 0` et `len(iterable) == 0`. Commencez par examiner l'implémentation actuelle du vérificateur dans le code source de pylint, probablement dans le répertoire `checkers/`. Consultez la discussion référencée à l'adresse https://github.com/astral sh/ruff/issues/16798#issuecomment 2729812152 pour obtenir un contexte supplémentaire. La solution implique d'ajouter une correspondance de motifs AST pour ces motifs supplémentaires dans la logique de la règle, de manière similaire à la détection de l'utilisation existante de `len()`.
Stack technique: python
Domaine: toolingdeveloper experience
Type d'issue: Fonctionnalité
Difficulté: 3
Temps estimé: 1-2 jours
Statut d'activité: Active
Clarté: Claire
Prérequis: Pythonpylint internalsAST knowledge
Accessibilité débutant: 45