Почему изменяется узел, который я создаю и засовываю в массив?

Question

[limaximy]

Создаю узел, который имеет пустой массив. Потом создаю еще один узел и засовываю его в этот массив (все через классы). И первый узел в массиве имеет этот узел, но почему то второй узел имеет в массиве ссылку на самого себя (хотя должен быть пустой массив)

class Node:
	def __init__(self, p_type, p_text, p_mas_connections = []):
		self.type = p_type
		self.text = p_text
		self.mas_connections = p_mas_connections

	def print_connections(self):
		print(self.type)
		print(self.text)
		print(self.mas_connections)

	def change_connections(self, p_position, p_node): 
		self.mas_connections[p_position] = p_node

	def append_connections(self, p_node):
		print(f'self.mas_connections {self.mas_connections}')
		self.mas_connections.append(p_node)
		print(f'self.mas_connections {self.mas_connections}')

	def return_selected_node_from_mas_connections(self, p_position):
		return self.mas_connections[p_position]

root = Node('key_word', 'program', [Node("identifier", None)])

current = root.return_selected_node_from_mas_connections(0)
current.print_connections()

s = Node('tochka_s_zapyatoy', ';')
s.print_connections()

current.append_connections(s)
s.print_connections()

Вывод
identifier
None
[]
tochka_s_zapyatoy
;
[]
identifier
None
[<__main__.Node object at 0x000001AEDE08C3E0>]
tochka_s_zapyatoy
;
[<__main__.Node object at 0x000001AEDE08C3E0>] # Вот здесь должен быть просто []

В чем я запутался?

Answer 1

[Everything_is_bad]

Стандартная ошибка новичка в

def __init__(self, p_type, p_text, p_mas_connections = []):

правильно p_mas_connections=None, а далее в коде например так

self.mas_connections = [] if p_mas_connections is None else p_mas_connections

гуглить "python mutable default argument"

Comments

[limaximy]

Да, странная работа интерпретатора, прочитал
Но за новичка обидно, поэтому не буду ставить решение ;)

[Everything_is_bad]

limaximy,

Да, странная работа интерпретатора

нет, не странная, это документированная работа и именно новички про это не знают.

[limaximy]

Everything_is_bad, От вас требуется лишь ответ на вопрос, а не установка шаблонов. Вместо полуответа могли бы и обьяснить, каким образом "задокументированная работа", которая похожа на баг, отличается от бага и в чем ее польза.

[Vindicar]

limaximy, польза в простоте, оптимизации и в необходимости для других механизмов.
Значения по умолчанию для аргументов функции вычисляются один раз - в момент создания объекта функции. Не забываем, что в питоне функции - это такие же объекты, как числа, строки, списки и т.д.
Объявление функции - это такой же оператор, как if или while.
А дальше классический вопрос про лямбду. Что выведет этот код?

funcs = []
for i in range(5):
  funcs.append(lambda: print(i))
for f in funcs:
  f()

Правильный ответ: 5 раз выведет 4, потому что лямбды ссылаются на саму переменную i, а не на её текущее значение. Чинится это так:

funcs = []
for i in range(5):
  funcs.append(lambda x=i: print(x))
for f in funcs:
  f()

Это сработает как надо, потому что значение по умолчанию аргумента x будет запомнено в момент создания лямбды, и не будет меняться. Если это поведение изменить, то не будет вменяемого способа зафиксировать текущее значение переменной при создании лямбды. В принципе, всё вышеописанное применимо и не к лямбдам, а к обычным функциям.

Если фиксировать значения вместо переменных, это мало того, что сломает глобальные переменные, но и создаст проблемы из-за того, что мутабельные и немутабельные типы будут вести себя по-разному - т.е. как раз то, от чего ты хочешь уйти. Но в ещё большем масштабе.