Herencia de clase de Python AttributeError - ¿por qué? ¿como arreglar?

Question

Preguntas similares en SO incluyen: this one y this. También he leído toda la documentación en línea que puedo encontrar, pero todavía estoy bastante confundido. Estaría agradecido por tu ayuda.

Quiero usar el atributo .wandtype de la clase Wand en mi método Castuspell clase lumus. Pero sigo obteniendo el error "AttributeError: el objeto 'CastSpell' no tiene ningún atributo 'wandtype'."

Este código funciona:

class Wand(object): 
    def __init__(self, wandtype, length): 
     self.length = length 
     self.wandtype = wandtype 

    def fulldesc(self): 
     print "This is a %s wand and it is a %s long" % (self.wandtype, self.length) 

class CastSpell(object): 
    def __init__(self, spell, thing): 
     self.spell = spell 
     self.thing = thing 

    def lumus(self): 
     print "You cast the spell %s with your wand at %s" %(self.spell, self.thing) 

    def wingardium_leviosa(self): 
     print "You cast the levitation spell." 

my_wand = Wand('Phoenix-feather', '12 inches') 
cast_spell = CastSpell('lumus', 'door') 
my_wand.fulldesc() 
cast_spell.lumus() 

Este código, de intento de herencia, no.

class Wand(object): 
    def __init__(self, wandtype, length): 
     self.length = length 
     self.wandtype = wandtype 

    def fulldesc(self): 
     print "This is a %s wand and it is a %s long" % (self.wandtype, self.length) 

class CastSpell(Wand): 
    def __init__(self, spell, thing): 
     self.spell = spell 
     self.thing = thing 

    def lumus(self): 
     print "You cast the spell %s with your %s wand at %s" %(self.spell, self.wandtype, self.thing) #This line causes the AttributeError! 
     print "The room lights up." 

    def wingardium_leviosa(self): 
     print "You cast the levitation spell." 

my_wand = Wand('Phoenix-feather', '12 inches') 
cast_spell = CastSpell('lumus', 'door') 
my_wand.fulldesc() 
cast_spell.lumus() 

He intentado utilizar el método super() en vano. Realmente apreciaría su ayuda para entender a) por qué la herencia de clase no funciona en este caso, b) cómo hacer que funcione.

Answer 1

En pocas palabras, se anula Wand.__init__ en la clase que hereda de él, así CastSpell.wandtype no se encuentra en CastSpell. Además de eso, my_wand no puede pasar información al cast_spell, por lo que está confundido acerca del rol de la herencia.

Independientemente de cómo lo haga, de alguna manera debe pasar length y wandtype a CastSpell. Una forma sería incluirlos directamente en CastSpell.__init__:

class CastSpell(Wand): 
    def __init__(self, spell, thing, length, wandtype): 
     self.spell = spell 
     self.thing = thing 
     self.length = length 
     self.wandtype = wandtype 

Otra forma más genérica sería pasar estos dos para la clase base propia __init__():

class CastSpell(Wand): 
    def __init__(self, spell, thing, length, wandtype): 
     self.spell = spell 
     self.thing = thing 
     super(CastSpell, self).__init__(length, wandtype) 

Otra forma sería dejar de haciendo CastSpell heredar del Wand (es CastSpell una especie de Wand o algo un Wand hace?) y en lugar de hacer que cada varita poder tener algunas CastSpell s en ella: en lugar de "is-a" (una CastSpell es una especie de Wand), tratar "has-a" (a Wand tiene Spell s).

Aquí está una gran manera tan sencilla, no tener una tienda Varita hechizos:

class Wand(object): 
    def __init__(self, wandtype, length): 
     self.length = length 
     self.wandtype = wandtype 
     self.spells = {} # Our container for spells. 
     # You can add directly too: my_wand.spells['accio'] = Spell("aguamenti", "fire") 

    def fulldesc(self): 
     print "This is a %s wand and it is a %s long" % (self.wandtype, self.length) 

    def addspell(self, spell): 
     self.spells[spell.name] = spell 

    def cast(self, spellname): 
     """Check if requested spell exists, then call its "cast" method if it does.""" 
     if spellname in self.spells: # Check existence by name 
      spell = self.spells[spellname] # Retrieve spell that was added before, name it "spell" 
      spell.cast(self.wandtype) # Call that spell's cast method, passing wandtype as argument 
     else: 
      print "This wand doesn't have the %s spell." % spellname 
      print "Available spells:" 
      print "\n".join(sorted(self.spells.keys())) 


class Spell(object): 
    def __init__(self, name, target): 
     self.name = name 
     self.target = target 

    def cast(self, wandtype=""): 
     print "You cast the spell %s with your %s wand at %s." % (
       self.name, wandtype, self.target) 
     if self.name == "lumus": 
      print "The room lights up." 
     elif self.name == "wingardium leviosa": 
      print "You cast the levitation spell.", 
      print "The %s starts to float!" % self.target 

    def __repr__(self): 
     return self.name 

my_wand = Wand('Phoenix-feather', '12 inches') 
lumus = Spell('lumus', 'door') 
wingardium = Spell("wingardium leviosa", "enemy") 

my_wand.fulldesc() 
lumus.cast() # Not from a Wand! I.e., we're calling Spell.cast directly 
print "\n\n" 

my_wand.addspell(lumus) # Same as my_wand.spells["lumus"] = lumus 
my_wand.addspell(wingardium) 
print "\n\n" 

my_wand.cast("lumus") # Same as my_wand.spells["lumus"].cast(my_wand.wandtype) 
print "\n\n" 
my_wand.cast("wingardium leviosa") 
print "\n\n" 
my_wand.cast("avada kadavra") # The check in Wand.cast fails, print spell list instead 
print "\n\n" 

Answer 2

Debe llamar al método init de la superclase. De lo contrario, wandtype y length nunca se configuran en la instancia de CastSpell actual.

class CastSpell(Wand): 
    def __init__(self, spell, thing): 
     super(CastSpell, self).__init__(A, B) # A, B are your values for wandtype and length 
     self.spell = spell 
     self.thing = thing 

Como alternativa, puede añadir wandtype y duración como atributos en el objeto fuera del método init:

class Wand(object): 
    wandtype = None 
    length = None 

Entonces, ellos siempre estarán disponibles (aunque tendrán un valor de None hasta ellos han sido inicializados).

---

Sin embargo, ¿estás seguro de que CastSpell debería ser una subclase de Wand? CastSpell es una acción, que suena más como debería ser un método de Wand.

class Wand(object): 
    [...] 
    def cast_spell(self, spell, thing): 
     [etc.] 

Answer 3

Sí, super() no es lo que desea. Consulte this article para obtener detalles sobre por qué no.

Las llamadas normales a la superclase en Python se realizan (desafortunadamente) explícitamente al referirse a la superclase.

Si interpreto correctamente su pregunta, se estará preguntando por qué los atributos .length y .wandtype no se muestran en instancias de CastSpell. Esto es porque la Vara. init() método no se está llamando. Deberías hacerlo así:

class CastSpell(Wand): 
    def __init__(self, spell, thing): 
     Wand.__init__(self, whateverdefaultvalue_youwantforwandtype, default_value_for_length) 
     self.spell = spell 
     etc. 

Dicho esto, parece que no estás usando el derecho de herencia. CastSpell es una "acción" mientras que la varita es una "cosa". Esa no es realmente una abstracción que tenga sentido para la herencia.