79 chars or more?

Neil Cerutti neilc at norwich.edu
Fri Aug 20 16:00:49 CEST 2010


On 2010-08-20, Bruno Desthuilliers <bruno.42.desthuilliers at websiteburo.invalid> wrote:
> make this :
>
>             self.expiration_date = translate_date(
>                 find(response, 'MPNExpirationDate').text,
>                 '%Y-%m-%d',
>                 '%m%d%Y'
>                 )
>
> I just HATE closing parens on the same line when the args don't
> fit on one single line.

It's been interesting to see my code scrutinized like this.
Thanks all. As a result of the discussion, I changed
translate_date to trans_date_from_to to make the order of
arguments clear.

find is just a small wrapper around Element.find calls, inserting
the namespace.

-- 
Neil Cerutti



More information about the Python-list mailing list