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

    # ...

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

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


当前回答

作为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

其他回答

只要用try/except语句包装你的析构函数,如果你的全局变量已经被处理了,它就不会抛出异常。

Edit

试试这个:

from weakref import proxy

class MyList(list): pass

class Package:
    def __init__(self):
        self.__del__.im_func.files = MyList([1,2,3,4])
        self.files = proxy(self.__del__.im_func.files)

    def __del__(self):
        print self.__del__.im_func.files

它会将文件列表填充到del函数中,该函数保证在调用时存在。weakref代理是为了防止Python或您自己删除self。Files变量(如果它被删除,那么它不会影响原始文件列表)。如果即使有更多对变量的引用,也没有删除该变量,那么可以删除代理封装。

标准的方法是使用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...

我认为问题可能是在__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

这样做的惯用方法似乎是提供一个close()方法(或类似方法),并显式地调用它。