Luke Paireepinart rabidpoobear at gmail.com
Thu Aug 12 05:09:49 CEST 2010

```Don't pass none. It's an optional parameter. It's accepted practice to not pass optional parameters.

Sent from my iPhone

On Aug 11, 2010, at 6:46 PM, Bill Allen <wallenpb at gmail.com> wrote:

> On Tue, Aug 10, 2010 at 11:11 AM, Hugo Arts <hugo.yoshi at gmail.com> wrote:
>> On Tue, Aug 10, 2010 at 9:59 AM, bob gailer <bgailer at gmail.com> wrote:
>>> On 8/10/2010 10:42 AM, Bill Allen wrote:
>>>>
>>>> Bob,
>>>>
>>>> I was really off on that algorithm and had way over complicated it.
>>>> I have it working correctly now, but for the sake of those who saw my
>>>> earlier really botched code, here is the resultant code that works.
>>>> The entire inner loop and intermediate variables are removed, but
>>>> shown commented out.  I had somehow thought I needed to read each
>>>> member of each subarray in individually.  That was not the case and
>>>> that inner loop was overwriting the array.
>>>>
>>>> # reads one line at a time from file and puts data into array
>>>>     for line in textf:
>>>>         #tempwords = line.split(None)
>>>>         #for n in range(0, len(room)-1):
>>>>         #    roomx[n] = tempwords[n]
>>>>         #room[m] = roomx
>>>>         room[m] = line.split(None)
>>>>         m += 1
>>>>
>>>>
>>>
>>> Great. Good work. Teach a man to fish?
>>>
>>> Now for the refinements. First you can use enumerate to obtain the room
>>> index:
>>>
>>>    for m, line in enumerate(textf):
>>>        room[m] = line.split(None)
>>>
>>>
>>>
>>>    rooms = []
>>>    for line in textf:
>>>        rooms.append(line.split(None))
>>>
>>> Third you can use list comprehension:
>>>
>>>    rooms = [line.split(None) for line in textf]
>>>
>>
>> Nice going, this is the best way of getting your answer from the
>> tutors list IMO, high fives all around. I wish more threads went like
>> this.
>>
>> You don't actually have to specify the None in line.split AFAIK, but
>> it might be better to be explicit. thoughts on this, anyone?
>>
>> Hugo
>>
>
> Hugo,
>