[C++] Condizione do while problema

[zzari]

Ciao a tutti!
Sto cercando di implementare un giochino dove bisogna inserire 2 coordinate numeriche (int righe, int colonne) comprese tra 0 e 10 ad esempio.
Quello che vorrei fare è questo praticamente:

Inserisci le coordinate: 4 C
*Errore, il numero delle colonne deve essere un intero

Inserisci le coordinate: 22 9
*Errore, il numero delle righe e colonne deve essere compreso tra 0 e 10

Inserisci le coordinate: 8 6

Le coordinate inserite sono (8,6).

Io ho pensato ad un ciclo do while del tipo:

Codice: Seleziona tutto

do{
    cout<<"Inserire il numero di righe e colonne ";
    cin>>r>>c;
        if (cin.fail())
	{
        cout << "ERROR: Il numero delle righe e colonne deve essere un numero intero." << endl << endl;

	}
        if ((r<0 || r>10) || (c<0 || c>10)
	{
        cout << "ERROR: Il numero delle righe e colonne deve essere un numero intero tra 0-10." << endl << endl;

	}
    cout<<endl;
}
while (((r<0 || r>10)||(c<0 || c>10)) || cin.fail());

Purtroppo se inserisco 2 numeri funziona senza problemi, anche per il controllo se sono minori o maggiori di 10... ma se inserisco una lettera va in LOOP infinito...
Qualche idea di come implementare il controllo sulle coordinate inserite o come corregere il mio codice!?

il succo è.. come fare a controllare che quello inserito da tastiera non sia un numero intero?!

[cobra90nj]

hai definito la variabile nella quale inserisci la lettera come "string" o sono entrambe intere?

[crap0101]

E' lo stream che resta sporco. btw, usare

Codice: Seleziona tutto

cin >> qualcosa

non è il metodo ideale per prendere input dall'utente, in ogni caso dovresti controllare che non ci siano stati errori (non solo con fail()), pulire lo stream e resettare le variabili che utilizzi.

[zzari]

hai definito la variabile nella quale inserisci la lettera come "string" o sono entrambe intere?

"r" e "c" sono definite come intere appunto perchè voglio che vengano inserite solo numeri interi, no float e no char..

E' lo stream che resta sporco. btw, usare

Codice: Seleziona tutto

cin >> qualcosa

non è il metodo ideale per prendere input dall'utente, in ogni caso dovresti controllare che non ci siano stati errori (non solo con fail()), pulire lo stream e resettare le variabili che utilizzi.

consigli di usare un getline() e prendere una stringa in input da convertire con "atoi()"? Pero come potrei fare un controllo completo per sapere se l'utente ha inserito correttamente dei numeri tra 0-10 oppure stringhe di caratteri a caso che non vanno bene?

[crap0101]

è una soluzione... puoi fare anche come stai facendo ora, però devi appunto aggiungere i controlli del caso.