为什么这个函数装饰器失败了？

Question

这是我的代码：

name    = "Arthur"
message = "Hello!"

def decorate(func_to_decorate):

    def wrap(*args, **kwargs):

        print           ("........................")
        func_to_decorate(*args, **kwargs)
        print           ("........................")
        return wrap


@decorate
def send_message(your_name="unassigned", your_message="blank"):

    print(your_name)
    print(your_message)

send_message(name, message)

我的错误在第 20 行：

send_message(name, message)
TypeError: 'NoneType' object is not callable

我的理解是，包装器正在用紧跟在装饰器后面的函数“替换”自己。当我没有将参数传递给被装饰的函数时，这似乎可行，但在装饰器存在的情况下则不行。

Answer 1

你的装饰器有两个问题。

首先，有一个缩进错误：

def decorate(func_to_decorate):

    def wrap(*args, **kwargs):

        print           ("........................")
        func_to_decorate(*args, **kwargs)
        print           ("........................")
        return wrap

return wrap 是 wrap 函数体的一部分，而不是 decorate 函数体的一部分。所以，decorate 没有return 语句，这意味着它返回None。因此，您会看到错误：装饰器实际上是用它返回的包装器“替换”包装的函数——但该包装器是 None，因此您最终尝试将 None 作为函数调用。

同时，您似乎明白wrap 应该返回一些东西，但绝对不应该是本身。通常，您要返回的是包装函数的结果（或该结果的某些后处理版本）。在您的测试中，您只是试图包装一个仅用于副作用的函数，更不用说由于您的第一个问题您永远不会调用包装器，所以您还不会注意到这个问题，但是你还想修复它。

所以：

def decorate(func_to_decorate):

    def wrap(*args, **kwargs):

        print           ("........................")
        retval = func_to_decorate(*args, **kwargs)
        print           ("........................")
        return retval

    return wrap