如何优化回调python代码以获得更好的可读性和pythonic实践？

Question

我有一个需要从构造函数调用的私有函数。 这个私有函数的工作是订阅外部程序通知的事件。 所有订阅函数都需要通用参数，如 freq、initialPublication、initialSuspended、回调和异步。 我希望所有订阅参数都相同。 但是，回调应该有所不同。 这是我创建的通用参数字典。

params = {'freq': 0, 'initialPublication': True,

                      'initiallySuspended': False, 'callback': None,

                      'asynchronous':   True,

                      }

                  

我添加了一个评论，我觉得很重复，即

params[callback] = self.callback_test_sub1
 subscribe_attrib = 'subscribe_testSub1'
 if not self._subscribe(params, subscribe_attrib):
            return False
 ......
 

我可以做些什么来改善这一点，或者它是否已经处于最佳状态？

class Test:
    
    def callback_test_sub1(self, sub1_data):
        print('got sub1 data')
        
    def callback_test_sub2(self, sub2_data):
        print('got sub2 data')
        
    def notify_test_sub3(self, sub3_data):
        print('got sub3 data')
        
    def _subscribe(self, params, subscribe_attribute):
        try:
            func = getattr(self.external_obj, subscribe_attribute)
            err_status = func(**params)
        except AttributeError:
            print('no such attribute')

        except Exception as e:
            print(e)
    
    def __init__(self):
        params = {'freq': 0, 'initialPublication': True,
        'initiallySuspended': False, 'callback': None, 'asynchronous': True}
        
        ###  while doing these 3 callbacks, I feel repetitive
        params[callback] = self.callback_test_sub1
        subscribe_attrib = 'subscribe_testSub1'
        if not self._subscribe(params, subscribe_attrib):
            return False
        params[callback] = self.callback_test_sub2
        subscribe_attrib = 'subscribe_testSub2'
        if not self._subscribe(params, subscribe_attrib):
            return False
        params[callback] = self.callback_test_sub2
        subscribe_attrib = 'subscribe_testSub3'
        if not self._subscribe(params, notify_test_sub3):
            return False
        ###  while doing these 3 callbacks, I feel repetitive end
        return True

Answer 1

如果你不想重复这些小块，你可以制作一个列表来迭代：

callbacks = (('subscribe_testSub1', self.callback_test_sub1), 
             ('subscribe_testSub2', self.callback_test_sub2), 
             ('subscribe_testSub3', self.callback_test_sub3))
for attribute, callback in callbacks:
    params['callback'] = callback
    if not self._subscribe(params, attribute)
        return False

鉴于您的代码示例已经显示了几个拼写错误/错误，减少这种重复实际上可能很有用。 如果回调和属性名称具有特定模式，您还可以计算它们并使用getattr检索，但代码可能会变得更加晦涩，无法静态验证。