setattr con kwargs, pythonic o no?

Question

Sto usando __init__() in questo modo in alcune classi ORM di SQLAlchemy che hanno molti parametri (fino a 20).

def __init__(self, **kwargs): 
    for k, v in kwargs.iteritems(): 
     setattr(self, k, v) 

È "pythonic" impostare attributi come questo?

Answer 1

Sì. Un altro modo per farlo è.

def __init__(self, **kwargs): 
    self.__dict__.update(kwargs) 

Answer 2

Per me sembra piuttosto pitonioso se ne hai solo bisogno in un posto nel tuo codice.

Il seguente link fornisce un approccio più 'generici' per lo stesso problema (ad esempio con un decoratore e alcune funzionalità extra), di dare un'occhiata a: http://code.activestate.com/recipes/551763/

Answer 3

Sì, se non c'è un modo "migliore" di fornire gli argomenti.

Ad esempio, utilizzando le classi ORM si parla, forse sarebbe più Python'y permettere ..

col = Varchar() 
col.index = True 
col.length = 255 

..rather di ..

col = Varchar(index = True, length = 255) 

Va bene che non è il miglior esempio, dal momento che il metodo **kwargs sarebbe effettivamente più bello .. ma il mio punto è che dovresti sempre considerare metodi alternativi per ottenere qualcosa, prima di usare cose a volte scoraggiate come **kwargs ..

Un'altra cosa da tenere a mente è che si potrebbe perdere il comportamento di un utente si aspetta, come l'aumento di un TypeError se l'utente fornisce un ARG parola chiave non valida, che potrebbe essere aggirato come ..

def __init__(self, **kwargs): 
    valid_kwargs = ['x', 'y', 'z'] 
    for k, v in kwargs.iteritems(): 
     if k not in valid_kwargs: 
      raise TypeError("Invalid keyword argument %s" % k) 
     setattr(self, k, v) 

Un ultima cosa da considerare:

class Hmm: 
    def __init__(self, **kwargs): 
     for k, v in kwargs.iteritems(): 
      setattr(self, k, v) 
    def mymethod(self): 
     print "mymethod should print this message.." 

x = Hmm(mymethod = None) 
x.mymethod() # raises TypeError: 'NoneType' object is not callable