PyCQA/flake8-bugbear

B031 does not take into account if-else statements

Aperta

#465 aperta il 24 mar 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Python (116 fork)github user discovery
bughelp wanted

Metriche repository

Star
 (1116 stelle)
Metriche merge PR
 (Merge medio 4h 17m) (3 PR mergiate in 30 g)

Descrizione

Bugbear reports false-positive "more that one" usage of the generator even if it occurs inside 2 different branches of the if-else statement.

This code

import itertools

for _, group in itertools.groupby([]):
    if True:
        print(group, True)
    else:
        print(group, False)

produces

b031.py:7:15: B031 Using the generator returned from `itertools.groupby()` more than once will do nothing on the second usage. Save the result to a list, if the result is needed multiple times.

but it's obvious that group cannot be used twice here.

flake8 --version:

7.0.0 (flake8-bugbear: 24.2.6, mccabe: 0.7.0, pycodestyle: 2.11.1, pyflakes: 3.2.0) CPython 3.11.2 on Linux

Guida contributor

Direzione di ricerca
Esamina l'implementazione della regola B031 in flake8 bugbear.py per capire come rileva gli usi multipli di oggetti generator. Aggiungi logica per gestire i rami if else in modo che l'uso in rami separati non venga segnalato come falso positivo. La correzione probabilmente implica tracciare lo scopo e assicurarsi che l'uso in diversi rami delle istruzioni condizionali non venga conteggiato come uso multiplo.
Tech stack
python
Dominio
devtools
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPythonFlake8 plugin basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70