[c++] controllo input

[carugo]

Ciao a tutti

#include <cstdlib>
#include <iostream>

using namespace std;

const int MAX_FIUMI = 300;
const int MAX_CITTA = 100;

int FiumiCitta [MAX_FIUMI] [MAX_CITTA] ;


void CaricaPerFiume (int FiumiCitta [MAX_FIUMI] [MAX_CITTA],
                     int n_fiumi, int n_citta)
{
                  
               
  for (int fiumi=0; fiumi<n_fiumi; fiumi++)
    for (int citta=0; citta<n_citta; citta++)
    {    
      do 
        {  
          cout <<endl <<" Inserire 1 se il fiume " <<fiumi +1<<
                       " bagna la citta " <<citta +1 << " ,0 se non la bagna ";
          cin >>FiumiCitta [fiumi] [citta]; cout <<endl;
            if (FiumiCitta[fiumi] [citta] != 0 && FiumiCitta[fiumi] [citta] != 1)
            
               cout  <<" Errore inserire 1 o 0" <<endl;
                
          }
            while (FiumiCitta[fiumi] [citta] != 0 && FiumiCitta[fiumi] [citta] != 1);

In questa funzione vorrei avere il controllo dell input da tastiera se inserisco dei numeri che non sono 1 o 0 mi da errore e ripete il ciclo tutto bene,ma se inserisco una lettera il programma va in loop.....

TNX a tutti

[Freaxxx]

riporta un esempio completo, incluso il main e un esempio di input per la tua funzione .

[vendettaaaaa]

Tempo fa avevo pensato un po' a come fare, ma non ricordo la soluzione che avevo scelto. A pensarci un po' sbrigativamente, e contando che non tocco il C++ da un po', ho tirato fuori questo:

#include <iostream>
#include <string>

using namespace std;

int read_int_from_console()
{
READ:
	string in;
	cin >> in;

	try
	{
		return stoi(in); // stoi converte una string ad int; se non riesce, lancia eccezione
	}
	catch (exception ex)
	{
		cout << "string " << in << " couldn't be cast to int!!" << endl << endl;
		goto READ;
	}
}

int main()
{
	cout << "Enter an integer number." << endl;

	int n = read_int_from_console();
	cout << "You entered: " << n << endl << endl;

	return 0;
}

Premesso che ti serve un compilatore abbastanza aggiornato (gcc 4.7 o più nuovo, mingw 4.7 o più nuovo, Visual Studio 2012 o più nuovo) per poter usare la funzione stoi, l'idea è leggere una stringa e tentare la conversione ad intero, intercettando le eccezioni che farebbero partire il loop.
Problema: se scrivi 12asdas in console, ti viene restituito 12 anzichè dare errore. Soluzione: pensaci tu, questa risposta è solo un suggerimento sulla strada da percorrere

[Silvia2003]

Nel caso suggerirei questa variante che evita il 'GOTO'.



int read_int_from_console() {

int ret;

string in;

bool loop = false;

while(!loop) {

cin >> in;

try {

ret = stoi(in);

loop = true;

}

catch (exception ex) {

cerr << "Error!" << endl;

}

}

return ret;

}

[carugo]

Grazie a tutti per le risposte questo è l 'intero codice

#include <cstdlib>
#include <iostream>

using namespace std;

const int MAX_FIUMI = 300;
const int MAX_CITTA = 100;

int FiumiCitta [MAX_FIUMI] [MAX_CITTA] ;


void CaricaPerFiume (int FiumiCitta [MAX_FIUMI] [MAX_CITTA],
                     int n_fiumi, int n_citta)
{
                  
               
  for (int fiumi=0; fiumi<n_fiumi; fiumi++)
    for (int citta=0; citta<n_citta; citta++)
    {    
      do 
        {  
          cout <<endl <<" Inserire 1 se il fiume " <<fiumi +1<<
                       " bagna la citta " <<citta +1 << " ,0 se non la bagna ";
          cin >>FiumiCitta [fiumi] [citta]; cout <<endl;
            if (FiumiCitta[fiumi] [citta] != 0 && FiumiCitta[fiumi] [citta] != 1)
            
               cout  <<" Errore inserire 1 o 0" <<endl;
                
          }
            while (FiumiCitta[fiumi] [citta] != 0 && FiumiCitta[fiumi] [citta] != 1);
         
     }
     
}

void StampaPerFiume (int FiumiCitta [MAX_FIUMI] [MAX_CITTA],
                     int n_fiumi, int n_citta)
{
  for (int fiumi=0; fiumi<n_fiumi; fiumi++)
    {
      cout << endl << "Situazione per il fiume" <<fiumi +1 <<": ";
      
      for (int citta=0; citta<n_citta; citta++)
      cout <<FiumiCitta [fiumi] [citta];
     
     
    }
     
}

void CaricaPerCitta (int FiumiCitta [MAX_FIUMI] [MAX_CITTA],
                     int n_fiumi, int n_citta)
{
  for (int citta=0; citta<n_citta; citta++)
    for (int fiumi=0; fiumi<n_fiumi; fiumi++)
    {
        cout <<endl <<" Inserire 1 se la citta " <<citta +1<<
              " e' bagnata dal fiume " <<citta +1 << " ,0 se non e' bagnata ";
        cin >>FiumiCitta [fiumi] [citta];
     }
     
}

int main(int argc, char *argv[])
{
      CaricaPerFiume (FiumiCitta,3,2);
    //CaricaPerCitta (FiumiCitta,3,2);
    //StampaPerFiume (FiumiCitta,3,2);
    
    cout <<endl;
    
    system("PAUSE");
    return EXIT_SUCCESS;
}

da neofita pensavo che il seguente codice

do 
        {  
          cout <<endl <<" Inserire 1 se il fiume " <<fiumi +1<<
                       " bagna la citta " <<citta +1 << " ,0 se non la bagna ";
          cin >>FiumiCitta [fiumi] [citta]; cout <<endl;
            if (FiumiCitta[fiumi] [citta] != 0 && FiumiCitta[fiumi] [citta] != 1)
            
               cout  <<" Errore inserire 1 o 0" <<endl;
                
          }
            while (FiumiCitta[fiumi] [citta] != 0 && FiumiCitta[fiumi] [citta] != 1);

avrebbe risolto il problema anche con simboli ,lettere......proverò vostre soluzioni

TNX A TUTTI