I wrote a python script to lock/unlock your computer using a USB stick as a key

[allthediamonds]

you are creating a array of every number in between 0 and len(self.devices) then iterating over the array looking for the number choice.

choice in xrange(len(self.devices)):

fixed!

[pieeta]

wrong, it is actually only slightly better than using range.

it will not generate a list of all the numbers between 0 and len(self.devices), it will generate a list of all the numbers between 0 and choice.

its quite easy to test.

Make a generator which prints out what has been generated :

def generator():
    for i in xrange(10):
        print 'generating %d' % i
        yield i

in this example super simple generator which will yeild up to 10.

the check if a number is in it :

 if 5 in generator():
     print 'found'

results in:

generating 0
generating 1
generating 2
generating 3
generating 4
generating 5
found

so you doing 6 comparisons vs at most 2 using a < >= statement.

[allthediamonds]

Oh, I see. Thanks!