Probleme indefini (debutant).

Probleme indefini (debutant). - C++ - Programmation

Marsh Posté le 03-02-2014 à 19:10:40    

Bonjour je débute en C++ et pour m'entrainer j'ai fait un petit programme pour calculer le volume d'un cylindre. Mais quand on entre un caractère par erreur, une lettre par exemple, le programme entre dans une boucle ou il répété la question à l'infini sans attendre la réponse... Je m'attendais à ce que le programme mette 0 en résultat...Je ne sais pas ou est le problème alors google à eu du mal à m'aider.
 

Code :
  1. #include <iostream>
  3. using namespace std;
  5. void messageerreur (double dimension)
  6.      {
  8.          if (dimension <=0)
  9.         {
  10.             cout << " Entrez un nombre positif. " << endl;
  11.         }
  13.      }
  15. int main()
  16. {
  17.     cout << "Calcul du volume d'un cylindre." << endl;
  19.    
  20.     double const pi(3.14);
  21.     double r(0),h(0),resultat(1);
  23.     //Demander d'entrer le rayon et la hauteur n'autoriser que les nombres positifs.
  24.      while (r<=0)
  25.      {
  26.         cout << "Rayon:"<<endl;
  27.         cin >> r;
  28.         messageerreur (r);
  30.      }
  32.      while (h<=0)
  33.      {
  34.         cout << "Hauteur:" <<endl;
  35.         cin >> h;
  36.         messageerreur (h);
  38.      }
  40.    
  41.         resultat = (pi*(r*r))*h;
  43.    
  44.         cout << "Le volume du cylindre est de: " << resultat << endl;
  46.     return 0;
  47. }


 
Comment afficher le message d'erreur si l'utilisateur entre une lettre ou tout autre caractere que le type double au lieu de cette boucle ? Merci

Reply

Marsh Posté le 03-02-2014 à 19:10:40   

Reply

Marsh Posté le 03-02-2014 à 19:20:58    

Ton problème doit venir du fait que cin ne peut pas mettre une lettre dans une variable de type double. Sauf que j'imagine que du coup, le caractère sur l'entrée standard n'est jamais supprimé. Du coup quand ton programme réexécute l'instruction cin >> r, il retente de mettre le caractère dans la variable double. Ce qui provoque ta boucle infinie.

 

Je crois que tu peux tester si tu as reçu une valeur adéquate de ton entrée standard en testant le failbit de cin. Si tu n'as pas reçu la donnée que tu voulais (dans ton cas, si tu as reçu une lettre au lieu d'un chiffre), tu dois pouvoir "nettoyer" cin de manière à ce que le caractère soit ignoré.

 

Donc en gros dès que tu fais

 
Code :
  1. cin >> r;
 

Tu rajoutes le test, et tu "supprimes" le caractère de l'entrée standard

 
Code :
  1. cin >> r;
  2. if (!cin)
  3. {
  4.    cout << "Erreur d'entree" << endl;
  5.    cin.clear();
  6.    cin.ignore(std::numeric_limits<std::streamsize>::max())
  7. }
 

Tout ça à prendre avec des pincettes, ça fait longtemps que j'ai pas touché à ça :D

 

EDIT: Je me suis renseigné, cin.clear() ne supprime pas du tout la mauvaise entrée utilisateur de cin. A priori il faut utiliser cin.ignore(std::numeric_limits<std::streamsize>::max()) pour ignorer tout ce qui a été écrit sur l'entrée standard.


Message édité par WiiDS le 03-02-2014 à 19:26:36

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray, 2010
Reply

Marsh Posté le 03-02-2014 à 22:35:10    

Je crois que j'ai visé un peu haut avec ce contrôle d'erreurs, je ne connais pas ces notions de failbit mais je vais essayer.
J'ai trouvé ca aussi  
http://cpp.developpez.com/faq/cpp/ [...] cin_verify
 
ca a éliminé le probleme de boucle sans fin mais ca deconne maintenant :D
 

Code :
  1. #include <iostream>
  2. #include <limits>
  4. using namespace std;
  6. bool messageerreur (double dimension)
  7.      {
  9.             while ( ! ( cin >> dimension ) )
  10.     {
  11.         if ( cin.eof() )
  12.         {
  13.            
  14.             return false;
  15.         }
  16.         else if ( cin.fail() )
  17.         {
  18.             cout << "Entrez un nombre positif : "<< endl;
  19.             cin.clear();
  20.             cin.ignore( numeric_limits<streamsize>::max(), '\n' );
  22.     }
  23.     }
  24.         return true; // succès
  25.      }
  27. int main()
  28. {
  29.     cout << "Calcul du volume d'un cylindre." << endl;
  31.     //Variables: constante pi, double rayon, hauteur, resultat.
  32.     double const pi(3.14);
  33.     double r(0),h(0),resultat(1);
  35.    
  36.         cout << "Rayon:"<<endl;
  37.         cin >> r;
  38.         messageerreur (r);
  40.         cout << "Hauteur:" <<endl;
  41.         cin >> h;
  42.         messageerreur (h);
  44.         resultat = (pi*(r*r))*h;
  46.         cout << "Le volume du cylindre est de: " << resultat << endl;
  48.     return 0;
  49. }


 
Il faut que je tapes 2 fois la variable.  
En tous cas merci d'avoir répondu aussi vite.


Message édité par MontGros le 03-02-2014 à 22:37:01
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed