Pythonオブジェクトを正しくクリーンアップするにはどうすればよいですか？

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

    # ...

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

__del__(self)上記はAttributeError例外で失敗します。が呼び出されたときにPythonが「グローバル変数」（このコンテキストのメンバーデータ？）の存在を保証しないことを理解してい__del__()ます。それが事実であり、これが例外の理由である場合、オブジェクトが適切に破壊されることを確認するにはどうすればよいですか？

withクリーンアップが必要なリソースの管理には、Pythonのステートメントを使用することをお勧めします。明示的なclose()ステートメントを使用する場合の問題はfinally、例外が発生したときにリソースリークを防ぐために、呼び出しをまったく忘れたり、ブロックに配置し忘れたりすることを心配する必要があることです。

withステートメントを使用するには、次のメソッドを使用してクラスを作成します。

  def __enter__(self)
  def __exit__(self, exc_type, exc_value, traceback)

上記の例では、

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

    def __enter__(self):
        return self

    # ...

    def __exit__(self, exc_type, exc_value, traceback):
        for file in self.files:
            os.unlink(file)

次に、誰かがあなたのクラスを使用したいと思ったとき、彼らは次のことをします：

with Package() as package_obj:
    # use package_obj

変数package_objは、Package型のインスタンスになります（これは、__enter__メソッドによって返される値です）。その__exit__メソッドは、例外が発生したかどうかに関係なく、自動的に呼び出されます。

このアプローチをさらに一歩進めることもできます。上記の例では、with句を使用せずに、コンストラクタを使用してパッケージをインスタンス化することができます。あなたはそれが起こりたくないのです。これを修正するには、__enter__および__exit__メソッドを定義するPackageResourceクラスを作成します。次に、Packageクラスが__enter__メソッド内で厳密に定義され、返されます。このようにして、呼び出し元はwithステートメントを使用せずにPackageクラスをインスタンス化できませんでした。

class PackageResource:
    def __enter__(self):
        class Package:
            ...
        self.package_obj = Package()
        return self.package_obj

    def __exit__(self, exc_type, exc_value, traceback):
        self.package_obj.cleanup()

これは次のように使用します。

with PackageResource() as package_obj:
    # use package_obj

標準的な方法は使用すること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...

クリントの答えの付録として、以下PackageResourceを使用して簡略化できますcontextlib.contextmanager。

@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):
        ...

そして単に使用しますwith Package(...) as package。

物事を短くするには、クリーンアップ関数に名前を付けてcloseを使用しますcontextlib.closing。この場合、変更されていないPackageクラスを使用するか、より単純なクラスに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.fileを別の場所に削除した可能性があります）。

ただし、他の人が指摘したように、の使用は避けてください__del__。これの主な理由は、のインスタンス__del__がガベージコレクションされないためです（参照カウントが0に達したときにのみ解放されます）。したがって、インスタンスが循環参照に関与している場合、アプリケーションが実行されている限り、インスタンスはメモリに存在します。（私はこれについてすべて間違っているかもしれませんが、gc docsをもう一度読む必要がありますが、このように機能することを確信しています）。

より良い代替案はweakref.finalizeを使用することです。ファイナライザオブジェクトおよびファイナライザと__del __（）メソッドの比較の例を参照してください。

__init__示されているよりも多くのコードがある場合、問題が発生する可能性があると思いますか？

__del____init__正しく実行されなかったり、例外がスローされたりした場合でも呼び出されます。

ソース

最小限の作業スケルトンは次のとおりです。

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()

重要：自分を返す

あなたが私のようなもので、return self（クリントミラーの正解の）部分を見落としている場合、あなたはこのナンセンスを凝視しているでしょう。

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

それが次の人に役立つことを願っています。

デストラクタをtry / exceptステートメントでラップするだけで、グローバルがすでに破棄されている場合でも例外はスローされません。

編集する

これを試して：

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変数を削除するのを防ぐためのものです（削除された場合、元のファイルリストには影響しません）。変数への参照がまだ多くても削除されない場合は、プロキシカプセル化を削除できます。

これを行う慣用的な方法は、close()メソッド（または同様の）を提供し、それを明示的に呼び出すことです。