Question

我正在编写这段代码作为面向对象编程的练习。

在这里，我试图将房屋列为房间列表，并将每个房间定义为设备列表（例如灯具）。

首先，我创建了所有对象，然后将两个房间附加到房屋，并将每个房间的不同设备附加到房间。非常基本。

问题在于设备似乎被附加到两个房间。那是为什么？

代码：

#! /usr/bin/python

class House:
        def __init__(self, rooms = list()):
                self.rooms = rooms
                print('house created')


class Room:
        def __init__(self, name = 'a room', devs = list()):
            self.name = name
            self.devs = devs
            print('room ' + self.name + ' created')


class Device:
        def __init__(self, name = 'a device'):
                self.name = name
                print('device ' + self.name + ' created')


def main():
        #1
        h = House()
        r1 = Room(name = 'R1')
        r2 = Room(name = 'R2')
        d1 = Device(name = 'lamp1')
        d2 = Device(name = 'lamp2')

        #2
        h.rooms.append(r1)
        h.rooms.append(r2)

        for room in  h.rooms:
                print room.name

        print h.rooms[0]
        print h.rooms[1]
        h.rooms[1].devs.append(d1)

        #3
        for room in h.rooms:
                print room.name
                for dev in room.devs:
                        print('room ' + room.name + ' > ' + dev.name)
                        print room
                        print dev


if __name__ == '__main__' : main()

输出。

house created
room R1 created
room R2 created
device lamp1 created
device lamp2 created
R1
R2
<__main__.Room instance at 0xb7d8a58c>
<__main__.Room instance at 0xb7d8a5ac>
R1
room R1 > lamp1
<__main__.Room instance at 0xb7d8a58c>
<__main__.Device instance at 0xb7d8a5cc>
R2
room R2 > lamp1
<__main__.Room instance at 0xb7d8a5ac>
<__main__.Device instance at 0xb7d8a5cc>

请注意，d1的相同实例位于两个房间，即r1和r2。

Answer 1

函数的默认参数值仅评估一次。这意味着House的所有实例将使用self.rooms的相同列表实例（如果在构造中未提供rooms参数）。同样，Room的所有实例都将共享self.devs的相同列表。

要解决此问题，请编写如下代码：

def __init__(self, rooms = None):
    if rooms is None:
        rooms = []
    self.rooms = rooms
    print('house created')

其他班级也是如此。

Answer 2

默认参数在声明方法时计算一次。然后在该方法的所有调用中使用该值。

stackoverflow exploring the reasons for this design以及如何best avoid these mutable default arguments还有其他问题。

Answer 3

def __init__(self, name = 'a room', devs = list()):
    self.name = name
    self.devs = devs
    print('room ' + self.name + ' created')

执行此操作时list()实际上始终是相同的列表。每次调用构造函数时，都不会得到 new 空列表，而是获得相同的空列表。为了解决这个问题，你需要制作副本。

list()更像是[]。

def __init__(self, name='a room', devs=[]):
    self.name = name
    self.devs = list(devs)
    print('room ' + self.name + ' created')

附加到列表实例的对象显示在该列表的不同实例中

3 个答案: