On Wed, Oct 21, 2015 at 9:29 AM, Marcel Franke <
[email protected]> wrote:
>
> Edward K. Ream wrote:
>
>>
>> How would you improve, say:
>>
>> for p in c.all_positions():
>> print(p.h)
>>
>
>
>
> for position in commander.find(all=True):
>
> print(position.headline)
>
>
> or
>
>
> for position in commander.walk('/**'):
>
> print(position.headline)
>
>
> or
>
>
> print('\n'.join(node for node in document.walk()))
>
> #Ok, kidding ;)
>
Unconvincing.
>
> Well, going for that simple example:
>
>
> Single-letter-names for objects and attributes is very bad style.
>
The naming principle I use is this: the more important the name, the
shorter it should be. You have no chance of convincing me to change c, g,
p and v into longer names. Think E = MC**2.
>
Then there is the design-flaw to use a new method for every single
edge-case of a general functionality,
instead of using arguments and not poisoning the namespace
.
If you want your comments to be taken seriously, you should refrain from
nit picking. I could defend the names I have chosen, but really, it's not
worth my time nor anyone else's.
EKR
--
You received this message because you are subscribed to the Google Groups
"leo-editor" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/leo-editor.
For more options, visit https://groups.google.com/d/optout.
