Discussion :

[NhyMbuS]

Bonjour,

j'ai voulu faire un petit menu me permettant de configurer certaines valeurs d'une classe, jusque là rien de compliqué mais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <iostream>
 
using namespace std;
 
// à l'origine c une methode de classe
// mais comme ça, ça plante également
 
void    menu()
{
    int v1 = 0; // remplacé par une variable de classe
    int v2 = 0; // de meme
    char    choix = '0';
 
    do {
        cout << "Choix ? " << endl;
        cin >> choix;
    } while (choix != '1'&& choix != '2');
 
 
    switch (choix)
    {
        case '1':
            cout << "Entrer variable v1 " << endl;
            cin >> v1;
            menu();  // je veux relancer menu pour réafficher mes variables
                         // afin de pouvoir en configurer plusieurs et lancer un
                         // traitement apres ( qui depend de ces variables )
                         // ca permet de visualiser la configuration
                         // et donc de remodifier avant de lancer le traitement
            break;
        case '2':
            cout << "Entrer variable v2 " << endl;
            cin >> v2;
            menu();
            break;
        default:
            menu();
            break;
    }
 
}
 
int main()
{
    menu();
    system("PAUSE");
}

Ce code boucle lorsque je rentre un caractere à la place d'un chiffre lors du cin sur une variable, et ceci aussi bien avec un 'a' qu'un '-' .

Choix ?
2

Entrer la variable v2 :
a

Choix ?
Choix ?
Choix ?
Choix ?
Choix ?
Choix ?
Choix ?
etc ...

quelqu'un peut il m'expliquer ce phénomène, et comment y remedier !
car pour moi en relançant menu();
mon choix est remis à 0, donc je dois forcement rentrer dans le do while et le cin devrait arreter la boucle car il attend que l'on tape quelquechose.



Merci d'avance.

[Skyounet]

Comment y remedier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
char buffer[20];
fgets(buffer, 20, stdin);
sscanf(buffer, "%d", &v1);

Il me semble que ca marche si ce n'est pas le cas, quelqu'un reagira surement.

[bruno_pages]

Choix ?
2

Entrer la variable v2 :
a

Choix ?
Choix ?
etc ...

je ne vois pas ce qu'il y a de bizarre, ton 'a' est le nom de la variable, une fois qu'il est lu tu rappelles menu (soit dit en passant, j'espère que ton compilateur voit qu'il s'agit d'une recursion terminale et transforme ton appel fonctionnel en un simple jump) ... et il n'y a aucun moyen d'en sortir car quoi qu'il arrive tu rappelle menu !

a part cela :

• la mise à '0' de choix ne sert strictement a rien, elle est donc génante et doit être retiree
• les inits de v1 et v2 ne servent à rien ici mais apparemment tu vas changer ces variables
• le cas default du switch ne sert à rien car tu ne sorts de la boucle do while que si choix est '1' ou '2'

[Laurent Gomila]

Il faut détecter les erreurs de saisie (mauvais type, comme ici un 'a' dans un int). Ensuite dans ces cas, il faut remettre le flux d'entrée (cin) dans un état valide et le vider avant de pouvoir resaisir quelque chose.

Voir la FAQ C++ pour les détails.

[NhyMbuS]

je ne vois pas ce qu'il y a de bizarre, ton 'a' est le nom de la variable, une fois qu'il est lu tu rappelles menu (soit dit en passant, j'espère que ton compilateur voit qu'il s'agit d'une recursion terminale et transforme ton appel fonctionnel en un simple jump) ... et il n'y a aucun moyen d'en sortir car quoi qu'il arrive tu rappelle menu !

c'est parce que mon menu en fait depend des variables v1 et v2 ( qui sont contenues dans une classe , la fonction menu étant une méthode de classe), en fait j'actualise le menu.
et ce qu'il y a de bizarre, c'est que le 'a' est un caractère, alors que j'attends un int, même si c'est à peu près la même chose c'est cette erreur que je dois intercepter, mais je pensais l'avoir déja fait.
J'utilise devc++, mais je ne sais pas si mingw le transforme en jump

a part cela :


la mise à '0' de choix ne sert strictement a rien, elle est donc génante et doit être retiree

les inits de v1 et v2 ne servent à rien ici mais apparemment tu vas changer ces variables

le cas default du switch ne sert à rien car tu ne sorts de la boucle do while que si choix est '1' ou '2'

ce sont des habitudes de programmation, on m'a toujours dit d'initialiser mes variables, comme ça je sais si elles n'ont pas été affectées. par exemple si je ne dois pas avoir de 0 dans mes variables, je fais un test et je sais que ça n'a pas été initialisé correctement, c'est une précaution .
Dans le cas d'un int, ça évite qu'il prenne une valeur aléatoire, comme 64452, qui par le plus grand des hasards pourrait être une valeur que j'attends.
De meme pour le default, on m'a toujours dit également qu'un switch sans default peut etre dangereux dans certains cas.

Il faut détecter les erreurs de saisie (mauvais type, comme ici un 'a' dans un int). Ensuite dans ces cas, il faut remettre le flux d'entrée (cin) dans un état valide et le vider avant de pouvoir resaisir quelque chose.

Voir la FAQ C++ pour les détails.

Je n'avais pas pensé à remettre le cin dans un état valide, je vais me plonger dans la FAQ plus assiduement ( car je l'avais déja fait avant mais surement d'une mauvaise façon )

Merci beaucoup pour vos réponses !
à vrai dire quand on est plongé dans un code, difficile de se sortir de ce que l'on croit être bon mais qui ne fonctionne pas.

cordialement
NhyMbuS

[NhyMbuS]

Merci beaucoup, mais je n'avais pas pensé à chercher dans la partie console !

ce que je ne comprennais pas, c'etait pourquoi une erreur dans les variables influait sur le choix, mais là c'est plus clair !

http://c.developpez.com/faq/cpp/?pag...#SL_cin_verify

Merci encore !