[Tutor] Function design

Dave Angel davea at ieee.org
Tue Oct 27 17:14:58 CET 2009


Benno Lang wrote:
> On Tue, Oct 27, 2009 at 8:21 AM, Dave Angel <davea at ieee.org> wrote:
>   
>> I agree with Luke's comments.  But I'd like to point out an apparent bug (I
>> haven't tried the code, this is just by inspection).
>>
>> You use the test
>>     if '0' in row[.....]
>>
>> that's not going to check for a zero value, it's going to check for a zero
>> digit character somewhere in the value.  So 504 would also be recognized, as
>> well as 540.
>>     
>
> I thought this sounded rather fishy, along the following lines:
> row is a list, so slicing it returns a list, so 'in' should compare
> the list elements (i.e. the strings contained therein) with '0'; this
> seems perfectly normal.
>
> I have checked, and thankfully the world is not upside down. It would
> only work the way you describe if row was a string, or if 'in' called
> itself recursively when it found a list.
>
> HTH,
> benno
>
>   
You are quite right, of course.  I somehow missed that it was a slice, 
and took it instead to be a simple subscript.

But if the file contained  0.00 as one of its items, this code would 
still have the other problem I mentioned.

DaveA


More information about the Tutor mailing list