Re: Case-insensitive string compare?
- From: Maric Michaud <maric@xxxxxxxxxxxxx>
- Date: Fri, 5 Sep 2008 13:31:40 +0200
Le Friday 05 September 2008 08:24:29 Fredrik Lundh, vous avez écrit :
Maric Michaud wrote:
"premature optimization is the root of all evil"
So is use by that statement by people who don't have the slightest idea
about what it actually means.
The full version is
"We should forget about small efficiencies, say about 97% of the time:
premature optimization is the root of all evil."
Note the use of "small efficiencies". That is, in Python, things like
local binding, inlining, slots, lazy generation of short sequences, etc.
That is, things that takes time to write and reduces maintainability.
It's not about having an excuse for writing crappy code with large
inefficienies.
And it's definitely not about programmers intentionally picking a dumb
solution so they can optimize it later.
Yes, I'm aware of that, thanks, but Chris Rebert was proposing to alter the
data structure in order to achieve this simple comparison, I don't think it
fails into your concern.
> If the OP's proposal seems already messy, how about ths one :
> if lib.lower() not in ( e[0] for e in stage_map.items() ) :
> ...
Sorry, should be read :
if lib.lower() not in ( e[0] for e in stage_map.values() ) :
and it was the expression for comparison Chris's solution lead to, not a
proposal.
Given that your solution is just a remarkably inefficient way to write
"lib.lower() not in stage_map", and thus doesn't solve the OP:s problem,
I suspect you're taking the "evil" part of Hoare's dictum a few bits too
literally.
I suspect you are coming to conclusions a bit quickly, without taking the pain
of understanding the whole discussion.
</F>
--
http://mail.python.org/mailman/listinfo/python-list
--
_____________
Maric Michaud
.
- Prev by Date: Re: Python and Cyrillic characters in regular expression
- Next by Date: Re: path slashes cleaning
- Previous by thread: Re: Case-insensitive string compare?
- Next by thread: Re: Case-insensitive string compare?
- Index(es):
Relevant Pages
|
