python面向对象的编程方法

Question

我已经定义了以下代码，但似乎有关于方法 load 和 damage 的问题。

（根据 ShadowRanger 的建议编辑）：

class RangedWeapon(Weapon):
        def __init__(self, name, min_dmg, max_dmg):
            super().__init__(name, min_dmg, max_dmg)
            self.shots=0

        def shots_left(self):
            return self.shots

        def load(self, ammo):
            if ammo.weapon_type()==self.name:
                self.shots+=ammo.get_quantity()
                ammo.remove_all()

        def damage(self):
            if self.shots==0:
                return 0
            else:
                self.shots-=1
                return super().damage()

_

bow = RangedWeapon('bow', 10, 40)
crossbow = RangedWeapon('crossbow', 15, 45)
arrows = Ammo('arrow', bow, 5)
bolts = Ammo('bolt', crossbow, 10)

bow.load(arrows)
print(bow.shots_left()) # should return 5
print(arrows.get_quantity()) #should return 0

但是对于print(bow.shots_left())，我得到了 0，而print(arrows.get_quantity()) 我得到了 5。它们是相反的。我认为我的问题是我没有加载弹药数量？我不太确定。任何帮助，将不胜感激。谢谢！

class Ammo(Thing):
    def __init__(self, name, weapon, quantity):
        self.name=name
        self.weapon=weapon
        self.quantity=quantity

    def get_quantity(self):
        return self.quantity

    def weapon_type(self):
        return self.weapon.name

    def remove_all(self):
        self.quantity=0

Answer 1

主要问题：Ammo 的 weapon_type 是一个方法，而不是属性或属性，并且您没有调用它，因此您将方法本身与 name 进行比较，而不是结果调用它。这就是load 什么都不做的原因；没有任何方法等于字符串。

其他问题：

看起来您是在类上调用方法，而不是在实例上。您将ammo（一个实例）作为参数传递，然后调用Ammo（类）上的方法。

同样，您的damage 方法可能应该调用super().damage() 而不是Weapon.damage()，因为后者不使用您的实例状态。而且你有错别字（shots vs. shot）应该使这段代码在其他方面不起作用。

短版：此代码被一百万种方式破坏，您在修复之前的问题时会遇到每种方式。