有没有一种更简单的方法来编写涉及多个组合框和嵌套字典的代码,同时避免KeyError



我对Python还很陌生,在花了很多小时之后,我成功地在不需要问任何问题的情况下完成了这项工作,然而,它似乎可以被更好地重写,并且在某种程度上可以摆脱KeyError: ''。关键错误只会出现(并暂停函数(,直到我从每个组合框中选择了一个项目,然后由于我在函数中的数学运算,它才会恢复,但我无法找到另一种方法来编写它,以解决这个问题并使代码更紧凑。我相信有办法,但我绝对可以用一个指针指向正确的方向,谢谢!这是我的程序的一个更简单的演示版本:

import tkinter as tk
import tkinter.ttk as ttk
#DICTIONARIES#
weapondict = {"Bronze Sword": {"atk":32, "def":4}, "Iron Sword": {"atk":47, "def":5}}
shielddict = {"Bronze Shield": {"atk":3, "def":10}, "Iron Shield": {"atk":5, "def":27}}
#FUNCTION#
def selected(func):
a = weapondict[weaponvar.get()]["atk"]
b = shielddict[shieldvar.get()]["atk"]
atkvar.set(a + b)
c = weapondict[weaponvar.get()]["def"]
d = shielddict[shieldvar.get()]["def"]
defvar.set(c + d)
#WINDOWLOOP#
root = tk.Tk()
root.geometry("250x125")
#VARIABLES#
weaponvar = tk.StringVar()
shieldvar = tk.StringVar()
atkvar = tk.IntVar()
defvar = tk.IntVar()
#COMBOBOXES#
weaponbox = ttk.Combobox(root, height=5, state="readonly", values=list(weapondict.keys()), textvariable=weaponvar)
weaponbox.place(x=10, y=10, width=130)
weaponbox.bind('<<ComboboxSelected>>', func=selected)
shieldbox = ttk.Combobox(root, height=5, state="readonly", values=list(shielddict.keys()), textvariable=shieldvar)
shieldbox.place(x=10, y=70, width=130)
shieldbox.bind('<<ComboboxSelected>>', func=selected)
#LABELS#
atklabel = tk.Label(root, text='Atk Bonus:')
atklabel.place(x=150, y=10, width=70, height=20)
deflabel = tk.Label(root, text='Def Bonus:')
deflabel.place(x=150, y=70, width=70, height=20)
atktotal = tk.Label(root, textvariable=atkvar)
atktotal.place(x=155, y=30, width=100, height=20)
deftotal = tk.Label(root, textvariable=defvar)
deftotal.place(x=155, y=90, width=100, height=20)
root.mainloop()

目标只是从每个组合框中选择一个并取其指定值,将该整数添加到另一个组合框中以获得总数,同时解决keyerror问题,使代码更可读,更易于编辑。我想放多个盒子,每个盒子都有多个物品,用这种方法会变得很乱,提前谢谢!

问题是tk.StringVarget()方法调用在没有任何内容时将返回""。一个简单的解决方案是在两个字典中添加一个与该"空"键匹配的条目,并在相应的Combobox:中未选择任何内容(即零(时为它们提供一些相关值

#DICTIONARIES#
weapondict = {"": {"atk":0, "def":0},  # Use when nothing is selected.
"Bronze Sword": {"atk":32, "def":4},
"Iron Sword": {"atk":47, "def":5}}
shielddict = {"": {"atk":0, "def":0},  # Use when nothing is selected.
"Bronze Shield": {"atk":3, "def":10},
"Iron Shield": {"atk":5, "def":27}}

另一种选择是修改selected()函数,以在尝试使用weaponvar.get()shieldvar.get()返回的值之前检查它们是否是相应字典中的键。如果不是,则可以提供abcd的一些默认值。

这是完全可行的,但更改这两个词典似乎要容易得多

还要注意这两个嵌套字典是如何定义的。我发现在多行上做这件事可读性更强。

最新更新