Existem alguns problemas no seu código.
Eu criei a variável input
aí no método mesmo, você deve ter criado em outro lugar. Pode até ter algum motivo para você ter feito isto, mas eu acho que dentro do método é o melhor lugar para declarar esta variável. Não vejo sentido em criar fora dele algo que parece ser usado só dentro dele. Use melhor o escopo.
Haviam variáveis desnecessárias. O código é mais simples. A única coisa que você precisava fazer era tentar retornar uma valor inteiro convertido dos caracteres lidos no teclado.
Se essa tentativa falha ela vai tentar executar algum catch
. Não necessariamente estes dois listados aí, poderia ser algum outro catch
no call stack da sua aplicação (inclusive o catch
oculto que toda aplicação Java tem para proteger sua aplicação de qualquer exceção não tratada).
Normalmente você só usa um único bloco de try
nessas situações. Você coloca quantos catch
es forem necessários para dar o tratamento correto para cada erro que você sabe como tratar.
Se der um destes dois problemas indicados pelo seu código ao invés de retornar o resultado, antes de concluir o return
ele vai desviar para um dos blocos que sabem como tratar o problema. No caso ambos apenas mostram a mensagem personalizada indicando para o usuário o que ocorreu de errado.
Como isto está dentro de um loop todo o processo será tentado novamente. No momento que deu tudo certo o return
é executado até o fim sem um desvio para um catch
, então o método é finalizado com o resultado adequado.
Este while
é chamado loop infinito. Será executado até que seja verdadeiro e eu estou dizendo que é verdadeiro sempre. Mesmo ele sendo infinito, é claro que existe um meio de escapar dele, que no caso é o return
.
Eu fiz ele um pouco diferente:
public int lerInt() {
BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
while (true) {
try {
return Integer.parseInt(input.readLine());
} catch (IOException e) {
System.out.println("Falha critica.");
} catch (NumberFormatException e) {
System.out.println("Você digitou uma entrada invalida.");
}
}
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Note que sua tentativa de fazer recursão do método lerInt()
pode trazer problemas de estouro da pilha em casos extremos de muitos dados inválidos. Raramente recursão é a solução.
Essa pergunta deve te interessar. Ela explica melhor sobre o uso do return
antes de um finally
que é praticamente a mesma coisa de um catch
.