我正在试图弄泥,但在制作装备命令时我遇到了这个错误。
if item.types == "weapon":
AttributeError: 'unicode' object has no attribute 'types'
我想修复我的代码,以便用户输入装备剑,它会在他/她的库存中搜索该项目并应用统计数据。
这是我的商品代码
#weapons
class Weapon():
def __init__(self, name, ATK, gold, types):
self.name = name
self.ATK = ATK
self.gold = gold
self.types = types
club = Weapon("club", 1, 3, "weapon")
sword = Weapon("sword", 3, 10,"weapon")
axe = Weapon("axe", 4, 15,"weapon")
bow = Weapon("Hailey's bow", 10, 40,"weapon")
swordX = Weapon("Gods Sword", 8000, 8000,"weapon")
这是我的播放器代码
players[id] = {
"name": None,
"room": "Tavern",
"ATK": 5,
"hp": 20,
"inventory": {},
"armorName": None,
"armorPT": None,
"weaponName": None,
"weaponPT": None,
}
并持续我的装备命令代码
elif command == "equip":
x = params.lower()
rm = rooms[players[id]["room"]]
if x in players[id]["inventory"]:
item = players[id]["inventory"][x]
weapon = players[id]["weaponName"]
wStat = players[id]["weaponPT"]
armor = players[id]["armorName"]
aStat = players[id]["armorPT"]
hp = players[id]["hp"]
ATK = players[id]["ATK"]
if item.types == "weapon":
weapon = item.name
wStat = item.ATK
ATK += wStat
mud.send_message(id,"you equip %s" % weapon)
mud.send_message(id,"NEW ATK: %d" % ATK)
elif item.type == "armor":
armor = item.name
aStat = item.DEF
hp += aStat
mud.send_message(id,"you equip %s" % armor)
mud.send_message(id,"NEW DEF: %d" % hp)
else:
mud.send_message(id,"not a vaid item type")
else:
mud.send_message(id,"you dont have this item")
如果您认为我需要添加更多代码才有意义,请告诉我。
编辑:我想我发现问题所在,但我不知道如何解决它我相信它是在take命令和锄头它被添加到库存它被添加为字符串我不知道如何解决这个问题这是代码
elif command == "take":
x = params.lower()
rm = rooms[players[id]["room"]]
if rm["item"] == "yes":
if x in rm["itemName"]:
players[id]["inventory"][x] = x
del rm["itemName"][x]
mud.send_message(id, "you picked up %s" % x)
print players[id]["inventory"][x]
else:
mud.send_message(id,"You dont see that item")
else:
mud.send_message(id,"there is no item here")
答案 0 :(得分:1)
错误消息告诉item
是一个unicode字符串,而不是类Weapon
的实例。
因此,当您尝试访问item.types
时会出现错误。
N.B。还不是下一个条件应该是elif item.types == "armor":
(缺少s
)。
答案 1 :(得分:1)
OO编程中的一个“难闻的气味”是自己检查对象的类型。如果您正在检查类型,那么您很可能应该更改实施设计的方式。
在这种情况下,我建议您创建名为equip(self, obj)
,set_weapon(self, obj)
,set_armor(self, obj)
的Player方法,并创建名为equip_on(self, Player)
和{的各种对象方法{1}}。
然后,您可以根据类和类型划分权限和责任,以及哪些对象知道哪些事情:
unequip_on(self, Player)
可以处理权重限制并从包含它的Player.equip
中获取对象。room
会知道它是武器,并在Weapon.equip_on
上调用相应的方法。Player
会知道必须首先装备现有武器。Player.set_weapon
可以拒绝装备。如果你正确地实现了你的方法,它们中的每一个都会显得简单明了,但每次都会为图片添加更多的信息。这个双重调度是一种快速,简便的方法来处理类型信息,而无需在整个代码中使用Weapon.unequip_on
语句。