the1fire

基于Django的个人博客系统

一个经典的传参错误

分类:Python   作者:edward   创建时间:2019年5月12日 17:02   pv:468   uv:467

python中的列表是可变的,在复制或把列表当做参数的时候要格外小心,下面是一个经典的传参错误。

class Company:
    def __init__(self, name, staffs=[]):
        self.name = name
        self.staffs = staffs

    def add(self, staff_name):
        self.staffs.append(staff_name)

    def remove(self, staff_name):
        self.staffs.remove(staff_name)

if __name__ == '__main__':
    com1 = Company('com1', ['jack1', 'jack2'])
    com1.add('jack3')
    com1.remove('jack1')

    print(com1.staffs)  # ['jack2', 'jack3']

     # 没有传入员工
    com2 = Company('com2')
    com2.add('jack')
    print(com2.staffs)  # ['jack']

    com3 = Company('com3')
    com3.add('jack5')
    print(com2.staffs)  # ['jack', 'jack5']

可以发现,实例对象com2的员工多了一个jack5。我们看一下Company的默认参数和实例对象com3的员工:

print(Company.__init__.__defaults__)  # (['jack', 'jack5'],)
print(com3.staffs)  # ['jack', 'jack5']

在看一下com2和com3的staffs是不是同一个对象:

print(com2.staffs is com3.staffs)  # True

造成这种现象是由两个原因引起的:

  1. 在初始化Company的时候我们传入了一个列表(staffs=[]),而列表是可变对象。
  2. com2和com3都没有传递list进来,所以他们都会使用默认的list。 com2和com3都使用了默认的list,而list本身又是一个可变对象,所以com2和com3就共用了一个对象。 com1传递了一个list进来,所以就不会使用__defaults__的值。

结论:初始化参数的时候尽量不要传递list。非得传递一定要意识到它是可以被修改的。