Primeiro vamos entender este código de exemplo:
Scanner sc = new Scanner(System.in);
try {
System.out.println("digite um número: ");
System.out.println(sc.nextInt());
} catch (InputMismatchException e) {
System.out.println("deu erro, tente novamente: ");
System.out.println(sc.nextInt());
}
Suponha que no primeiro nextInt()
eu não digite um número. A saída será:
digite um número:
abc
deu erro, tente novamente: Exception in thread "main"
java.util.InputMismatchException
at java.util.Scanner.throwFor(Scanner.java:864)
at java.util.Scanner.next(Scanner.java:1485)
at java.util.Scanner.nextInt(Scanner.java:2117)
at java.util.Scanner.nextInt(Scanner.java:2076)
...
O que acontece é que, como eu não digitei um número, o primeiro nextInt()
lança a exceção e o scanner não avança na string (ele continua parado antes do abc
). Em seguida, quando eu chamo nextInt()
novamente, ele tenta ler novamente o abc
, e dá erro novamente (InputMismatchException
).
É isso que está acontecendo no seu loop. Quando o nextInt()
recebe um valor inválido, cai dentro do catch
(e o scanner não avança na string). Em seguida, ele vai para a próxima iteração do for
e o nextInt()
tenta ler o valor inválido que havia sido digitado anteriormente (e cai dentro do catch
de novo, e volta para o nextInt
, que cai no catch
...). E como dentro do catch
você decrementa a variável de controle do loop, ele nunca sai do for
.
O que você pode fazer é chamar nextLine()
enquanto o valor digitado não for um número (o que pode ser verificado com hasNextInt()
), assim o scanner pula o trecho inválido:
for (int c = 0; c < students; c++) {
try {
while(! sc.hasNextInt()) {
// consome a entrada até que seja encontrado um número
sc.nextLine();
}
listOfStudents.add(sc.nextInt());
} catch (InputMismatchException e) {
System.out.println("Invalid code! Try again");
}
}
Assim eu garanto que o nextInt()
só será chamado quando o próximo valor a ser lido é um número. Note também que agora você não precisa ficar decrementando o c
, pois ele só vai para a próxima iteração do for
quando conseguir ler um número.
for
? incrementa de novo. Para tratar isso, do valor inválido, seria melhor colocar esse bloco dentro de umwhile
, como uma condição de ficar executando até que o valor seja válidowhile
que só vai sair se o valor for válido, sem alterar o valor do contador dofor
, tente fazer dessa formafor
é ser um contador linear, vai de 1 a 10, etc... se precisa ficar alterando um contador, ofor
não é a melhor opção, mas alterar o valor da variável pode, mas não é a melhor forma de controlar o loop :)