[SOLUCIONADO] Objetos repetidos en Arraylist

[Hanzo_lx]

Hola existe algun problema al añadir elementos a un ArrayList?

Me explico

Tengo este Arraylist

List<Pregunta> preguntasleidas= new ArrayList<Pregunta>();

Desde un metodo voy leyendo preguntas desde un xml
Mientras las voy leyendo compruebo que los datos esta bien por ejemplo

Pregunta 1 respuesta b
Pregunta 2 respuesta a

A cada pregunta le hago un preguntasleidas.add(preguntaleida)

Una vez finalizada la lectura listo los objetos del arraylist y me doy cuenta que estan repetidos de esta forma

Pregunta 2 respuesta a
Pregunta 2 respuesta a

Estoy haciendo algo mal?

[jomaruro]

Hola:

Sin ver el código es difícil saber donde está el error.

Saludos.

[chuidiang]

La verdad es que tampoco está muy clara la duda, pero entre el título "Objetos repetidos en Arraylist" y "existe algun problema al añadir elementos a un ArrayList", te comento un error típico de los que empiezan.

Si haces algo como esto

Código Java:

unElemento = new UnElement();
bucle {
         unElemento.setValor(valor);
         lista.add(unElement);
}
 

como solo has hecho un new, lo que has hecho es añadir ese unico elemento muchas veces por lo que tendrás una lista en que todas las posiciones son el mismo elemento. Con setVAlor() has ido cambiando el contenido del único elemento que tenías, por lo que al final, en la lista todas las posiciones tienen el mismo elemento con el último valor que hayas puesto.

Debes hacer un new por cada elemento nuevo

Código Java:

bucle {
       // el new dentro del bucle.
       unElemento = new UnElemento();
       unElemento.setValor(unValor);
       lista.add(unElemento);
}
 

Se bueno.

[Hanzo_lx]

Iniciado por chuidiang

La verdad es que tampoco está muy clara la duda, pero entre el título "Objetos repetidos en Arraylist" y "existe algun problema al añadir elementos a un ArrayList", te comento un error típico de los que empiezan.

Si haces algo como esto

Código Java:

unElemento = new UnElement();
bucle {
         unElemento.setValor(valor);
         lista.add(unElement);
}
 

como solo has hecho un new, lo que has hecho es añadir ese unico elemento muchas veces por lo que tendrás una lista en que todas las posiciones son el mismo elemento. Con setVAlor() has ido cambiando el contenido del único elemento que tenías, por lo que al final, en la lista todas las posiciones tienen el mismo elemento con el último valor que hayas puesto.

Debes hacer un new por cada elemento nuevo

Código Java:

bucle {
       // el new dentro del bucle.
       unElemento = new UnElemento();
       unElemento.setValor(unValor);
       lista.add(unElemento);
}
 

Se bueno.

Ese era el problema, habia olvidad reinicializar el objeto cadad vez

Muchas gracias