Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

pylint: Using possibly undefined loop variable 'n'

Pylint says

W:  6: Using possibly undefined loop variable 'n'

... with this code:

iterator = (i*i for i in range(100) if i % 3 == 0)

for n, i in enumerate(iterator):
    do_something(i)

print n

because if the iterator is empty (for example []) n is undefined, ok. But I like this trick. How to use it in a safe way?

I think that using len(list(iterator)) is not the best choice because you have to do two loops. I think that using a counter and incrementing it is not very pythonic.

like image 230
Ruggero Turra Avatar asked Feb 26 '10 19:02

Ruggero Turra


2 Answers

Have you considered merely initializing n to None before running the loop?

like image 134
Bryan Oakley Avatar answered Nov 17 '22 19:11

Bryan Oakley


Define a default value for n before the for statement:

iterator = (i*i for i in range(100) if i % 3 == 0)

n=None
for n, i in enumerate(iterator):
    do_something(i)

print n
like image 4
jldupont Avatar answered Nov 17 '22 18:11

jldupont