Python如何将if/else命令判断逻辑改写为for循环实现

你遇到所有函数一次性执行的问题，核心是两个写法错误：

1. 构建命令字典时，你在每个函数名后面加了()，Python会在字典初始化阶段直接执行所有函数，把函数的返回值存为字典的值，而不是存储函数本身等待后续调用。
2. 遍历代码里你把循环变量命名为cmds，和外层的命令字典重名，第一次循环后原字典对象就会被覆盖，逻辑完全混乱。另外这个场景本身不需要用for循环做匹配，字典的键值查询是O(1)复杂度，比遍历所有项高效得多。

第一步：构建正确的命令映射字典 #

字典的值存储函数引用即可，不要加()调用函数：

cmd_map = {
    "help": help,
    "calculator": calculatorb,
    "showdate": showdate,
    "quit": quit
}

第二步：匹配并执行命令（推荐写法，无需for循环） #

直接判断用户输入是否存在于字典的键集合中，存在就取出对应函数调用即可，完全替代原来的if/else分支：

# upp是你之前处理好的用户输入字符串
if upp in cmd_map:
    # 匹配成功时才加()执行对应函数
    cmd_map[upp]()
else:
    print(cmderror)

如果你必须要求用for循环实现（不推荐，效率更低），注意不要重名变量，匹配到后及时break跳出循环：

is_cmd_matched = False
# 注意循环变量不要和字典同名
for cmd_name, cmd_handler in cmd_map.items():
    if upp == cmd_name:
        cmd_handler()
        is_cmd_matched = True
        break
if not is_cmd_matched:
    print(cmderror)

注意：Python中函数是一等公民，可以直接作为值存储在变量、字典中，只有在名称后加()时才会触发函数执行。

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