クラスメソッドがPython 2.7.1で間違った数値を返す

Question

私はPythonを学ぶためのミニゲームを書いています。 main.pyファイルにインポートできる武器クラスを作成しました。ここで

は私が作ったクラスです。ここ

class weapon(object): 

    def __init__(self, name): 
     self.weaponName = name 

    def weaponStrength(self, level, strength): 
     self.weaponLevel = level 
     self.weaponStrength = strength 
     damage = self.weaponStrength * level 

     print "Damage is equal to %r" % damage 

     return damage 

は武器クラスを使用して作成されたオブジェクトです。

# Creates an Object called sword using the weaponsClass 
sword = weapon("sword") 

# Calls a method of the weaponsClass to calculate weapon Strength. Returns a int 
sword.weaponStrength(3, 20) 

# Creates an Object called Magic Staff using the weaponsClass 
magicStaff = weapon("Magic Staff") 

# Calls a method of the weaponsClass to calculate weapon Strength. Returns a int 
magicStaff.weaponStrength(5, 30) 


# Sets a variable 
swordStrength = sword.weaponStrength 

# Sets a variable 
magicStaffStrength = magicStaff.weaponStrength 

# Prints the variable 
print swordStrength 

# Prints the variable 
print magicStaffStrength 

私はswordStrengthとmagicStaffStrengthは、メソッドに渡された強度値に等しい理由を把握しようとしています。

ご迷惑をおかけして申し訳ありません。

ありがとうございました。あなたはweapon名前空間にweaponStrengthを上書きしている

Answer 1

：

self.weaponStrength = strength 

と

def weaponStrength(...): 

は実際に矛盾しています。以来、被害

私はこれは悪いデザインだと思います

を計算して返すweaponLevelとweaponStrength

を

保存

：たぶん、このメソッドは、2つのことを行うしようとしている

Answer 2

あなたの命名規則について考えます最初の目的は、関数がsetWeaponLevelAndStrengthのように呼び出されるべきであり、2番目の関数がcalculateDamageと呼ばれるべきであることを示唆しています。 weaponStrengthは明らかに名前の恐ろしい選択をするあなたも

def weaponStrength(self, level, strength): 
    self.weaponLevel = level    #first purpose 
    self.weaponStrength = strength   #first purpose 
    damage = self.weaponStrength * level #second purpose 

    print "Damage is equal to %r" % damage #second purpose 

    return damage 

その名前の属性を持っているしたい場合、私はあなたが2

def setWeaponLevelAndStrength(self, level, strength): 
    self.weaponLevel = level 
    self.weaponStrength = strength 

def calculateDamage(self) 
    damage = self.weaponStrength * self.weaponLevel 

    print "Damage is equal to %r" % damage 

    return damage