Question

我对编程仍然很陌生，我遇到了一个问题，我似乎无法找到一个好的答案。基本上，我做了一个小程序，允许你添加和删除字典中的项目。这应该像一个RPG游戏的库存，你可以打开一个箱子，内容将被添加到库存中，当你使用物品时，它们将被从库存中删除。我想设置它，如果库存中的特定项目为零或更少，那么该项目就会从字典中消失。这是我遇到麻烦的地方。这是我完成的程序

 inventory = {}

def add_to_inventory(item_tuple):
    inventory[item_tuple[0]]=item_tuple[1]


def add_all_items(tuple_list):
    for tup in tuple_list:
        if tup[0] not in inventory:
            add_to_inventory(tup)
        else:
            inventory[tup[0]]+=tup[1]
    return inventory 

def remove_items(item,quantity):
    global inventory
    updated_inventory={}
    for key,value in inventory.items():
        if key==item:
           value-=quantity
           inventory[key]=value
           if inventory[key]<= 0:
              inventory[key]=0
    for key,value in inventory.items():
        if value!=0:
            updated_inventory[key]= value
    inventory=updated_inventory

这是工作中的程序：

>>> chest=[("healing potion",3),("sword",1),("bread",5),("gold",50)]
>>> inventory
{}
>>> add_all_items(chest)
{'bread': 5, 'healing potion': 3, 'gold': 50, 'sword': 1}
>>> remove_items("bread",3)
>>> inventory
{'bread': 2, 'healing potion': 3, 'gold': 50, 'sword': 1}
>>> remove_items("bread",2)
>>> inventory
{'healing potion': 3, 'gold': 50, 'sword': 1}
>>>

我对前两个函数的外观非常满意，但我的remove_items函数看起来非常草率，我知道必须有更好的方法来实现它，即使它在技术上有效。我尝试使用＆＃34; pop＆＃34;和＆＃34; del＆＃34;但是我会得到一个运行时错误，告诉我我的字典在迭代期间改变了大小。所以现在我不得不制作一个名为updated_inventory的全新库存，并最终在一个环形交叉流程之后设置它等于＆＃34;库存＆＃34;当它是正确的。就像我说的那样，我只在这个月待了一个月，所以我假设我可以在这里做得更好，但到目前为止，这是唯一有效的方法。从阅读它我可以看出它看起来很糟糕。关于我能做些什么更好的建议？谢谢您的帮助。

Answer 1

我不明白为什么你需要搜索项目列表。字典的要点是可以立即访问。

您也不需要复制字典。代替：

def remove_items(item,quantity):
    global inventory
    if item not in inventory: #make sure the item is in the inventory first
        return
    inventory[item] -= quantity
    if inventory[item] <= 0: #remove item if the quantity is less than 0
        inventory.pop(item,None)

如果值为零，则从库存（字典）中删除项目

1 个答案: