Python复合模式异常处理& 皮林特

Question

我正在这种方式实现复合模式：

1）“抽象”组件是：

class Component(object):
    """Basic Component Abstraction"""
    def __init__(self, *args, **kw):
        raise NotImplementedError("must be subclassed")

    def status(self):
        """Base Abstract method"""
        raise NotImplementedError("must be implemented")

2）叶子：

class Leaf(Component):
    """Basic atomic component
    """
    def __init__(self, *args, **kw):
        self.dict = {}

    def status(self):
        """Retrieves properties
        """
        return self.dict

问题是 pylint 当然会生成此警告：

Leaf.__init__: __init__ method from base class 'Component' is not called

但在我的叶子中，我无法调用：

def __init__(self, *args, **kw):
    Component.__init__(self, *args, **kw)
    self.dict = {}

不引发异常。

我必须忽略 pylint 警告还是存在一些错误的编码？

Answer 1

抽象初始化器是一个坏主意。 您的代码可能会演变，以便您想要在根组件中进行一些初始化。 即使您不知道为什么需要实现初始化程序。 对于某些子类，空的初始值设定项是可以接受的选择。

如果您不需要 Component 类的任何实例，请在初始化程序中检查：

class Component(object):
    def __init__(self, **kwargs):
        assert type(self) != Component, "Component must not be instantiated directly"

class Leaf(Component):
    def __init__(self, some, args, **kwargs):
        # regular initialization
        Component.__init__(self, **kwargs)

Answer 2

补充 Markus 想法的另一个建议：

如果您确实必须这样做，我建议您使用 __new __ 并检查给定的对象类型。 当它是“Component”时，您可以触发异常：

class Component(object):
"""Basic Component Abstraction"""

def __new__(objType, *args, **kwargs):
    if objType == Component:
       raise NotImplementedError("must be subclassed")
    return object.__new__(type, *args, **kwargs)

创建子类时，objType 将是！= Component，一切都会好起来的！

Answer 3

将类 Component 重命名为 AbstractComponent 应该会有所帮助。 如果子类不调用该方法，则不要在基类中提供 __init__ 方法。

Answer 4

您想要保证基类 Component 没有被实例化。 这是其他编程语言（如 C++）中常见的高贵姿态（您可以将构造函数设置为私有，以防止直接使用）。

但Python不支持它。 Python 不支持所有编程概念，而且更加“动态”。 因此，初始化是以“Pythonic”方式完成的，因此不支持您的概念。

与其他语言相比，Python 更多地基于信任——因此，例如，不支持静态变量，并且私有变量也仅以有限的方式支持。

您可以做什么（当您不信任模块的用户时）——您可以通过将基类命名为“_Component”来隐藏它——使其成为内部秘密。 但这当然会带来其他麻烦。

Answer 5

这样的编码不错，但是根本不需要组件的 __init__ 。 如果你想要它，你可以忽略 pylint，但更好的主意是简单地从组件中删除 __init__ 。

拥抱动态主义！