10

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.

Błażej Michalik
  • 4,474
  • 40
  • 55
Ruggero Turra
  • 16,929
  • 16
  • 85
  • 141

2 Answers2

14

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

Błażej Michalik
  • 4,474
  • 40
  • 55
Bryan Oakley
  • 370,779
  • 53
  • 539
  • 685
5

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
dbr
  • 165,801
  • 69
  • 278
  • 343
jldupont
  • 93,734
  • 56
  • 203
  • 318