Si bien eftshift0 ya respondió, te explicaré por qué no funciona y cómo solucionarlo.
Para empezar, haces un recorrido sobre el vector de DNIs, pero por cada uno de los elementos del vector (6 en este caso), intentas comparar si es igual al que has introducido por teclado, imprimiendo el mensaje "No existe ningún alumno con DNI...".
Si bien el recorrido se realiza correctamente (recorres todo el vector), el mensaje se imprime para cada uno de sus elementos, por lo que cuando introduces un DNI por teclado, se ven 6 mensajes en la pantalla (consola). Esto es así porque se ha recorrido 6 elementos (DNIs), y todos han sido distintos al que se ha introducido por teclado.
Lo que no es correcto en este caso es anidar un bucle dentro de otro, porque solo estarías realizando las mismas comprobaciones muchas veces más.
Para saber si has encontrado un elemento no debes imprimir un mensaje cada vez que realizas una comparación, sino que debes recorrer el vector, y al terminar el recorrido, imprimir el resultado de la operación. Esto se consigue almacenando en alguna variable fuera del bucle algún valor que te indique si se ha encontrado o no el elemento introducido por teclado.
Una opción puede ser guardar en una variable el índice del elemento encontrado. Y si el índice está fuera del rango de índices del vector, se considerará que no lo has encontrado.
int indice = -1; // Valor inicial inválido
for (int i = 0; i < dni.length; i++) if (/* son iguales */) indice = i;
if (indice == -1) System.out.println("No se ha encontrado");
else System.out.println("Se ha encontrado");
La comprobación de igualdad de cadenas se debe realizar con el método equals
, y no con el operador de igualdad ==
. Esto se debe a que las cadenas son objetos. Y para objetos, el operador de igualdad ==
no comprueba si el contenido de las cadenas son iguales, sino que comprueba si las dos cadenas son la misma (un solo objeto).
Sin embargo, puede haber dos (o más) objetos con identificadores (OID) distintos, que ocupan zonas de memoria distintas, pero teniendo el mismo contenido. En estos casos, el operador de igualdad dice que son objetos distintos, aunque tengan el mismo contenido. Por ello surge el método equals
, que compara no solo la referencia sino el contenido de los objetos.
Puedes obtener más información sobre esto en mi respuesta a otra pregunta, y aún más en esta pregunta.
Por tanto, la comparación de igualdad de cadenas sería:
if (dni[i].equals(consulta)) ...
Ahora bien, explicado todo lo anterior, una mejor opción es realizar una operación de búsqueda en vez de un recorrido, ya que no son lo mismo: un recorrido es una operación en la que se recorre todos los elementos de una colección para realizar alguna operación; mientras que una búsqueda es la operación de encontrar un elemento concreto de la colección, deteniéndose en el momento en que se encuentra dicho elemento.
int i = 0;
while (i < dni.length && dni[i].equals(consulta)) i++;
if (dni[i].equals(consulta)) System.out.println("Se ha encontrado");
else System.out.println("No se ha encontrado");
Por último te comentaré que se puede hacer una búsqueda con un bucle for
, como ha indicado eftshift0 en su respuesta, es decir, en el cuerpo del bucle se comprueba si hemos encontrado el elemento, y si se ha encontrado se rompe el bucle con la instrucción break
.
Si bien esto funciona perfectamente, se suele considerar que es un mal diseño, especialmente en el ámbito universitario.