辞書のコールバックへのPythonリファレンス

Question

コールバック関数のセットを指定するクラスがあります（ここにはcb1とcb2と表示されています）。私はいくつかのイベントの後に電話したいこれらのマップを保持しています。

class Foo: 
    cb1 = None 
    cb2 = None 

    def test(self, input): 
     for (name, callback) in map: 
      if name == input: 
       if callback: callback() 
       ... 

    map = {'one':cb1, 'two':cb2} 

def mycallback(): 
    print "mycallback()" 

f = Foo() 
f.cb1 = mycallback # Register our callback 
f.test('one')  # Nothing happens 

問題を見つけることができますか？

何が起こるかは、クラスが初期化されるとき、（両方Noneある）値cb1のとcb2をマップにコピーされることです。したがって、ユーザーがコールバックを（cb1に割り当てることによって）コールした後でさえ、マップの値はまだNoneであり、何も呼び出されません。

Pythonで「参照」というようなことはないので、これをどのように修正すればよいですか？

Answer 1

あなたのクラスを明示的に登録しないようにしましょう。

import collections 

class Foo(object): 
    handlers = None 

    def __init__(self): 
     self.handlers = collections.defaultdict(set) 

    def register(self, event, callback): 
     self.handlers[event].add(callback) 

    def fire(self, event, **kwargs): 
     for handler in self.handlers.get(event, []): 
      handler(**kwargs) 

foo = Foo() 
foo.register('one', mycallback) 
foo.fire('one') 

Answer 2

登録機能を追加します。

def register(self, name, cb): self.map[name] = cb 

、代わりの：

f.cb1 = mycallback 

使用：Fooのクラスでは逆に

f.register('one', mycallback) 

Answer 3

、すべてがPythonで "参照によって" です。しかし、Noneへの参照を辞書にコピーしていて、元のスロットを変更してもその参照には何もしません。余計なレベルの間接参照を保持したい場合は、最も簡単な方法は文字列を格納することです。すべてのコールバックがこのクラスの属性である場合は、mapを取り除き、コールバック属性名のリストを格納します。 callback_names = ['cb1', 'cb2']を呼び出し、getattr(self, callback_name)()を使用してコールバックを呼び出します。マップが必要な場合は、map = {'one': 'cb1', 'two': 'cb2'}を実行できます。

あなたはプロパティで何か面白いこともできますが、それは不必要に複雑に思えます。

Answer 4

デリゲートディスクリプタとトリッキーな属性があります。

Answer 5

実際にコールバックを実行するために使用するものとは異なるコールバックをカスタマイズするために、別の変数を設定する必要があるのはなぜですか？同じ変数を使用すると、問題は消滅します。それはこのようになりますいくつかのシンタックスシュガーで

：

class CallbackMap(object): 
    pass 

class Foo(object): 
    callbacks = CallbackMap() 

    def test(self, input): 
     callback = getattr(Foo.callbacks, input) 
     if callback: callback() 

# setup defaults 
Foo.callbacks.one = None 
Foo.callbacks.two = some_default_callback 

# customize 
def mycallback(): 
    print "mycallback()" 

f = Foo() 
Foo.callbacks.one = mycallback # Register our callback 
f.test('one') # works