Forcer l'entré d'un int par l'utilisateur C++ Résolu/Fermé

Question

Bonjour,

Étant débutant en programmation je me suis occupé a créer un programme simple de calcul d'heure. En gros en fonction d'une heure d'arrivé cela renvoi une heure de sortie en ajoutant une durée prédéterminée.

Tout marche nickel SAUF un petit détails dont je ne comprend pas l'origine (rendant difficile toute recherche pour résoudre mon problème).

Je demande à l'utilisateur d'entrer une valeur type int (nombre d'heures ou de minutes) afin de faire les calculs nécessaires. Cela marche tant que l'utilisateur rentre bien un nombre entier.
CEPENDANT si l'utilisateur entre un autre caractère (comme une lettre), mon programme plante complétement...

Comment dois je procéder? Je pensais forcer l'entrée d'un int mais comment? Existe t il une solution mieux adaptée?

Je vous envoie la partie de mon code qui pause problème afin de mieux comprendre.
Merci de votre aide.


#include <iostream>     
using namespace std;     

bool valideHeure (int heure)     
    {return (heure>= 7 && heure<= 19); }     

bool valideMinute (int minute)     
    {return (minute>= 0 && minute<= 60); }     


int main()     
{     
  int hArrivee;     
  int mArrivee;     

        do     
        {     
        cout << endl << endl << "Indiquer l'heure d'arrivee en heure (sans les minutes) : ";     
            cin >> hArrivee;     
        } while (!valideHeure (hArrivee));     

        do     
        {     
        cout << endl << "Indiquer les minutes de votre heure d'arrivee : ";     
            cin >> mArrivee;     
        } while (!valideMinute (mArrivee));     

    cout << endl << "Vous etes arrivee a : " << hArrivee << "h" << mArrivee << endl;     

    return 0;     
}

matthoffman · Accepted Answer

Voici le code:

 if (!(cin >> hArrivee))
          {
          cerr << "Merci d'entrer un entier valide" << endl;
          cin.clear();
          cin.ignore(1);
        }

NB: Pense a initialiser ta variable hArrivee de maniere a ce que tu puisses refaire un tour de boucle

choubaka · Answer

bonjour

C'est ce qu'on appelle la gestion d'exception (erreurs).
Vous devez contrôler les valeurs entrées par l'utilisateur avant de les traiter.
Un simple test "If". si l'entrée est correcte, on continue le traitement sinon on demande à l'utilisateur d'entrer à nouveau les valeurs correctes.

Lanthane · Answer

D'accord je pensé que mon booleen suffisait à définir cette exception.
Par contre créer un test "if" n'est pas un problème mais comment dire au programme :

"Si a n'est pas un int" ?

Je doit simplement remplacer mon booleen par un if effectuant la meme verification?

edit: Je vien de creer ce test IF est j'ai toujorus le meme probleme :s

int valideHeure2 (int heure) 
{ 
    if (heure>= 7 && heure<= 19) 
    { 
        return heure; 
    } 
    else 
    { 
    cout << endl << endl << "Indiquer l'heure d'arrivee en heure (sans les minutes) : "; 
    cin >> hArrivee; 
    valideHeure2 (hArrivee); 
    } 
    return 0; 
} 

int main() 
{ 
  int hArrivee; 

    cout << endl << endl << "Indiquer l'heure d'arrivee en heure (sans les minutes) : "; 
    cin >> hArrivee; 
    valideHeure2 (hArrivee); 

return 0; 
}

choubaka · Answer

c'est du C++ ?

Lanthane · Answer

En effet j'ai oublié de préciser cette information capitale :s
j'ai édité le titre pour ajouter l'information ^^

choubaka · Answer

je crois que tu dois utiliser isdigit

Lanthane · Answer

Bon soit j'ai mal utilisé isdigit, soit cela n'a pas résolu mon problème.
Déjà cela fonctionne bizarrement ^^.

Le programme me demande de nouveau d'entré mon heure d'arrivé jusqu'à 47 inclus. Au delà il a l'air de passer a l suite du programme!

Ensuite si je tape une lettre par exemple j'ai toujours le même problème! Le programme plante et lance une boucle indefiniment!

J'ai modifié mon If comme ci dessous :

int valideHeure2 (int heure)
{
    if (isdigit(heure))
    {
        return heure;
    }
    else
    {
    int hArrivee;
    cout << endl << endl << "Indiquer l'heure d'arrivee en heure (sans les minutes) : ";
    cin >> hArrivee;
    valideHeure2 (hArrivee);
    }
    return 0;
}

Lanthane · Answer

Petit up si quelqu'un a une idée merci!
Merci tout de meme a choubaka pour avoir essayé de resoudre mon probleme.

Yoda · Answer

Bonjour,

Il y aurait bien la fonction atoi() pour résoudre ton problème.
Il faut prendre l'entrée utilisateur comme un char[] et la transformer en int.
mes sources:
http://www.cplusplus.com/reference/clibrary/cstdlib/atoi/

;)

matthoffman · Answer

Tant de reponse et aucune vraie solution.
atoi ne renvoie pas d'erreurs s'il y a des caracteres non numeriques, a mon souvenir.
isdigit check les caracteres pas la string en entier, donc si tu dois faire un parcours de chaine juste pour ca c'est quand meme vraiment moche.

Bref pour faire simple voici ce qui resoudra ton probleme:

  if (!(cin >> hArrivee))
    cerr << "Merci d'entrer un entier valide" << endl;

Lanthane · Answer

Merci pour vos réponses!

La fonction atoi() me semble plutôt intéressante (même si c'est un peu tuer une mouche avec un canon).
Cependant si celle ci renvoi 0 lorsqu'on n'entre pas un entier, alors cela rentre dans mes conditions pour la demande des minutes... Donc si l'utilisateur entre une lettre et non un entier (pour les minutes), au lieu d'une erreur, le programme prendra la valeur 0... C'est mieux qu'un plantage mais ce n'est pas propre non plus!

Matthoffman, je n'ai pas vraiment compris comment utiliser tes lignes de code (a quel endroit exact par ex etc)

Peut tu me l'expliquer afin que je puisse éventuellement le réutiliser dans d'autres programmes?

Merci bien

je l'ai inséré comme ci dessous mais j'ai toujours un plantage lorsque que j'entre une lettre! Est moi qui ai mal utilisé ta fonction?
Mise à part ce problème avec l'entrée d'entiers cela marche toujours!

        do     
        {     
        cout << endl << endl << "Indiquer l'heure d'arrivee en heure (sans les minutes) : ";     
        if (!(cin >> hArrivee))     
        cerr << "Merci d'entrer un entier valide" << endl;     
        } while (!valideHeure (hArrivee));

matthoffman · Answer

Le code que je t'ai donne verifie si on peut faire rentrer le contenu du stream cin dans un entier.

Si on ne peut pas alors on ecris sur la sortie d'erreur "Merci de blablabla...."

Bien sur ce code n'empeche en rien, d'executer ta condition de boucle:
valideHeure (hArrivee)
(qui du coup en cas d'erreur ne contient rien vu que cin >> hArrivee a ete un echec)

Donc apres le comportement de ton programme c'est a toi de savoir ce que tu fais en cas d'erreur.

Un exemple, si en cas d'erreur tu veux sortir de ta boucle alors tu ecris ca:

if (!(cin >> hArrivee))  
{ 
        cerr << "Merci d'entrer un entier valide" << endl;  
        break; 
}

Lanthane · Answer

Je croyais avoir compris mais apparemment pas...

Si j'entre une lettre j'ai toujours ce fichu problème, le programme s'emballe.

En gros j'aimerais que si l'utilisateur entre tout autre caractère qu'un entier, mon programme lui redemande l'entré d'une valeur. Éventuellement accompagné par un petit message d'erreur.

J'ai essayé d'appliquer ce que tu m'a dit comme ceci :

       do   
        {   
        cout << endl << endl << "Indiquer l'heure d'arrivee en heure (sans les minutes) : ";   
        if (!(cin >> hArrivee))   
        {cerr << "Merci d'entrer un entier valide" << endl;   
        break;}   
        } while (!valideHeure (hArrivee));

vordano · Answer

bonjour,

pour vérifier le caractère, tu peux te baser sur son code ascii -> http://table-ascii.com/

pour info tu t'es trompé dans ton test sur les minutes :)
(minute>= 0 && minute<= 60) //ici tu as un total de 61 minutes, et la minute 0 et la 60 est la même minute
devrais être
(minute>= 0 && minute< 60)

Lanthane · Answer

wahou merci a toi matthoffman ca marche ENFIN!  

Donc en fait le problème venais simplement du fait que ma boucle gardé en mémoire ma variable? 

edit: juste un petit détails! si j'entre plusieurs lettre a la suite, le message d'erreur se répéte. En gros si j'entre 3 lettre j'aurais 3 messages d'erreurs.
Je supose que je doit simplement mettre un nombre plus elevé dans cin.ignore(1)?

matthoffman · Answer

De rien :).

"edit: juste un petit détails! si j'entre plusieurs lettre a la suite, le message d'erreur se répéte. En gros si j'entre 3 lettre j'aurais 3 messages d'erreurs."

Oui desole c'est a cause du ignore ^^:
cin.ignore(numeric_limits<streamsize>::max(), '
')

Voila.

"Donc en fait le problème venais simplement du fait que ma boucle gardé en mémoire ma variable? "

Si tu parles du probleme de boucle infini, non car avec le break tu sors de ta boucle de toute facon.
Si tu parles du fait que tu repassais sans cesse dans le message d'erreur alors oui c'etait ca le pb.

Lanthane · Answer

D'accord merci beaucoup. C'était rageant d'être bloqué sur un tel détail. Détail qui rendait mon programme vraiment bancal ^^

La j'ai mi le chiffre 10 dans cin.ignore. Mais est t'il possible de mettre quelque chose qui dise cin.ignore (nombre infini)?

C'est peu utile je l'accorde car je doute que quelqu'un s'amuse a entrer un grand nombre de chiffre...

Merci bien a tous ceux qui ont pris la peine de me répondre et plus particulièrement a matthoffman pour cette solution simple mais efficace ainsi que tes explications.

edit : je commence a croire que je suis plutôt du genre chiant!
j'ai fait un test en tapant 8h (erreur qui sera je pense courante).

le programme de demande donc d'entrée mes minutes (donc la suite du programme) puis affiche le message d'erreur (des minutes et non des heures).

Voici mon code avec les modification apportées.

#include <iostream>
using namespace std;

bool valideHeure (int heure)
    {return (heure>= 7 && heure<= 19); }

bool valideMinute (int minute)
    {return (minute>= 0 && minute<= 59); }


int main()
{
  int hArrivee;
  int mArrivee;


do
        {
        cout << endl << endl << "Indiquer l'heure d'arrivee (sans les minutes) : ";
        if (!(cin >> hArrivee))
          {
          cerr << "Merci d'entrer un chiffre entre 7 et 19" << endl;
          cin.clear();
          cin.ignore(10);
        } }while (!valideHeure (hArrivee));


        do
        {
        cout << endl << "Indiquer les minutes de votre heure d'arrivee : ";
        if (!(cin >> mArrivee))
          {
          cerr << "Merci d'entrer un chiffre entre 0 et 59" << endl;
          cin.clear();
          cin.ignore(10);
        }} while (!valideMinute (mArrivee));

    cout << endl << "Vous etes arrivee a : " << hArrivee << "h" << mArrivee << endl;

    return 0;
}

Lanthane · Answer

Pk la moitié du sujet a été supprimé? :s

Bon bha petit rappel j'en été la:

do
        {
        cout << endl << endl << "Indiquer l'heure d'arrivee (sans les minutes) : ";
        if (!(cin >> hArrivee))
          {
          cerr << "Merci d'entrer un chiffre entre 7 et 19" << endl;
          cin.clear();
          cin.ignore (numeric_limits<streamsize>::max(), '\n');
        } }while (!valideHeure (hArrivee));

Et j'avais le souci que si j'entré 8h, j'avais un message d'erreur avant d'avoir la demande pour la suite du programme...

Mattfoffman m'a deja beaucoup aidé! Mais je n'arrive pas a "vidé" mon cin pour évité les erreurs!

matthoffman · Answer

Désolé pour la réponse tardive. Les messages n'ont pas disparus, ils ont juste été "uppés". regarde plus haut dans le topic ;).

Pour ton programme si je me souviens bien, tu veux que 8h soit reconnu comme une erreur, et non pas que le 8 soit sauvergardé et on s'en fout du reste.

Donc voici ce que je te propose:

 if (!(cin >> hArrivee) || (cin.peek() != '\n'))
      {
        hArrivee = -1;
        cerr << "Merci d'entrer un chiffre entre 7 et 19" << endl;
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
      }

J'ai donc rajouté une vérification: cin.peek() qui te renvoie le prochain caractère présent dans ton stream. On vérifie donc si l'utilisateur a fait un retour a la ligne directement après son nombre.
J'ai rajouté une initialisation a -1 (pour que tu refasses un tour de boucle). En effet dans le cas ou l'utilisateur rentre "8h", comme je te l'avais dit cette aprem, ta variable se verra quand même attribué la valeur 8, et du coup tu sortiras de ta boucle et donc tu continueras ton programme comme si 8h n'était pas une erreur.

Voila tiens moi au courant :). Bonne nuit, et encore désolé de répondre que maintenant.

ZeRo0Cold · Answer

une fonction qui permet de forcer saisir un INT en C

int saisieInt(void)
{char s[20] , c ;
 int k=0,h;

do{
	 do{	 c=getch();
	   } while(!isdigit(c)&& c!=13&& c!=8);

		 if(isdigit(c))
		 {s[k]=c ;k++;putchar(c);}

	        if(c==8&&k>0) {putchar(8);putchar(' ');putchar(8);k--;}

}while(c!=13);

		 s[k]='\0';
		 printf("
");

		 h=atoi(s);
		 return(h);
}

seulement traduire en C++ 

bn courage;

fiddy · Answer

Bonjour, Forcer l'entrée d'un int par l'utilisateur n'est pas possible nativement. En revanche, il est tout à fait possible de contrôler la valeur et de vérifier qu'il s'agit d'un int. Voici un exemple : int nbJoueur=0; cout << "nombre:"; while( !(cin >> nbJoueur) ) { string tmp; cin.clear(); cin >> tmp; cout << "incorrect" << endl; } issu de https://forums.commentcamarche.net/forum/affich-13403731-verifier-si-une-valeur-est-un-nombre-c

Lanthane · Answer

Merci pour le code, mais la personnellement je ne comprend pas grand chose XD!

sinon j'ai trouvé ceci dans mes recherches du soir (suite aux information fournies par mattoffman) : 
http://www.siteduzero.com/tutoriel-3-308268-la-saisie-securisee-en-c.html

Tres intéressant, mais est ce vraiment nécessaire d'avoir un code si énorme pour ce problème???

Enfin si cela peu aider d'autre personnes!

Lanthane · Answer

matthoffman,

Je te remercie, après avoir effectué quelques tests sur le code que tu m' fourni, cela marche impeccable.
Quand j'aurais un peu plus de temps je relirai un peu tout ca histoire de bien comprendre le système.
Ensuite je vais l'appliquer aux différentes sections de mon programme qui on ce même problème.

Grâce a toi je vais enfin pouvoir finalisé mon programme et obtenir quelque chose de propre (et à l'abri des utilisateurs maladroits XD).

Pour le petit lien que j'ai trouvé et posté ci dessus, qu'en pense tu?

Forcer l'entré d'un int par l'utilisateur C++

23 réponses

Discussions similaires