MIME-Version: 1.0
In-Reply-To: <3ffa457e-7836-46d0-8246-03b6bd90a025@googlegroups.com>
References: <e738542a-f864-4ad0-9aaa-738047356841@googlegroups.com> <3ffa457e-7836-46d0-8246-03b6bd90a025@googlegroups.com>
From: Ian Kelly <ian.g.kelly@gmail.com>
Date: Wed, 12 Sep 2012 09:09:08 -0600
Subject: Re: generators as decorators simple issue
To: Python <python-list@python.org>
Content-Type: text/plain; charset=ISO-8859-1
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.566.1347462581.27098.python-list@python.org>
Lines: 68
NNTP-Posting-Host: 2001:888:2000:d::a6
Path: csiph.com!usenet.pasdenom.info!news.stben.net!border3.nntp.ams.giganews.com!border1.nntp.ams.giganews.com!nntp.giganews.com!news2.euro.net!newsfeed.xs4all.nl!newsfeed6.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
Xref: csiph.com comp.lang.python:28972

On Wed, Sep 12, 2012 at 4:22 AM, pyjoshsys <j.m.dagenhart@gmail.com> wrote:
> The output is still not what I want. Now runtime error free, however the output is not what I desire.

[SNIP]

> class Trial(object):
>     '''class to demonstrate with'''
>     def __init__(self):
>         object.__init__(self)
>         self.name = None
>
>     @classmethod
>     def SetName(cls, name):
>         cls.name = name

[SNIP]

> if __name__ == '__main__':
>     test = Test()
>     print 'instance'
>     print '', test.name #should be Test
>     print 'class'
>     print '', Test.name
>
>
> The output is: python decors2.py
> instance
>  None
> class
>  Test
>
> I want:
> instance
>  Test
> class
>  Test
>
> Is this possible in this manner?


The SetName class method sets the name on the *class* dictionary.  The
class's __init__ method also sets a name (None) on the *instance*
dictionary.  From an instance's perspective, the instance dictionary
will shadow the class dictionary.  If you remove the attribute from
the instance dictionary entirely (delete the "self.name = None" line),
and leave the class dictionary as is, then you will get the output you
want (although from your later post I am not certain that this is the
behaviour you want).


On Wed, Sep 12, 2012 at 5:15 AM, pyjoshsys <j.m.dagenhart@gmail.com> wrote:
> so decorators only pass the object and not any instance of the object as the implied argument? Is this right?

Right.

> The idea was to use @setname instead of instance.SetName(instance.__name__).

The appropriate place to do this so that it applies to all instances
of the class rather than to the class would be inside the __init__
method.

Also, instances don't have a __name__ attribute, so it's still unclear
to me what you're looking for.  Did you mean the effect to be that of
"instance.SetName(cls.__name__)"?  If so, then the decorator approach
(with the line "self.name = None" removed) should be fine for your
purposes -- you'll just have the name stored in the class dict instead
of in each instance dict, but it will still be visible as long as you
haven't shadowed it.