Lo que te ocurre no está relacionado con el método __init__()
ni con la programación orientada a objetos, sino que es consecuencia de haber asignado a uno de los parámetros la lista vacía []
como valor por defecto.
Esto causa efectos secundarios inesperados como el que has observado. Para que esto quede más claro voy a poner el ejemplo más simple posible que ni siquiera usa OOP sino funciones normales.
def ampliar_lista(dato, lista=[]):
lista.append(dato)
return lista
Si esto lo llamas así:
l = [1, 2]
l2 = ampliar_lista(3, l)
produce el resultado esperado, la lista l
resulta ampliada y pasa a valer [1,2,3]
y ese mismo valor es asignado a l2
, pues la función retorna la lista que acaba de ampliar. En ese punto l
y l2
son la misma lista (l is l2
devolvería True
).
Si lo llamas así:
l2 = ampliar_lista(5)
Entonces ya que no estamos pasando segundo parámetro, la lista
dentro de la función tomará el valor []
, luego meterá el 5 y se retornará la lista resultante, con lo que l2
valdrá [5]
. Hasta aqui todo bien.
Y ahora viene la "sorpresa". Si tras haber ejecutado la línea anterior hacemos:
l3 = ampliar_lista(10)
cabría esperar que en l3
hubiera [10]
, por las mismas razones que hemos explicado en el caso anterior. Sin embargo encontrarás l3
igual a [5, 10]
. Y lo que es peor ¡l2
también habrá cambiado de valor y será [5,10]
! Compruébalo.
¿Por qué ocurre esto? Porque resulta que l2
y l3
son en realidad la misma lista, como puedes verificar si miras l2 is l3
que te dirá True
.
Explicación
Al asignar el valor []
como valor por defecto para el parámetro lista
, Python crea una lista vacía antes incluso de que la función se ejecute. El parámetro lista
se referirá a esa lista vacía pre-creada. Digamos que la línea:
def ampliar_lista(dato, lista=[]):
es de algún modo equivalente a:
lista_pre_creada = []
def ampliar_lista(dato, lista=lista_pre_creada):
Así que cada vez que llames a ampliar_lista()
omitiendo el segundo parámetro, se usará la lista_pre_creada
, que siempre es la misma. Por esa razón esa lista va acumulando el 5 y el 10, porque es la misma en todas las llamadas que omitan el segundo parámetro.
Solución
Como regla general nunca pongas []
como valor por defecto para un parámetro pues te encontrarás este tipo de problemas.
Se resuelve dando como valor por defecto None
y detectando ese caso en el interior de la función:
def ampliar_lista(dato, lista=None):
if lista is None:
lista = []
lista.append(dato)
return lista
De este modo si omites el segundo parámetro, se crea una nueva lista vacía cada vez, dentro de la función.