Cambiar un objeto de clase en un ArrayList

[Damarsito]

Hola a todos.

Antes de nada, querría presentarme: mi nombre es Daniel y soy estudiante de ingeniería. En la asignatura de fundamentos de informática, estamos trabajando el JAVA y en la práctica que debo entregar me surje el siguiente problema:

Tengo una clase GestorDeCliente en el que he creado un ArrayList para añadir un listado de objetos de dos tipos de clases: Clientes y ClientesVIP (éste último, una subclase de la clase Clientes). Me piden que cree un método para seleccionar un objeto del tipo Cliente y lo cambie a ClienteVIP y viceversa. Cuando utilizo el ciclo "for" para seleccionar el cliente (por medio del nif), utilizo los métodos get para obtener todos los campos de definición del objeto, para luego añadir un nuevo ClienteVIP. El compilador me complia, pero cuando ejecuto, me sale un error de "concurrencia". Llevo varios días intentándolo, con lo cual agradecería cualquier orientación sobre como afrontar el problema, teniendo en cuenta que soy muy novato (tan sólo llevo dos meses estudiando JAVA) y que nunca antes he programado en ningún lenguaje.

Perdón por el ladrillo y espero vuestra inestimable ayuda...

Slds,
Daniel M.

[chuidiang]

¿Cual es el error exacto que te da y el trozo de código en donde te da?
Se bueno.

[Damarsito]

Gracias por la rápida respuesta chuidiang y perdona por los pocos datos que he reflejado. El método que he escrito es:

Código Javascript:

public void cambiarClienteAVIP(String nifCliente)
{
int indice = 0;
for(Cliente clienteCambiado : clientes) {
if (nifCliente.equals(clienteCambiado.getNif())) {
ClienteVIP nuevoCliente = new ClienteVIP(clienteCambiado.getNombre(), clienteCambiado.getApellidos(),
clienteCambiado.getNif(), clienteCambiado.getSexo(), clienteCambiado.getAñoDeNacimiento(),
clienteCambiado.getNumeroDeCliente(), clienteCambiado.getAñoDeAlta());
clientes.remove(clienteCambiado);
clientes.add(nuevoCliente);
indice++;
}
}
if (indice == 0) {
System.out.println("El NIF indicado no corresponde a ningún cliente");
}
 
}
 

Me salta el error:

java.util.ConcurrentModificationException
at java.util.AbstractList$Itr.checkForComodification( AbstractList.java:372)
at java.util.AbstractList$Itr.next(AbstractList.java: 343)
at GestorDeClientesAvanzado.cambiarClienteAVIP(Gestor DeClientesAvanzado.java:69)

Y me pone en amarillo (utilizo BlueJ) la línea de inicio del ciclo "for".

Aparecen ya los datos necesarios para analizar el problema?? (Es que estoy perdidísimo, jejeje...)

Gracias por adelantado!!

Daniel M.

[DiabloGuardian]

Hola!!
La documentacion de java dice:

Esta excepción puede ser lanzada por los métodos que se han detectado modificaciones simultáneas de un objeto cuando tal modificación no está permitido.

...

Por ejemplo, si un hilo modifica una colección directamente mientras se está interactuando sobre la colección con un iterador no-rápido, el iterador lanzarán esta excepción.

Eso quiere decir que:

Código java:

public void cambiarClienteAVIP(String nifCliente)
{
int indice = 0;
for(Cliente clienteCambiado : clientes) {
if (nifCliente.equals(clienteCambiado.getNif())) {
ClienteVIP nuevoCliente = new ClienteVIP(clienteCambiado.getNombre(), clienteCambiado.getApellidos(),
clienteCambiado.getNif(), clienteCambiado.getSexo(), clienteCambiado.getAñoDeNacimiento(),
clienteCambiado.getNumeroDeCliente(), clienteCambiado.getAñoDeAlta());
 
 
clientes.remove(clienteCambiado);  <<-- Aqui intentas eliminar un objeto que esta siendo usado
 
 
clientes.add(nuevoCliente);
indice++;
}
}
if (indice == 0) {
System.out.println("El NIF indicado no corresponde a ningún cliente");
}
 
}
 

Se me ocurren dos posibles soluciones:
1) puedes clonar tu objeto haciendo algo asi:

Código java:

Cliente clienteTemp = clienteCambiado.clone(); 
clientes.remove(clienteTemp);
 

2) Podrias hacer unos castings, si dices que ClientesVIP es una subclase de Clientes, intenta hacer algo asi:

Código java:

public void cambiarClienteAVIP(String nifCliente)
{
int indice = 0;
for(Cliente clienteCambiado : clientes) {
if (nifCliente.equals(clienteCambiado.getNif())) {
clienteCambiado=(ClienteVIP)clienteCambiado;
//aqui solo agregarias los valores de los atributos que cambian
indice++;
}
}
if (indice == 0) {
System.out.println("El NIF indicado no corresponde a ningún cliente");
}
 
}
 

Ojala te sirva esto, saludos

[Damarsito]

Gracias por la respuesta, DiabloGuardian.

Creo que el problema no es el intentar borrar el objeto seleccionado por el ciclo "for", ya que tengo otro método de "eliminarClientePorNif" que lo hace sin problemas. He probado ambas soluciones que me has aconsejado y no sirve ninguna, pues ni siquiera compilan.

Voy a cambiar de táctica, pues creo que el problema viene por agregar un nuevo objeto a un ArrayList por el que corre un ciclo "for". Voy a intentar cambiar el ciclo por un "while" a ver que tal...

Cualquier otra propuesta, será bienvenida!!

Gracias,
Daniel M.

[Fuzzylog]

Construye un segundo arraylist actualizandolo a medida que recorres el for del primero, se añade el elemento sin modificar si no hay que hacer nada, y se añade como VIP cuando haya que hacerlo. Luego asignas el nuevo arraylist a la referencia del primero y ya puedes seguir procesándolo.

[Damarsito]

Gracias Fuzzylog, pero llevo un rato intentándolo y no lo consigo. Me compila, pero sigue saltando el error de "ConcurrentModificationException" en la misma línea del primer ciclo "for"... No entiendo nada!!

Me puedes ampliar la info, para hacerlo correctamente, por favor??

Gracias!!
DM

[GreenEyed]

Si recorres una coleccion con un bucle for-each, no puedes borrar elementos de ella mientras lo haces. Para poder borrar un elemento, lo que tienes que hacer es declarar y recorrer explicitamente la coleccion con un Iterator y usar el metodo remove() que tiene.

Lo explican en la documentación del bucle for-each, al final:
http://java.sun.com/j2se/1.5.0/docs/...e/foreach.html

Unfortunately, you cannot use it everywhere. Consider, for example, the expurgate method. The program needs access to the iterator in order to remove the current element. The for-each loop hides the iterator, so you cannot call remove.

S!

[Damarsito]

Gracias GreenEyed, voy a buscar información del Iterator, que no se como funciona, y lo intento...

Slds,
DM