使用Python字典选择函数执行时BACKUP重复运行，求排查问题

哦，我一眼就瞅出问题所在啦！你遇到的BACKUP执行两次的根源，是在定义options字典时犯了一个常见的小错误——你直接调用了execute函数，而不是把可执行的函数引用存到字典里。

具体来说，这两行代码：

options = {
    "BACKUP": execute(self.jobtype, self.src, self.dst),
    "RESTORE": execute(self.jobtype, self.dst, self.src),
}

当Python创建这个字典的时候，会立刻计算execute(...)的结果，也就是直接运行一次BACKUP对应的逻辑。之后你调用options[jobtype]()，又会触发一次执行，这就导致了BACKUP被运行两次。而且更糟的是，你的execute函数没有返回值，所以字典里存的其实是None，理论上调用options[jobtype]()还会报错，但可能你的代码里刚好没触发这个问题。

修复方案 #

我们需要把可调用的对象（而不是函数执行的结果）存到字典里，这里有两种常用的方式：

方式1：用lambda表达式延迟执行 #

lambda可以帮你把参数和函数绑定起来，只有当你主动调用它的时候，才会触发execute的执行：

def execute(jobtype, src, dst):
    if jobtype == "FULL":
        _o_src = fs.Index(src)
        fs.MakeFolders(_o_src.GetFolders(), dst)
        fs.MakeFiles(src, dst, _o_src.GetFiles())
    if jobtype == "INCREMENTAL":
        print("DO INCREMENTAL BACKUP " + src + " TO " + dst)
        # 这里写增量备份的逻辑

# 用lambda包裹，避免立刻执行
options = {
    "BACKUP": lambda: execute(self.jobtype, self.src, self.dst),
    "RESTORE": lambda: execute(self.jobtype, self.dst, self.src),
}
# 这时候调用才会真正执行对应的逻辑
options[jobtype]()

方式2：拆分函数，直接存储函数引用 #

如果你的逻辑更复杂，建议把不同的操作拆成单独的函数，这样代码可读性更高：

# 把不同备份类型的逻辑拆成独立函数
def full_backup(src, dst):
    _o_src = fs.Index(src)
    fs.MakeFolders(_o_src.GetFolders(), dst)
    fs.MakeFiles(src, dst, _o_src.GetFiles())

def incremental_backup(src, dst):
    print("DO INCREMENTAL BACKUP " + src + " TO " + dst)
    # 增量备份逻辑

def execute(jobtype, src, dst):
    # 内部用字典映射不同的任务类型
    job_handlers = {
        "FULL": full_backup,
        "INCREMENTAL": incremental_backup
    }
    # 调用对应的函数
    job_handlers[jobtype](src, dst)

# 上层的options同样用lambda延迟执行
options = {
    "BACKUP": lambda: execute(self.jobtype, self.src, self.dst),
    "RESTORE": lambda: execute(self.jobtype, self.dst, self.src),
}
options[jobtype]()

这样修改后，BACKUP就只会在你调用options[jobtype]()的时候执行一次啦！

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