Discussion :

[Ziarrow]

Salut, j'ai un problème avec un switch au début de mon programme, si je tape un chiffre tout va bien mais si je tape un caractère en guise de choix, le programme bug et affiche des lignes à profusion, je ne sais pas comment résoudre ce problème.

Dois-je utiliser un autre type de variable pour "choix" ?

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
 
#include ....
.............
 
unsigned int choix = 0;
 
int main(void)
{
  while(1)
  {
    cout << "........." << endl;
    cin >> choix;
 
    switch(choix)
    {
      case 1:  instruction();
                  break;
 
      case 7:  exit(0);
 
      default:  
                {
                  cout << "\t\t Veuillez taper un chiffre entre 1 et 7" << endl;
                }
    }
  }
  return EXIT_SUCCESS;
}

[Ziarrow]

Ok, merci c'est corrigé mais je viens de tester et le prog bug quand même si j'appuie sur un caractère plutôt qu'un entier.....

[jabbounet]

J'avais dit une bétise, par contre la FAQ peut t'aider.

il te manque des contrôles sur le flux d'entrée.

http://cpp.developpez.com/faq/cpp/?p...#SL_cin_verify

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
 
#include <iostream>
#include <limits>
 
int main() {
 
  while(1) {
 
    int choix = 0;
    std::cout << "........." << std::endl;
    std::cin >> choix;
 
    if (std::cin.eof()) {
      exit(0); 
    } else if (std::cin.fail()) {
      std::cout << "Saisie incorrecte, recommencez : "; 
      std::cin.clear(); 
      std::cin.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
    } else {
 
      switch(choix)
        {
        case 1:  
          std::cout << "ok" << std::endl;
          break;
 
        case 7:  
          exit(EXIT_SUCCESS);
 
        default:  
          {
            std::cout << "\t\t Veuillez taper un chiffre entre 1 et 7" << std::endl;
          }
          break;
        }
    }
  }
  return EXIT_SUCCESS;
 
}

[Ziarrow]

Un immense merci, tu m'as épargné des heures de prise de tête......

Je vais lire ça.

P.S. : désolé, j'aurai du lire la FAQ C++ avant de poster.

[MisterS]

Un petit bonjour ,

Ne serais-ce pas plus digeste d'utilisée un using namespace ici aussi ?

Je met ma petite contribution

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
#include <iostream>
#include <limits>
 
int main() {
  using namespace std; 
 
  while(1) {
 
    int choix = 0;
    cout << "........." << endl;
    cin >> choix;
 
    if (cin.eof()) {
      exit(0); 
    } else if (cin.fail()) {
      cout << "Saisie incorrecte, recommencez : "; 
      cin.clear(); 
      cin.ignore( numeric_limits<streamsize>::max(), '\n' );
    } else {
 
      switch(choix)
        {
        case 1:  
          cout << "ok" << endl;
          break;
 
        case 7:  
          exit(EXIT_SUCCESS);
 
        default:  
          {
            cout << "\t\t Veuillez taper un chiffre entre 1 et 7" << endl;
          }
          break;
        }
    }
  }
  return EXIT_SUCCESS;
}

[jabbounet]

Un petit bonjour ,

Ne serais-ce pas plus digeste d'utilisée un using namespace ici aussi ?

Je met ma petite contribution

Code :

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

1
2
 
...

Simple question de goût, j'aime bien connaitre l'origine de la viande que je mange, il en va de même pour le c++, j'aime bien connaitre l'origine des objets, méthodes,... que je manipule.
Sur de gros projet, j'admets qu'ici ce n'est pas le cas, cela évite les conflits de nommages.