[C++] Controllo validità nell'immissione di valori

[meclix]

Salve a tutti!

Non riesco a capire il comportamento che si verifica quando cerco di eseguire il controllo dell'input di un semplice numero intero...
Non voglio usare le eccezioni.. (essendo principiante è un'argomento che ancora non ho affrontato)..

Pensavo di cavermela in questo modo:

#include <iostream>

using namespace std;

int value;

int main() {

    while(true) {
        cout << "Type a number: ";
        if(!(cin >> value))
            break;
    }

    cout << "Errore!\n";

    return(0);

} 

In questo caso se ad esempio si dovesse verificare l'immissione di un carattere anziché un numero, chiude il ciclo e stampa il messaggio d'errore!

Ma se io non volessi interrompere il ciclo, avvisare dell'errore e ripetere il ciclo sino a quando il valore è inserito correttamente, ho pensato di cavarmela in questo modo:

#include <iostream>

using namespace std;

int value;

int main() {

    while(true) {
        cout << "Type a number: ";
        if(!(cin >> value))
            cout << "Error!\n";
        else
            break;
    }

    cout << "Number is: " << value << '\n';

    return(0);

} 

Ma questa volta non me la sono cavata molto bene.. poiché alla pressione di un carattere (e dopo l'invio ovviamente).. succede il delirio.. ovvero il ciclo ricomincia da capo, stampa il messaggio "Type a number: ", ma questa volta non aspetta l'immissione del valore ma ricomincia nuovamente il ciclo da capo.. come se fosse infinito!!
Come mai si comporta così?
Pensavo che ad ogni modo avesse aspettato l'immissione di un nuovo valore invece... no..
Devo abbandonare questo ragionamento ?
Il problema è che non riesco ad aspettare le eccezioni... (ancora lontane) e l'idea che si possa verificare un errore mi fa saltare un pò i nervi..

Grazie ancora!

[Squall1988]

Secondo me c'è qualche problema anche a livello di algoritmo... però non ricordo bene se la cin dia effettivamente in uscita uno 0 quando viene inserito un valore errato, ma non ricordavo una cosa del genere.
Ad ogni modo, prova a scrivere i cicli in modo più carino, senza quei break che se no se passa qualche persona serissima ci sviene sul colpo e non lo riprendiamo più...

[meclix]

Ciao,
A dire il vero ho provato a consultare il libro sul quale sto 'studiando', e sinceramente non dice se la cin restituisce zero in caso di errore...

Poi credo di non conoscere altri modi per terminare un ciclo, intendi che devo chudere i cicli con return; ?

In ogni caso il metodo che ho usato è 'casalingo', ma sarà sicuramente errato...
però non capisco perchè si comporta così, se l'istruzione riceve un dato errato viene saltata e si chiude il programma (questo quando non c'è un ciclo) con il ciclo invece mi trovo costretto a terminarlo altrimenti succede quello che ho anche detto sopra.. non capisco...

dove sto sbagliando ?

[Squall1988]

Dal punto di vista "stilistico" conviene mettere una condizione sensata sul ciclo ed evitare return e break all'interno, questo contribuisce ad una programmazione più strutturata.
Per l'algoritmo di per sé, sarò abbastanza franco: scrivendo un programma console non ho mai prestato particolare attenzione al fatto che l'utente inserisca un carattere al posto di un numero, o meglio, accettavo che in caso contrario il programma "scoppiasse".
Sinceramente non ho mai provato neppure con i blocchi try...catch, quindi non ti saprei dare un aiuto diretto.
Il metodo che usavo quando lavoravo in Assembly era di leggere tutto come una stringa, poi farmi le conversioni che mi servivano. Ad esempio, leggi una stringa e poi usi la funzione atoi() per tentare una conversione in numero. In questo modo la lettura non darà mai errori e l'atio() di certo non scoppia se gli passi una stringa senza numeri, al massimo dà 0.

[oregon]

Devi usare il metodo clear e l'ignore ...

  if(!(cin >> value)) 
  {
     cout << "Error!\n"; 
     cin.clear();
     cin.ignore(100, '\n');
  }

[Squall1988]

cin.clear() è come la fflush del C??

[oregon]

Originariamente inviato da Squall1988
cin.clear() è come la fflush del C??

No ... la fflush e' assimilabile alla ignore ... la clear serve a resettare la condizione d'errore ...

[Squall1988]

io però ricordo che con la ignore avevo non pochi problemi se dovevo leggere dati dopo la stringa...

[oregon]

Ovvero?

[MarkRed]

Originariamente inviato da oregon
Devi usare il metodo clear e l'ignore ...

  if(!(cin >> value)) 
  {
     cout << "Error!\n"; 
     cin.clear();
     cin.ignore(100, '\n');
  }

Scusate l'intromissione sono un alle primissime armi con il C/C++ ed ho notato che il codice suggerito funziona egregiamente ma, purtroppo, non si ha il totale controllo d'inserimento di SOLI numeri, in quanto se si inserisce un numero seguito da una lettera, salta il controllo - lo considera come se si avesse inserito un intero.

Grazie dell'attenzione.
Marco.