使用类的 __new__ 方法作为工厂：__init__ 被调用两次

我在 python 中遇到了一个奇怪的错误，其中使用__new__将类的方法作为工厂会导致__init__实例化类的方法被调用两次。

这个想法最初是为了使用__new__母类的方法根据传递的参数返回其子类之一的特定实例，而无需在类外部声明工厂函数。

我知道使用工厂函数将是此处使用的最佳设计模式，但在项目的此时更改设计模式将代价高昂。因此我的问题是：有没有办法避免双重调用__init__并且只需要一次调用__init__在这样的模式中？

class Shape(object):
    def __new__(cls, desc):
        if cls is Shape:
            if desc == 'big':   return Rectangle(desc)
            if desc == 'small': return Triangle(desc)
        else:
            return super(Shape, cls).__new__(cls, desc)

    def __init__(self, desc):
        print "init called"
        self.desc = desc

class Triangle(Shape):
    @property
    def number_of_edges(self): return 3

class Rectangle(Shape):
    @property
    def number_of_edges(self): return 4

instance = Shape('small')
print instance.number_of_edges

>>> init called
>>> init called
>>> 3

非常感谢任何帮助。

当你构造一个对象时，Python 会调用它的__new__创建对象的方法然后调用__init__在返回的对象上。当您从内部创建对象时__new__通过致电Triangle()这将导致进一步致电__new__ and __init__.

你应该做的是：

class Shape(object):
    def __new__(cls, desc):
        if cls is Shape:
            if desc == 'big':   return super(Shape, cls).__new__(Rectangle)
            if desc == 'small': return super(Shape, cls).__new__(Triangle)
        else:
            return super(Shape, cls).__new__(cls, desc)

这将创建一个Rectangle or Triangle而不触发调用__init__进而__init__仅被调用一次。

编辑回答@Adrian 关于 super 如何工作的问题：

super(Shape,cls)搜索cls.__mro__找到Shape然后向下搜索序列的其余部分以查找属性。

Triangle.__mro__ is (Triangle, Shape, object) and Rectangle.__mro__ is (Rectangle, Shape, object) while Shape.__mro__只是(Shape, object)。 对于任何这些情况，当您致电时super(Shape, cls)它忽略 mro 序列中的所有内容，包括Shape所以剩下的就是单元素元组(object,)用于查找所需的属性。

如果您拥有钻石继承权，这会变得更加复杂：

class A(object): pass
class B(A): pass
class C(A): pass
class D(B,C): pass

现在 B 中的方法可能会使用super(B, cls)如果是 B 实例，则会搜索(A, object)但如果你有一个D实例相同的调用B会搜索(C, A, object)因为D.__mro__ is (B, C, A, object).

因此，在这种特殊情况下，您可以定义一个新的 mixin 类来修改形状的构造行为，并且您可以拥有从现有三角形和矩形继承但以不同方式构造的专用三角形和矩形。