如何将装饰器与@contextmanager装饰器混合使用?

时间:2011-01-19 04:42:41

标签: python decorator contextmanager

以下是我正在使用的代码:

from contextlib import contextmanager
from functools import wraps
class with_report_status(object):

    def __init__(self, message):
        self.message = message

    def __call__(self, f):
        @wraps(f)
        def wrapper(_self, *a, **kw):
            try:
                return f(_self, *a, **kw)
            except:
                log.exception("Handling exception in reporting operation")
                if not (hasattr(_self, 'report_status') and _self.report_status):
                    _self.report_status = self.message
                raise

        return wrapper

class MyClass(object):

    @contextmanager
    @with_report_status('unable to create export workspace')
    def make_workspace(self):
        temp_dir = tempfile.mkdtemp()
        log.debug("Creating working directory in %s", temp_dir)
        self.workspace = temp_dir
        yield self.workspace
        log.debug("Cleaning up working directory in %s", temp_dir)
        shutil.rmtree(temp_dir)

    @with_report_status('working on step 1')
    def step_one(self):
        # do something that isn't a context manager

问题是,@with_report_status不会产生,正如@contextmanager所期望的那样。但是,我不能用其他方式包装它,因为@contextmanager返回一个生成器对象(我认为!)而不是值本身。

如何让@contextmanager与装饰师玩得很好?

2 个答案:

答案 0 :(得分:1)

这是一个奇怪的问题:@contextmanager返回一个上下文管理器,而不是一个生成器。但出于某种原因,您希望将上下文管理器视为一个函数?这不是你可以做的事情,它们没有任何共同之处。

我认为您想要的是MyClass.make_workspace,它是上下文管理器,并且在出现异常时也有report_status字段。为此你需要自己编写一个上下文管理器,用__exit__方法设置这个字段,@contextmanager在这里无法帮助你。

您可以继承contextlib.GeneratorContextManager以避免大部分工作。它没有记录,所以请使用源代码Luke。

答案 1 :(得分:0)

尝试在装饰者列表的底部移动@contextmanager。