--- Pete Forman <[EMAIL PROTECTED]> wrote: > Steve Howell <[EMAIL PROTECTED]> writes: > > > Feel free to change the page as you see fit, > although thanks for > > discussing it here first. > > Done. I've moved classes up as unittest depends on > it. > > The changes that I made to classes were: > > 1) Use new style class. > 2) Demonstrate Pythonic use of attribute (no > get/set). > 3) Pare some lines. > --
Thanks! Here's the link, if anybody wants to kibitz: http://wiki.python.org/moin/SimplePrograms My comments: 1) I'm not crazy about having to inherit from object for such a simple class, but I understand that it's best practice. 2) I'm glad you moved the class example up. 3) The median example raises the question of what would happen with an empty list. I don't think it's important to cover that case in the simple example (it's really the responsibility of the caller IMO not to call median with nonsense arguments), but perhaps there is a simple way to fix this? 4) Agreed on Pythonic use of the "balance" attribute. ____________________________________________________________________________________ Shape Yahoo! in your own image. Join our Network Research Panel today! http://surveylink.yahoo.com/gmrs/yahoo_panel_invite.asp?a=7 -- http://mail.python.org/mailman/listinfo/python-list
