REPOST: Re: How best to check for existance of an attribute?

[Alex Martelli]

"Greg Krohn" <infinitystwin.SPAM at IS.BAD.yahoo.com> wrote in message
news:3c28f373$0$79558$6e49188b at news.goldengate.net...
> "Roy Smith" <roy at panix.com> wrote in message
> news:roy-DD8C6B.16193025122001 at news1.panix.com...
> > I've got a function which augments an object passed to it by adding a
new
> > attribute.  Something like this:
> >
> > def addStuff (obj):
> >    obj.newAttr = "stuff"
> >
> > I want it to be an error to call addStuff() more than once for a given
    ...
> This is what I would do:
>
> def addStuff (obj):
>     if hasattr(obj, 'stuff'):
>         raise MultipleCallError
>     else:
>         setattr(obj, 'stuff', None)

class Tricky:
    newAttr = "aDefault"

dick = Tricky()


Now, addStuff(dick) would fail.  Is this what the OP wants?  Maybe (some
of his examples "ways to do it" would behave the same way) but it doesn't
quite sound right given the specs of "calling more than once".

The point is just that hasattr also finds the attributes in the object's
class.  If you only want attributes _in the object itself_, NOT ones the
object gets from its class, it's a bit trickier, depending on what exact
range of cases you want to handle (e.g., can obj be assumed to have a
__class__ attribute, as in Python 2.2, or do you have to check for that,
too?).  Assuming only objects with a __class__ are of interest (or that
only Python 2.2 and up is of interest), then, maybe, a sentinel-like
approach might be simplest:

def addStuff(obj):
    class Unique: pass
    inclass = getattr(obj.__class__, 'stuff', Unique())
    inobject = getattr(obj, 'stuff', inclass)

    if inobject is inclass:
        obj.newAttr = "stuff"
    else:
        raise MultipleCallError, obj

This might still fail, depending on interning of "stuff",
for:
    addStuff(obj.__class__)
    addStuff(obj)
where the second call would be wrongly diagnosed as multiple.  Is
there a need to support this usage?

Testing if obj.__dict__.has_key('stuff') might often help,
but would still fail in 2.2 if obj didn't have a __dict__.
Is THIS a problem?

Only the OP knows...


Alex

========= WAS CANCELLED BY =======:
Path: news.sol.net!spool1-milwwi.newsops.execpc.com!newsfeeds.sol.net!newspump.sol.net!newsfeed.direct.ca!look.ca!newsfeed.dacom.co.kr!feeder.kornet.net!news1.kornet.net!ua4canc3ll3r
From: "Alex Martelli" <aleax at aleax.it>
Newsgroups: comp.lang.python
Subject: cmsg cancel <a0i5p4$od5$1 at serv1.iunet.it>
Control: cancel <a0i5p4$od5$1 at serv1.iunet.it>
Date: Mon, 31 Dec 2001 02:03:26 GMT
Organization: A poorly-installed InterNetNews site
Lines: 2
Message-ID: <cancel.a0i5p4$od5$1 at serv1.iunet.it>
NNTP-Posting-Host: 211.57.49.2
X-Trace: news2.kornet.net 1009775273 27193 211.57.49.2 (31 Dec 2001 05:07:53 GMT)
X-Complaints-To: usenet at news2.kornet.net
NNTP-Posting-Date: Mon, 31 Dec 2001 05:07:53 +0000 (UTC)
X-No-Archive: yes
X-Unac4ncel: yes
X-Commentary: I love NewsAgent 1.10 and the Sandblaster Cancel Engine Build 74 (19 March 1999)

This message was cancelled from within Mozilla.