Estoy usando este código para recorrer una lista de trabajadores que puede contener objetos de tres clases: Mecánico, Comercial y Jefe.
Para otra lista que tengo de solo clientes uso for:each, pero solo contiene objetos de una misma clase Clientes. Uso el iterator porque hasta donde yo sé es la única forma de recorrer la lista entera si contiene objetos de diferentes clases.
Iterator <Trabajador> miIterador = trabajadores.iterator();
System.out.println("\nIntroduzca el DNI de la persona que desea consultar:");
entrada = new Scanner(System.in);
dni = entrada.nextLine();
while(miIterador.hasNext()) {
if(miIterador.next().getDni().compareToIgnoreCase(dni)==0) {
encontrado=1;
System.out.println(miIterador.next().toString());
}
}
if(encontrado==0) {
System.out.println("\nCliente no encontrado.");
}
El problema es que me salta el siguiente error:
Exception in thread "main" java.util.NoSuchElementException at java.util.LinkedHashMap$LinkedHashIterator.nextNode(Unknown Source) at java.util.LinkedHashMap$LinkedKeyIterator.next(Unknown Source) at taller.Main.main(Main.java:103)
Pero esto solo ocurre cuando se encuentra en el último objeto, por ejemplo:
Si yo introduzco manualmente a tres trabajadores con dni 1, 2 y 3 respectivamente, puedo buscar e imprimir el 1 y 2 sin problema, pero el 3 me da ese error. He pensado en crear un objeto "invisible" automaticamente e ir moviéndolo cada vez que se da de alta un nuevo trabajador para que los creados por el usuario nunca sean los últimos y nunca de error al consultarlos, pero creo que probablamente estaré teniendo algún error de códugo que desconozco y no es necesario liarlo tanto.
Gracias de antemano!!
next()
te retorna el siguiente. Primero ejecutasnext
para obtener y comparar el DNI y luego ejecutasnext()
otra vez para imprimir, y este es el siguiente elemento del usado para comparar el DNI, con lo que cuando llegas al último, el segundonext
te da el error. Guarda el retorno del primernext()
, que será un objetoTrabajador
, y usa el objeto para imprimir, no ejecutes otra veznext()
.for ( : )
es azúcar sintáctico que internamente usa un iterador