class Package:
    def __init__(self):
        self.files = []

    # ...

    def __del__(self):
        for file in self.files:
            os.unlink(file)

上面的__del__(self)失败,出现AttributeError异常。我理解当调用__del__()时,Python不保证存在“全局变量”(在此上下文中是成员数据?)如果是这种情况,这是异常的原因,我如何确保对象销毁正确?


当前回答

我认为问题可能是在__init__如果有更多的代码比显示?

即使__init__没有正确执行或抛出异常,__del__也将被调用。

其他回答

我认为问题可能是在__init__如果有更多的代码比显示?

即使__init__没有正确执行或抛出异常,__del__也将被调用。

作为Clint回答的附录,你可以使用contextlib.contextmanager简化PackageResource:

@contextlib.contextmanager
def packageResource():
    class Package:
        ...
    package = Package()
    yield package
    package.cleanup()

或者,尽管可能不像Pythonic那样,你可以重写Package.__new__:

class Package(object):
    def __new__(cls, *args, **kwargs):
        @contextlib.contextmanager
        def packageResource():
            # adapt arguments if superclass takes some!
            package = super(Package, cls).__new__(cls)
            package.__init__(*args, **kwargs)
            yield package
            package.cleanup()

    def __init__(self, *args, **kwargs):
        ...

简单地使用Package(…)作为Package。

为了更简洁,请将清理函数命名为close并使用contextlib。关闭,在这种情况下,你可以使用未修改的Package类via with contextlib.closing(Package(…))或覆盖它的__new__为更简单

class Package(object):
    def __new__(cls, *args, **kwargs):
        package = super(Package, cls).__new__(cls)
        package.__init__(*args, **kwargs)
        return contextlib.closing(package)

这个构造函数是继承的,所以你可以简单地继承,例如。

class SubPackage(Package):
    def close(self):
        pass

我认为在__del__被调用之前,实例成员不可能被移除。我的猜测是,您的特定AttributeError的原因是在其他地方(可能您错误地删除了self。文件在其他地方)。

然而,正如其他人指出的那样,你应该避免使用__del__。主要原因是带有__del__的实例不会被垃圾收集(只有当它们的refcount达到0时才会被释放)。因此,如果你的实例涉及循环引用,那么只要应用程序运行,它们就会一直存在于内存中。(我可能对所有这些都错了,我必须再次阅读gc文档,但我相当肯定它是这样工作的)。

标准的方法是使用atexit.register:

# package.py
import atexit
import os

class Package:
    def __init__(self):
        self.files = []
        atexit.register(self.cleanup)

    def cleanup(self):
        print("Running cleanup...")
        for file in self.files:
            print("Unlinking file: {}".format(file))
            # os.unlink(file)

但是您应该记住,这将保存所有已创建的Package实例,直到Python终止。

演示使用上述代码保存为package.py:

$ python
>>> from package import *
>>> p = Package()
>>> q = Package()
>>> q.files = ['a', 'b', 'c']
>>> quit()
Running cleanup...
Unlinking file: a
Unlinking file: b
Unlinking file: c
Running cleanup...

下面是一个最小的工作框架:

class SkeletonFixture:

    def __init__(self):
        pass

    def __enter__(self):
        return self

    def __exit__(self, exc_type, exc_value, traceback):
        pass

    def method(self):
        pass


with SkeletonFixture() as fixture:
    fixture.method()

重要:返回自我


如果你像我一样,忽略了(克林特·米勒正确答案中的)“回归自我”部分,你将会看到下面这段废话:

Traceback (most recent call last):
  File "tests/simplestpossible.py", line 17, in <module>                                                                                                                                                          
    fixture.method()                                                                                                                                                                                              
AttributeError: 'NoneType' object has no attribute 'method'

希望它能帮助到下一个人。