如何编写函数从文本文件生成货币转换按钮？

完全可行，不过你当前的代码存在几处逻辑问题，导致无法正常生成按钮，以下是问题梳理和修正后的代码：

原代码存在的问题 #

• 不必要的全局变量：文件对象c、字典cdict、按钮列表buttons无需设为全局，局部变量即可满足需求，全局变量会增加代码维护成本
• 无效循环：第一个遍历字典的循环for i in cdict: i=Button1()完全无意义，既没有关联货币名称，也没有将创建的Button1实例保存到buttons列表
• 未定义变量与逻辑混乱：第二个循环中的i.name()、i1均未定义，且Button(...)调用后直接链式调用pack()，会将pack()返回的None赋值给i，导致丢失按钮对象引用
• 文件未关闭：打开文件后未手动关闭，存在资源泄漏风险

修正后的代码示例 #

def cvals(sbfr):
    cdict = {}
    buttons = []
    # 用with语句自动管理文件，无需手动关闭
    with open('currencysheet.txt', 'r') as c:
        for line in c:
            stripped_line = line.strip()
            # 跳过空行
            if not stripped_line:
                continue
            currency, rate = stripped_line.split(',')
            cdict[currency.strip()] = float(rate.strip())
    
    # 遍历货币字典，生成对应按钮
    for currency_name in cdict:
        # 假设Button1的实例需要关联货币名称，这里根据实际需求调整
        btn_instance = Button1()
        # 假设Button1的name()方法需要传入货币名称，这里按需修改
        btn_instance.name(currency_name)
        # 先创建按钮对象，再调用pack()，避免丢失对象引用
        btn = Button(sbfr, text=currency_name, command=btn_instance.cmd)
        btn.pack(side=LEFT, fill=X, expand=True)
        buttons.append(btn)
    return cdict, buttons

关键说明 #

• 使用with语句读取文件，确保文件自动关闭，避免资源泄漏
• 简化字典构建逻辑，直接拆分每行得到货币名称和汇率，同时处理可能的空格
• 先创建Button对象再调用pack()，保留按钮对象的引用并添加到buttons列表，方便后续操作
• 去掉全局变量，改为通过函数参数传递父容器sbfr，返回生成的字典和按钮列表，代码更模块化

内容的提问来源于stack exchange，提问作者user19621201