使用装饰器返回添加属性的内部函数

Question

我有很多函数，这些函数在尝试保存对象时会作为测试运行，所以我想出了一种更方便的方法来把这些函数注册为对象的约束（测试）。

我对在我的解决方案中使用装饰器感到困惑。我原本打算做的是：

• 创建一个装饰器，用来给对象的方法添加属性
• 给我想标记为约束的方法加上装饰器。
• 遍历我对象中的所有方法，执行所有带有标记的方法

使用装饰器没有效果，但我自己添加标记却可以。

这是我的代码：

from functools import wrap

def constraint(func):
    @wraps(func)
    def inner(*args, **kwargs):
        func._is_constraint = True  # Here after returning the function I add a flag
        return func(*args, **kwargs)
    return inner

class Record(object):
    param1 = 100  # some defaults
    param2 = 20

    @constraint  # This dont works
    def check_test1(self):
        if param1 < 0:
            return 'No value less than zero'

    # This works fine
    def check_test2(self):
        if param2 < 0:
            return 'No value less than zero'
    check_test2._is_constraint = True

    def a_random_method(self):
        print 'some random thing'

所以我现在尝试的是：

>>> rec = Record()
>>> rec.param1 = -100
>>> rec.param2 = -100
>>> for prop in dir(rec):
...     if hasattr(getattr(rec, prop), '_is_constraint'):
...         constraint = getattr(rec, prop)
...             print prop, constraint()
...    
'check_param2: No value less than zero'
>>>

注意到 check_param1 方法没有被执行。

那么，我该如何使用装饰器让它工作呢？这是可能的吗？

我在这个特定的案例中尝试使用 https://gist.github.com/mariocesar/4684561

Answer 1

首先，我看到的第一件事是，你应该在装饰器中把参数添加到函数对象上，而不是闭包里。闭包是在从装饰过的范围内调用函数时被调用的，所以在闭包里添加参数其实是事后再说：

def constraint(func):
    func._is_constraint = True
    @wraps(func)
    def inner(*args, **kwargs):
        return func(*args, **kwargs)
    return inner

另外，如果你只是直接在闭包里调用函数，而没有增加任何功能，仅仅是添加标签，那么这个闭包就完全没必要了：

 def constraint(func):
    func._is_constraint = True
    return func

在这种情况下，你可能会更适合使用其他的方式来实现：

class ConstraintsType(type):

    def __new__(cls, name, bases, attrs):

        attrs['constraint_names'] = []
        for attr in attrs:
            if attr.startswith('constraint_'):
                attrs['constraint_names'].append(attr)
        return super(ConstraintsType, cls).__new__(cls, name, bases, attrs)

class Constraints(object):

    __metaclass__ = ConstraintsType

    @property
    def constraints(self):

        for name in self.constraint_names:
            yield getattr(self, name)

class Record(Constraints):

    def __init__(self, params=(100, 20)):

        self.params = params

    def constraint_greater_than_0(self, value):

        return value > 0

   def run(self):

       for index, value in enumerate(self.params):
           for func in self.constraints:
               if not func(value):
                   print 'param %d (%s) did not satisfy constraint %s' % (index, value, func.__name__)

Record().run()
for value_set in ((-100, -100), (0, 0), (-1,1), (1,-1)):
    Record(value_set).run()