Eu fiz um outro teste e o código original que postei não funciona em qualquer situação. Fiz um outro que é mais complicado, mas funciona. Ele pega uma string e tenta convertê-la para int
. Se falhar é o mesmo que se o número seja inválido. Antes funciona quando havia um erro específico, agora trata qualquer entrada inválida. E como é feito através de função especializada, não precisa se preocupar com o estado do stream padrão.
#include <iostream>
#include <string>
#include <sstream>
using namespace std;
int ActionGenerator() {
int Action;
string Input = "";
while(true) {
cout << endl << "SELECIONE UMA ACTION: " << endl;
cout << "[1 = ActionX]" << endl << "[2 = ActionZ]" << endl;
getline(cin, Input);
stringstream myStream(Input);
if ((myStream >> Action) && (Action == 1 || Action == 2)){
return Action;
} else {
cout << endl << "ACTION INVALIDA !" << endl;
}
}
}
int main() {
cout << ActionGenerator() << endl << "ok" <<endl;
return 0;
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Note que fiz um teste com uma string mais que um caractere. Isto falhava no código anterior.
O código está difícil de entender porque não está mal indentado. Pode parecer bobagem mas isto faz uma diferença enorme para entender o código.
Tem parte que nunca será executada. E de fato não existe lógica para sair do loop a não ser que digite 1 ou 2. E nestes casos encerra-se a execução da função.
Mas o problema mesmo é que você não está liberando o estado do stream. Quando ocorre um erro uma flag de erro é setada para você saber que houve um problema quando tentou pegar um valor válido. Quando você digita algo que não pode ser convertido no tipo esperado, no caso um int
, além do número ser convertido para 0, está flag fica setada e só se você explicitamente limpar isto é que poderá voltar trabalhar com o stream. Se a flag estiver setada o stream não funciona.
Outro problema ocorre que o dado digitado ainda fica no buffer e você precisa ignorar o que está nele.
Ou seja, o stream guarda estado e você tem que resolver isto.
Aproveitei e fiz uma melhoria para este caso:
#include <iostream>
using namespace std;
int ActionGenerator() {
int Action;
while(true) {
cout << endl << "SELECIONE UMA ACTION: " << endl;
cout << "[1 = ActionX]" << endl << "[2 = ActionZ]" << endl;
cin >> Action;
if (Action == 1 || Action == 2){
return Action;
} else {
cout << endl << "ACTION INVALIDA !" << endl;
cin.clear(); //Limpa a flag de erro quando há falha no parse do valor entrado
cin.ignore(); //Limpa o buffer
}
}
}
int main() {
cout << ActionGenerator() << endl << "ok" <<endl;
return 0;
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Tem outras maneira de tratar isto mas acho que para o exemplo simples que você deu não tem porque fazer nada mais complexo que isto.
Note que eu simplifiquei o loop também. Não havia razão para existir aquele controle neste caso.