El problema está en la forma en que usas los métodos hasNextInt()
y nextInt()
.
Si leemos la documentación:
hasNextInt:
Devuelve true
si el valor del siguiente token
de entrada de este scanner
puede ser interpretado como un valor tipo int
...
nextInt:
Escanea el siguiente token
de la entrada como int
. Este método arrojará una InputMismatchException
si el siguiente token
no puede ser convertido a un valor int
válido ... Si la conversión es satisfactoria, el scanner
avanzará hasta después de la entrada que coincide.
Tu bloque de decisión entonces sólo ingresará en la primera condición (if (hasNextInt)
) si se cumple lo que vimos en la documentación.
Cuando el usuario ingresa un valor entero válido, se ingresa en la primera condición y no tienes problemas, pero cuando no se introduce un valor inválido ocurre el problema.
PROBLEMA
Cuando lees la entrada usando el método nextInt
, se lee sólo el token
que coincida con un valor int
, pero sólo lee dicho token
, no se sigue leyendo más.
Esto quiere decir que si se introduce lo siguiente:
6 5 4
se leerá sólo el valor 6 y el resto queda allí.
Claramente no es un valor numérico válido para una entrada simple, pero para el método nextInt()
lo es, ya que la entrada se compone de sólo números enteros separados por espacios.
¿Y qué sucede con el resto de números, y con los espacios entre ellos?
Pues que serán válidos también, y los espacios entre ellos son considerados como delimitadores entre números. Veamos lo que dice la documentación de la clase Scanner
:
Un scanner
rompe o separa su entrada en tokens
usando un patrón de delimitación, el cual por defecto coincide con un espacio en blanco.
El resto de números seguirán estando allí como tokens
válidos y serán devueltos uno a uno por cada llamada a nextInt()
.
De hecho en tu programa se puede hacer lo siguiente y el resultado será:
Enter number #1
1 2 3 4 5 6 7 8 9 10
Enter number #2
Enter number #3
Enter number #4
Enter number #5
Enter number #6
Enter number #7
Enter number #8
Enter number #9
Enter number #10
Sum of numbers = 55
Esto puede considerarse como un bug
en tu programa, y aunque no da errores ciertamente es un problema.
SOLUCION
Ya tu has hallado una forma de detener el bucle infinito usando nextLine()
, cuya documentación claramente establece lo siguiente:
Avanza el scanner
más allá de la línea actual y devuelve la entrada que se omitió. Este método devuelve el resto de la línea actual excluyendo cualquier salto de línea final. La posición se establece al inicio de la siguiente línea.
Entonces una solución es descartar el resto de la linea una vez obtenido el valor que nos importa. Y esto podemos hacerlo en cada iteración (justo lo que haces al agregar ese método al final de tu bucle).
Por lo tanto si se introduce más de un valor separado por espacios, se leerá el primero y el resto será descartado cuando se llama nextLine()
.
Otra forma de resolver el problema es usar nextLine()
directamente e intentar la conversión de la entrada a un tipo int
. Esto nos da un control total sobre toda la linea ingresada por el usuario.
Por ejemplo:
System.out.println("Enter number #" + count);
try {
int number = Integer.parseInt(scanner.nextLine());
sum += number;
count++;
}
catch(Exception e) {
System.out.println("Invalid number");
}
En este ejemplo estoy usando un bloque try catch
, ya que usaré el método parseInt()
de la clase Integer
.
Si la entrada es convertida satisfactoriamente a int
, se realiza la suma y se itera nuevamente. En cambio si la entrada no puede ser convertida a int
, el método parseInt()
arrojará un error, el cual capturo en la sección catch
y muestro el mensaje pertinente.
Ahora, si un usuario ingresa algo como:
3 4 5
el programa le mostrará el mensaje de número inválido y volverá a pedirle la entrada:
Enter number #1
3 4 5
Invalid number
Enter number #1
Espero que esto aclare un poco el problema que se te ha presentado y te ayude a entender el porqué de la solución que tú mismo hallaste.