关于tkinter OptionMenu调用Select_Sensor函数筛选DataFrame的咨询

嘿，我帮你梳理下代码里的问题，还有可以优化的地方：

1. 语法错误：OptionMenu的command参数写错了 #

你在创建OptionMenu时写的command= Select Sensor是个低级错误——函数名里的空格会被Python当成两个独立的标识符，直接触发语法报错。得改成下划线连接的正确函数名Select_Sensor。

2. 逻辑bug：Dev02分支的列选择完全错了 #

看你MyCurrentDev == "Dev02"的分支，居然复制了Dev01的列名，这会导致选中Dev02时，显示的还是Dev01的数据，明显是复制粘贴时的疏忽，得改成对应Dev02的列。

3. 冗余的全局变量赋值 #

函数开头那句mydf = df完全没必要，因为后面每个分支都会重新给mydf赋值，这行代码等于白写。

4. OptionMenu选项写得太啰嗦 #

你一个个手动写LoRa_Devices[0], LoRa_Devices[1]...到LoRa_Devices[11]，太麻烦了！直接用*LoRa_Devices就能把列表里的所有元素展开成选项，省事儿多了。

修正后的完整代码 #

def Select_Sensor(MyCurrentDev):
    global mydf
    # 用字典映射设备和对应的列，比一堆elif清爽多了
    device_col_map = {
        "Dev00": ["Date", "Time", "Température du toit #1 (Dev00) [°C]", "Humidité du toit #1 (Dev00) [%Hr]"],
        "Dev01": ["Date", "Time", "Température du toit #2 (Dev01) [°C]", "Humidité du toit #2 (Dev01) [%Hr]"],
        "Dev02": ["Date", "Time", "Température du toit #3 (Dev02) [°C]", "Humidité du toit #3 (Dev02) [%Hr]"]
        # 后面的Dev03到Dev11直接在这里加对应的列映射就行
    }
    # 取出对应设备的列，如果选中的设备不在映射里，就保留原df的所有列（你也可以改成默认显示空或者其他逻辑）
    target_cols = device_col_map.get(MyCurrentDev, df.columns)
    mydf = df.loc[:, target_cols]
    return mydf

# 简化OptionMenu的创建，不用手动列所有选项
drop = tk.OptionMenu(
    root,
    clicked,
    "Choose DevXX",
    *LoRa_Devices,
    command=Select_Sensor
)

额外小建议 #

• 用字典来管理设备和列的映射，比一堆elif分支维护起来方便太多了，后续加新设备只要在字典里加一行就行，不用改一堆条件判断。
• 如果你觉得全局变量mydf不太优雅，可以考虑把整个逻辑封装到一个类里，把df和mydf作为类的属性，但在tkinter的快速原型开发中，全局变量其实是很常用的方式。

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