t+1 - Latest Comments in Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

cheap condoms — Mon, 21 Sep 2009 06:25:59 -0000

In my opinion, __init__ is for initializing stuff, so what's wrong with initializing instance attributes there?You can always break up the argument parsing bit from the default values bit with a comment.

Re: Instead of setting instance attributes within init

bc — Thu, 26 Mar 2009 11:06:14 -0000

Here's an indented version, in case it's not obvious how it should have read:

class MyClass(HasTraits):
....a = Float(1.234) #default value declared statically
.
.
class AnotherClass(HasTraits):
....a = Float()

....def _a_default(self): #dynamic init setup using naming convention
........return 1.234

Re: Instead of setting instance attributes within init

bc — Thu, 26 Mar 2009 11:03:05 -0000

You might be interested in Traits (http://code.enthought.com/p.... Traits-based classes use declarative (static) initialisation or dynamic (on demand) initialisation like so:

from enthought.traits.api import HasTraits, Float

class MyClass(HasTraits):
a = Float(1.234) #default value declared statically

class AnotherClass(HasTraits):
a = Float()

def _a_default(self): #dynamic init setup using naming convention
return 1.234

for the later, the 'a' attribute is calculated on demand.

Traits also does runtime type checking, notification, delegation and GUI-generation. There are also Property traits which are cacheable and handle depenencies. There are predefined traits for all the standard types and you can subclass your own.

Re: Instead of setting instance attributes within init

saluk — Thu, 26 Mar 2009 03:31:16 -0000

I have the opposite opinion - __init__ should not be doing anything OTHER than setting the default attributes. When I type a = Object(), I want a to be valid. Putting too much stuff inside of __init__ can make it harder to see why something is actually failing. Sometimes though, I do delegate out to other functions from __init__ which do more work if necessary.

The above just looks like a mess!

Re: Instead of setting instance attributes within init

Ben Finney — Wed, 25 Mar 2009 18:34:44 -0000

One problem is it's more verbose, and rather unusual. The reader isn't going to expect what you're doing, which is always a negative when deciding whether to be clever in code :-)

Another problem is you've defined only a read-only property: the decorator wraps the getter, but the property has no setter (or deleter or docstring).

Re: Instead of setting instance attributes within init

Wyatt Baldwin — Wed, 25 Mar 2009 15:31:52 -0000

>>> class C(object):
............@property
............def a(self):
................try:
....................self._a
................except AttributeError:
....................# Do possibly expensive stuff here
....................self._a = <result of="" possibly="" expensive="" stuff="">
................return self._a

Re: Instead of setting instance attributes within init

Wyatt Baldwin — Wed, 25 Mar 2009 15:25:02 -0000

I use this technique frequently, though with a different style, and usually just for something that's expensive to calculate (and can be cached):

>>> class C(object):
...
... @property
... def a(self):
... try:
... self._a
... except AttributeError:
... # Do possibly expensive stuff here
... self._a = <result of="" possibly="" expensive="" stuff="">
... return self._a

I think this has better performance than using hasattr. Also, I wonder if having a @property and instance variable with the same name would cause problems somewhere; that's why I use `_a`.

Note: I didn't read other comments first. Sorry for any repetition.

Re: Instead of setting instance attributes within init

sk — Wed, 25 Mar 2009 15:18:24 -0000

How about initializing instance variables first thing inside __init__. Then you can call descriptively named methods that each do their own "interesting thing". Finally, one line above the class declaration can be a class decorator (PDF) that "makes sure that instances of my class have a bunch of attributes".

Re: Instead of setting instance attributes within init

Paul Hildebrandt — Wed, 25 Mar 2009 11:07:04 -0000

Seems like a lot of mechanism. If the goal is to reduce the size of __init__ why not use a function like
def __init_vars(self):
self.a=1
self.b=2
and call that from the __init__ function.

In both cases I would worry about tools not understanding what is going on. Pulling the variable initialization out of the __init__ function will fool programs like pylint and doxygen.

I would probably just create sections in my __init__ like:

def __init(self):
#
# Variable
#
self.a=1
self.b=2

#
# Object setup
#
do interesting stuff here

But then again I do a lot of support programming and I like boring simple programs. They make it easier for me to understand.

Re: Instead of setting instance attributes within init

Rodrigo — Wed, 25 Mar 2009 10:22:09 -0000

Guess you should really use the __init__ methods to initialize your attributes. If you really don't want to do that, you can use some metaprogramming though:

class Test:
def __getattr__(self,attr):
self.__dict__[attr] = 1
return self.__dict__[attr]

Re: Instead of setting instance attributes within init

Chuck — Wed, 25 Mar 2009 10:21:38 -0000

why not something like:

class C(object):
a = 1
b = 2
def __init__(self):
# Init code goes here

Re: Instead of setting instance attributes within init

James — Wed, 25 Mar 2009 09:26:00 -0000

Oops, comments erased my spaces. Hopefully more readable version:

>>>class D(object):
... __slots__ = ('a', 'b')
... def __init__(self, attr_defaults={'a': 1,'b': 2}):
... [setattr(self, key, value) for key, value in attr_defaults.items()]

Re: Instead of setting instance attributes within init

James — Wed, 25 Mar 2009 09:24:08 -0000

If you're really worried about length, I guess you could do something like this:

class D(object):
__slots__ = ('a', 'b')

def __init__(self, attr_defaults={'a': 1,'b': 2}):
[setattr(self, key, value) for key, value in attr_defaults.items()]

...seems to me to retain readability, although it's not the usual way of doing things.

Re: Instead of setting instance attributes within init

Jack Diederich — Wed, 25 Mar 2009 08:58:34 -0000

The best reason to do it in __init__ is because everyone reading your code expects it to be done in __init__. In a few cases I've done it in a reset() method that is the first thing called in __init__.

Also, are you sure you're not getting a loop in that 'a' method? The first time you actually access the property it should loop and eventually bail with an error. [I just tried it - it quits after 999 recursive calls and raises an AttributeError]

Re: Instead of setting instance attributes within init

Floris Bruynooghe — Wed, 25 Mar 2009 05:46:53 -0000

Maybe you shouldn't try to do real work in __init__()?

http://misko.hevery.com/cod...

I'm not saying this is always the best solution, but do find that when I think about this while wanting to do real work in __init__() often I find an alternative way of doing things that's rather elegant. And the line of what "real" work is can be quite fine too sometimes.

Oh, I also agree with the comments about properties taking up so many more lines.

Re: Instead of setting instance attributes within init

Noah Gift — Wed, 25 Mar 2009 05:05:48 -0000

Excellent question! Because we don't want Python to be like Perl :) Recently I stumbled on this quote from Damian Conway in Perl Best Practices,
"Perl's approach to object oriented is almost excessively Perlish: there are far too many ways to do it....
There are just so many possible combinations of implementation, structure, and semantics that it's quite rare
to find two unrelated class hierarchies that use precisely the same style of Perl OO."

Re: Instead of setting instance attributes within init

Simon Wittber — Wed, 25 Mar 2009 04:15:32 -0000

Have you tried to run your code, it does not, and can never work.

>>> class D(object):
... @property
... def a(self):
... if not hasattr(self, 'a'):
... self.a =1
...
>>>
>>> d = D()
>>> print d.a
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "<stdin>", line 5, in a
AttributeError: can't set attribute
>>>

Re: Instead of setting instance attributes within init

Paddy3118 — Wed, 25 Mar 2009 03:44:16 -0000

Hi,
Every property takes 5 lines instead of 1? And you were worried about length???

You could call a separate method from init to do the setting.

- Paddy.

Re: Instead of setting instance attributes within init

Simon Hibbs — Wed, 25 Mar 2009 03:12:56 -0000

What ben said. This way I'd have to look through the much more verbose code for every property to get a view of the default values, rather than having them all in one place in a more concise form.

You can always break up the argument parsing bit from the default values bit with a comment.

Simon Hibbs

Re: Instead of setting instance attributes within init

Richard Jones — Wed, 25 Mar 2009 01:20:04 -0000

From the example it looks like you just want some class attributes holding those default values...

Re: Instead of setting instance attributes within init

Benjamin W. Smith — Tue, 24 Mar 2009 23:29:14 -0000

I can't speak for any technical pitfalls you may have, but I do have a concern with maintainability..

In my opinion, __init__ is for initializing stuff, so what's wrong with initializing instance attributes there? Seems to me that your way abstracts that in the sense that it takes what would I would normally look for in __init__ and put it somewhere else.. Just a thought that I'm sure you've had already :)

t+1 - Latest Comments in Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

Re: Instead of setting instance attributes within __init__

t+1 - Latest Comments in Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init

Re: Instead of setting instance attributes within init