python - 如果值为零，则从库存(字典)中删除项目

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 函数看起来非常草率，而且我知道必须有更好的方法来实现它，即使它在技术上是可行的。我尝试使用“pop”和“del”，但是我会收到一个运行时错误，告诉我我的字典在迭代过程中改变了大小。所以现在我必须制作一个名为 updated_inventory 的全新库存，并最终在迂回过程后将其正确设置为等于“库存”。就像我说的，我只在这里待了一个月，所以我假设我可以在这里做得更好，但到目前为止，这是唯一可行的方法。我可以从阅读中看出它看起来设计不佳。关于我可以做得更好的任何建议？谢谢您的帮助。

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)