Home Original page

Request for criticism - tell me how to become more Pythonic

Jon spam_buster_2000 at yahoo.com
Mon Sep 10 17:21:50 EDT 2001 
In article <9nj0tk$v2o$1 at slb2.atl.mindspring.net>, 
dalke at dalkescientific.com says...
> Jon:
> > If anyone has the time, I'd be interested in comments
> > about my style: what I am doing right, and what I need
> > to improve.
> 

Thanks to everyone that responded to my post (particularly Andrew Dalke, 
who seems to have spent far too much of his own precious time reading my 
horrible code :).

I've suddenly become a lot busier than I expected, so I won't have time 
to read everything you have written for the last couple of days. However, 
a couple of points --

1) This wasn't supposed to indicate how I write an immaculate, designed 
from the beginning piece of code. It was a challenge I set myself, to see 
whether I could write something in Python in a way that seemed 'obvious' 
to me, and actually have it work (which it did :). As others have pointed 
out, there are many ways I could refactor the code -- and as soon as my 
workload drops I probably will.

Responses to Andrew's comments:

2) 
> You could also recast your input code as a reader

Very good idea, and the skeleton code after is just the pointer I was 
looking for. As someone else noted, I'm orginally a C coder, and haven't 
really immersed myself in OO (apart from an optional software engineering 
course I took back when I was an undergrad). From my brief skimming, 
Python's approach to OO feels more like Pascal (or Modula-2) than C++ -- 
which is a good thing :). 

3)
>>              for (option, value) in options.items():
>>                     globals()[option] = value
>Very non-Pythonic. 

Sorry :). And looking at it with the benefit of sleep, you are right -- 
it's ugly and I should be ashamed. I still think Python is funky for 
allowing you to do this, however.

4) Particular thanks for your suggestions regarding how to deal with 
command line parsing -- again, I did it in the way that seemed obvious to 
me at the time. I am sure that, after close reading, your way will become 
obvious... 

'it won't take you more than a week of analysis to conclude that this is 
obvious' as a lecturer of mine once said... :)
More information about the Python-list mailing list