settattr 让父类在子类中使用

Question

我有一个图书馆，有一个家长和十几个孩子：

# mylib1.py:
#
class Foo(object):
  def __init__(self, a):
    self.a = a


class FooChild(Foo):
  def __init__(self, a, b):
    super(FooChild, self).__init__(a)
    self.b = b

# more children here...

现在我想用一个简单的（但有点特别，用于另一种方法）方法来扩展该库。所以我想改变父类并使用它的孩子。

# mylib2.py:
#
import mylib1

def fooMethod(self):
  print 'a={}, b={}'.format(self.a, self.b)

setattr(mylib1.Foo, 'fooMethod', fooMethod)

现在我可以这样使用它了：

# way ONE:
import mylib2

fc = mylib2.mylib1.FooChild(3, 4)
fc.fooMethod()

或者像这样：

# way TWO:
# order doesn't matter here:
import mylib1
import mylib2

fc = mylib1.FooChild(3, 4)

fc.fooMethod()

所以，我的问题是：

1. 这是好事吗？
2. 这应该如何以更好的方式完成？

Answer 1

一种常见的方法是使用mixin

如果需要，您可以动态添加How do I dynamically add mixins as base classes without getting MRO errors?。

Answer 2

编程中有一条通用规则，即您应该避免依赖全局状态。换句话说，这意味着您的全局变量应该尽可能保持不变。类（大部分）是全局的。

您的方法称为猴子修补。如果你没有一个非常好的理由来解释它，你应该避免它。这是因为猴子补丁违反了上述规则。

假设您有两个独立的模块，并且它们都使用这种方法。其中之一将 Foo.fooMethod 设置为某个方法。另一个 - 另一个。然后你以某种方式在这些模块之间切换控制。结果是，很难确定 fooMethod 在哪里使用。这意味着很难调试问题。

有些人（例如 Brandon Craig-Rhodes）认为即使在测试中修补也很糟糕。

我的建议是使用您在实例化 Foo() 类（及其子类）的实例时设置的一些属性，该属性将控制 fooMethod 的行为。那么这个方法的行为将取决于你如何实例化对象，而不是全局状态。