我究竟做错了什么? Python对象实例化保留以前实例化的数据?

时间:2010-07-01 21:29:28

标签: python instantiation

有人能指出我做错了什么或我的理解错了吗?

对我来说,下面的实例化两个对象的代码似乎应该为每个实例化提供单独的数据。

class Node:
    def __init__(self, data = []):
        self.data = data

def main():
    a = Node()
    a.data.append('a-data') #only append data to the a instance

    b = Node() #shouldn't this be empty?

    #a data is as expected
    print('number of items in a:', len(a.data))
    for item in a.data:
        print(item)

    #b data includes the data from a
    print('number of items in b:', len(b.data))
    for item in b.data:
        print(item)

if __name__ == '__main__':
    main()

但是,第二个对象是使用第一个数据创建的:

>>> 
number of items in a: 1
a-data
number of items in b: 1
a-data

3 个答案:

答案 0 :(得分:8)

您不能将可变对象用作默认值。所有对象将共享相同的可变对象。

这样做。

class Node:
    def __init__(self, data = None):
        self.data = data if data is not None else []

创建类定义时,它会创建[]列表对象。每次创建类的实例时,它都使用相同的列表对象作为默认值。

答案 1 :(得分:3)

问题出在这一行:

def __init__(self, data = []):

当你编写data = []来设置一个空列表作为该参数的默认值时,Python只创建一个列表,并且每次调用该方法时都使用相同的列表而没有明确的data论点。在您的情况下,这会在创建ab时发生,因为您没有为任何一个构造函数提供明确的列表,因此ab都是使用与data列表相同的对象。您对其中的任何更改都将反映在另一个中。

要解决这个问题,我建议用

替换构造函数的第一行
def __init__(self, data=None):
    if data is None:
        data = []

答案 2 :(得分:2)

为函数或方法提供默认值时,通常需要提供不可变对象。如果您提供一个空列表或一个空字典,您将最终调用该函数或共享该对象的方法。

一个好的解决方法是:

def __init__(self, data = None):
    if data == None:
        self.data = []
    else:
        self.data = data