Path: csiph.com!usenet.pasdenom.info!news.etla.org!news.stack.nl!newsfeed.xs4all.nl!newsfeed4.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
MIME-Version: 1.0
In-Reply-To: <f9lxt.156155$tu1.23696@fx20.am4>
References: <f9lxt.156155$tu1.23696@fx20.am4>
From: Joshua Landau <joshua.landau.ws@gmail.com>
Date: Sat, 22 Jun 2013 19:20:22 +0100
Subject: Re: n00b question on spacing
To: Alister <alister.ware@ntlworld.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Cc: python-list <python-list@python.org>
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.3702.1371925271.3114.python-list@python.org>
Lines: 87
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:48939

On 22 June 2013 18:28, Alister <alister.ware@ntlworld.com> wrote:
> On Sat, 22 Jun 2013 17:11:00 +0100, Joshua Landau wrote:
>
>> On 22 June 2013 16:55, Rick Johnson <rantingrickjohnson@gmail.com>
>> wrote:
>>> On Saturday, June 22, 2013 10:40:24 AM UTC-5, Joshua Landau wrote:
>>>> > Plus, your use of the format syntax is incorrect.
>>>> Wut?
>>>
>>> Well what i mean exactly is not that it's illegal, i just find the use
>>> of the "getattr sugar", from WITHIN the format string, to be
>>> excessively noisy.
>>
>> Sure...
>> So in other words you realised you were wrong but you're too scared to
>> admit it. Eh? That's it, isn't it! You just don't want to loosen your
>> proud persona :P.
>
> In this argument I tend to find myself siding with Rick.
> Considering his general reputation in this forum am I missing something
> or do I need help? ;-)

I wasn't mocking the preference against it, but rather that that was
completely not what he said originally. One cannot let slip when the
mighty Rick misses his mark.

That said, yes, it is quite a noisy construct. I still prefer it to:

message =3D "Item wrote to MongoDB database {}/{}"
message =3D message.format(
        settings['MONGODB_DB'],
        settings['MONGODB_COLLECTION']
)
log.msg(message, level=3Dlog.DEBUG, spider=3Dspider)

Because this feels undescriptive - the first and "second" lines feel
detached. Instead of Rick's hilarious unpacking, you could always do:

message =3D "Item wrote to MongoDB database {database}/{collection}"
message =3D message.format(
        database=3Dsettings['MONGODB_DB'],
        collection=3Dsettings['MONGODB_COLLECTION']
)
log.msg(message, level=3Dlog.DEBUG, spider=3Dspider)

Which is probably as good, if longer. You'll see the real problem is
that "MONDODB_DB" and "MONGODB_COLLECTION" are awful names; would you
really have so detested

message =3D "Item wrote to MongoDB database "
message +=3D "{settings[database]}/{settings[collection]}".format(settings=
=3Dsettings)
log.msg(message, level=3Dlog.DEBUG, spider=3Dspider)

or even, now,

message =3D "Item wrote to MongoDB database
{settings[database]}/{settings[collection]}"
message =3D message.format(settings=3Dsettings)
log.msg(message, level=3Dlog.DEBUG, spider=3Dspider)

?

I think those options are quite nice, =C3=A1 mon avis. I could even write a
wrapper (oh noes!):

# Shortened for EMail
class SettingsWrapper:
    def uglify_key(self, key): return key if key.startswith("MONGODB")
else "MONGODB_" + key.upper()
    def __init__(self, settingsdict): self.wrapped =3D settingsdict
    def __len__(self): return self.wrapped.__len__()
    def __iter__(self): return self.wrapped.__iter__()
    def __getitem__(self, key): return self.wrapped[self.uglify_key(key)]
    def __contains__(self, key): return self.uglify_key(key) in self.wrappe=
d
    def __setitem__(self, key, value):
self.wrapped[self.uglify_key(key)] =3D value
    def __delitem__(self, key, value): del self.wrapped[self.uglify_key(key=
)]

settings =3D SettingsWrapper(settings)

message =3D "Item wrote to MongoDB database {settings[db]}/{settings[collec=
tion]}"
message =3D message.format(settings=3Dsettings)
log.msg(message, level=3Dlog.DEBUG, spider=3Dspider)