Utilisation switch case [Débutant(e)]

**sebounty** · 05/06/2007, 16h58

Bonjour,

J'ai une question qui peut paraitre bête, mais après plusieurs essaies, je ne trouve pas la solution.

Pour parfaire mon explication j'ai repri un code

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 <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
  long choixMenu;
 
  printf("=== Menu ===\n\n");
  printf("1. Royal Cheese\n");
  printf("2. Mc Deluxe\n");
  printf("3. Mc Bacon\n");
  printf("4. Big Mac\n");
  printf("\nVotre choix ? ");
  scanf("%ld", &choixMenu);
 
  printf("\n");
 
  switch (choixMenu)
  {
    case 1:
       printf("Vous avez choisi le Royal Cheese. Bon choix !");
       break;
    case 2:
       printf("Vous avez choisi le Mc Deluxe. Berk, trop de sauce...");
       break;
    case 3:
       printf("Vous avez choisi le Mc Bacon. Bon, ca passe encore ca ;o)");
       break;
    case 4:
       printf("Vous avez choisi le Big Mac. Vous devez avoir tres faim !");
       break;
    default:
       printf("Vous n'avez pas rentre un nombre correct. Vous ne mangerez rien du tout !");
       break;
  }
 
  printf("\n\n");
 
  system("PAUSE");
}

Le probleme avec ce code c'est que lorsqu'on l'execute, et que l'on tape un caractère autre qu'un chiffre, le programme se mettra à partir en boucle Infinit. Donc Il plante.

Quel type de variable doit on utiliser ?
Car un long ou int, fera planter l'application, et l'utilisation d'un char nous fera toujours revenir au case "default" si l'on utilise un chiffre !

Certe ce probleme me paraissait simple, mais je reste tout de même bloqué dessus.

Merci de votre aide

**odsen.s** · 05/06/2007, 17h07

Salut,

Deux solutions :

Utiliser des caractères, et remplacer les entiers par des caractères dans ton switch (1 par '1', 2 par '2', etc).
Utiliser d'autres fonctions d'entrées que scanf() :
- Construire ses propres fonctions pour lire des valeurs numériques : http://benoit.aun.free.fr/article_c_entrees_fiables.php

Les entrées sont un des aspects gores du C...

**Emmanuel Delahaye** · 05/06/2007, 17h30

Envoyé par odsen.s

Construire ses propres fonctions pour lire des valeurs numériques : http://benoit.aun.free.fr/article_c_entrees_fiables.php

Très bien cet article. Je craignais le plagiat, mais non.

**stranger** · 05/06/2007, 18h12

Envoyé par sebounty

Bonjour,

J'ai une question qui peut paraitre bête, mais après plusieurs essaies, je ne trouve pas la solution.

Pour parfaire mon explication j'ai repri un code

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 <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
  long choixMenu;
 
  printf("=== Menu ===\n\n");
  printf("1. Royal Cheese\n");
  printf("2. Mc Deluxe\n");
  printf("3. Mc Bacon\n");
  printf("4. Big Mac\n");
  printf("\nVotre choix ? ");
  scanf("%ld", &choixMenu);
 
  printf("\n");
 
  switch (choixMenu)
  {
    case 1:
       printf("Vous avez choisi le Royal Cheese. Bon choix !");
       break;
    case 2:
       printf("Vous avez choisi le Mc Deluxe. Berk, trop de sauce...");
       break;
    case 3:
       printf("Vous avez choisi le Mc Bacon. Bon, ca passe encore ca ;o)");
       break;
    case 4:
       printf("Vous avez choisi le Big Mac. Vous devez avoir tres faim !");
       break;
    default:
       printf("Vous n'avez pas rentre un nombre correct. Vous ne mangerez rien du tout !");
       break;
  }
 
  printf("\n\n");
 
  system("PAUSE");
}

Le probleme avec ce code c'est que lorsqu'on l'execute, et que l'on tape un caractère autre qu'un chiffre, le programme se mettra à partir en boucle Infinit. Donc Il plante.

Quel type de variable doit on utiliser ?
Car un long ou int, fera planter l'application, et l'utilisation d'un char nous fera toujours revenir au case "default" si l'on utilise un chiffre !

Certe ce probleme me paraissait simple, mais je reste tout de même bloqué dessus.

Merci de votre aide

Bizarre quand j'execute ton prog sous linux, et en rentrant un caractere : c par exemple il ne boucle pas.
Mais bon essaye de faire comme les collégues

ont dit et tiens nous au courant

**Emmanuel Delahaye** · 05/06/2007, 18h22

Envoyé par sebounty

Le probleme avec ce code c'est que lorsqu'on l'execute, et que l'on tape un caractère autre qu'un chiffre, le programme se mettra à partir en boucle Infinit. Donc Il plante.

Ce programme n'étant pas bouclé, ça ne risque pas. Je pense que tu n'as pas posté le bon code...

**sebounty** · 06/06/2007, 10h04

Bonjour,

Ce programme n'étant pas bouclé, ça ne risque pas. Je pense que tu n'as pas posté le bon code...

Effectivement se programme ne plante pas, ce qui est logique car on lui demande de réaliser le switch case qu'une seule fois.

Je m'excuse d'avoir posté trop hativement ce code.

Le programme que j'essaye de faire, intègre une boucle while autour du switch case, permettant de réaliser celui tant que l'on ne sort pas de la boucle.

Je me suis permi de modifier le programme précédent, pour reproduire l'erreur.

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
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
	long choixMenu;
	int quit = 1;
  	while (quit != 0)
	{  
		printf("=== Menu ===\n\n");
		printf("1. Royal Cheese\n");
		printf("2. Mc Deluxe\n");
		printf("3. Mc Bacon\n");
		printf("4. Big Mac\n");
		printf("\nVotre choix ? ");
		scanf("%ld", &choixMenu);
		printf("\n");
 
		switch (choixMenu)
		{
			case 1:
				printf("Vous avez choisi le Royal Cheese. Bon choix !");
				break;
			case 2:
				printf("Vous avez choisi le Mc Deluxe. Berk, trop de sauce...");
				break;
			case 3:
				printf("Vous avez choisi le Mc Bacon. Bon, ca passe encore ca ;o)");
				break;
			case 4:
				printf("Vous avez choisi le Big Mac. Vous devez avoir tres faim !");
				break;
			case 5:
				quit = 0;
				break;
			default:
				printf("Vous n'avez pas rentre un nombre correct. Vous ne mangerez rien du tout !");
				break;
		}
	}
 
	printf("\n\n");
 
	system("PAUSE");
}

Donc comment modifier celui-ci, afin de permettre l'appui sur un caractère et de me retrouver dans le case par defaut.

Merci Sebounty

**straasha** · 06/06/2007, 10h16

le probleme c'est que tu ne repose pas la question donc des qu'un choix est fait (autre que 5) tu ne sort plus du while
pose ta question dans le while et tout ira bien

[edit]
tiens il y a eu un edit pendant mon post

[/edit]

**sebounty** · 06/06/2007, 10h20

le probleme c'est que tu ne repose pas la question donc des qu'un choix est fait (autre que 5) tu ne sort plus du while
pose ta question dans le while et tout ira bien

[edit]
tiens il y a eu un edit pendant mon post
[/edit]

oui il ya eu edit car je me suis aperçu de mon erreur

Mais tu peux essayer comme cela, tu verras que le code plante quand même sous l'appui d'un caractère.

**odsen.s** · 06/06/2007, 10h23

Donc comment modifier celui-ci, afin de permettre l'appui sur un caractère et de me retrouver dans le case par defaut.

Salut,
As-tu lu ma réponse plus haut ?

**sebounty** · 06/06/2007, 10h29

Salut,
As-tu lu ma réponse plus haut ?

Oui j'ai lu ta réponse. L'utilisation de caractères à la place d'un long ne fonctionne pas vraiment.

Le programme s'execute 2 fois avant de redonner la main.

Je suis entrain de faire des essais avec l'utilisation de getchar au lieu de scanf

Au passage très bien fait le tutorial ^^

**odsen.s** · 06/06/2007, 10h37

Oui j'ai lu ta réponse. L'utilisation de caractères à la place d'un long ne fonctionne pas vraiment.

Le programme s'execute 2 fois avant de redonner la main.

Le problème doit venir de scanf() (conservation du caractère '\n' dans stdin).

Je suis entrain de faire des essais avec l'utilisation de getchar au lieu de scanf

Très bien. Avec getchar(), n'oublie pas de vider le flux d'entrée standard après chaque appel

N'hésite pas à nous montrer tes essais.

**sebounty** · 06/06/2007, 10h53

Bonjour,

J'ai fait des copier coller enorme de ton lien

:http://benoit.aun.free.fr/article_c_entrees_fiables.php

J'ai modifié le programme précédent, et effectivement avec un bon getchar ca fonctionne

Bon certe le programme a pris en taille mais l'erreur n'y ai plus.

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
#include <stdio.h>
#include <stdlib.h>
 
enum
{
    ENTREE_OK,
    ERR_TAILLE_SAISIE,
    ERR_TAILLE_TAB,
    ERR_P_NULL,
    ERR_FORMAT
};
 
#define NB_CAR_LONG 32
 
void vider_stdin(void)
{
    int c;
    while((c = getchar()) != '\n' && c != EOF);
}
 
int lire_ligne(char* s, size_t t)
{
    int c;
    int fin;
    int err;
    size_t i;
 
    /* le pointeur ne doit pas valoir NULL */
    if (s != NULL)
    {
        /* la taille du tableau doit etre positive */
        if(t > 0)
        {
            i = 0;
            fin = 0;
 
            while(!fin)
            {
               /* lecture du caractere sur stdin */
               c = getchar();
 
               /* est-ce la fin de la saisie ? */
               if(c == '\n' || c == EOF)
               {
                  err = ENTREE_OK;
                  fin = 1;
               }
 
               /* reste-t-il de la place pour ce caractere et 
                  le caractere de fin de chaine ? */
               else if(i < t-1)
               {
                  s[i++] = c;
               }
 
               /* sinon, il faut vider stdin */
               else
               {
                  vider_stdin();
                  err = ERR_TAILLE_SAISIE;
                  fin = 1;
               }
            }
 
            /* on ajoute le caractere de fin de chaine */
            s[i] = '\0';
        }
 
        else
        {
            err = ERR_TAILLE_TAB;
        }
    }
 
    else
    {
        err = ERR_P_NULL;
    }
 
    return err;
}
 
/* lire_long : lit une chaine de caracteres et convertit en long */
int lire_long(long *p_l)
{
    int err;
 
    /* le pointeur ne doit pas valoir NULL */
    if(p_l != NULL)
    {
        /* lecture de la ligne */
        char temp[NB_CAR_LONG] = {0};
        err = lire_ligne(temp, sizeof temp);
 
        if(err == ENTREE_OK)
        {
            /* conversion en base 10 */
            char *p_conversion = NULL;
            *p_l = strtol(temp, &p_conversion, 10);
 
            /* la valeur pointee doit valoir zero */
            if(p_conversion != NULL)
            {
                if(*p_conversion != 0)
                {
                    err = ERR_FORMAT;
                }
            }
        }
    }
 
    else
    {
        err = ERR_P_NULL;
    }
 
    return err;
}
 
 
 
 
 
int main(int argc, char *argv[])
{
	long choixMenu;
	int quit = 1;
 
  	while (quit != 0)
	{  
		printf("\n\n=== Menu ===\n\n");
		printf("1. Royal Cheese\n");
		printf("2. Mc Deluxe\n");
		printf("3. Mc Bacon\n");
		printf("4. Big Mac\n");
		printf("5. quit\n");
		printf("\nVotre choix ? ");
		lire_long(&choixMenu);
		printf("\n");
 
		switch (choixMenu)
		{
			case 1:
				printf("Vous avez choisi le Royal Cheese. Bon choix !");
				break;
			case 2:
				printf("Vous avez choisi le Mc Deluxe. Berk, trop de sauce...");
				break;
			case 3:
				printf("Vous avez choisi le Mc Bacon. Bon, ca passe encore ca ;o)");
				break;
			case 4:
				printf("Vous avez choisi le Big Mac. Vous devez avoir tres faim !");
				break;
			case 5:
				quit = 0;
				printf("\nExit\n");
				break;
			default:
				printf("Vous n'avez pas rentre un nombre correct. Vous ne mangerez rien du tout !");
				break;
		}
	}
 
	printf("\n\n");
 
	system("PAUSE");
}

N'y a t'il pas plus light ?

Sebounty

**Emmanuel Delahaye** · 06/06/2007, 11h05

Envoyé par sebounty

Bonjour,

J'ai fait des copier coller enorme de ton lien

:http://benoit.aun.free.fr/article_c_entrees_fiables.php

J'ai modifié le programme précédent, et effectivement avec un bon getchar ca fonctionne

Bon certe le programme a pris en taille mais l'erreur n'y ai plus.
<...>
N'y a t'il pas plus light ?

L'idée, c'est que ce genre de code on le place un fois pour toutes dans un module séparé (par exemple input.h / input.c) que l'on compile et que l'on lie à l'application. C'est le principe de bon sens, apparemment peu connu et pas du tout enseigné de 'réutilisation du code'.

On peut aussi le mettre dans sa bibliothèque personnelle.

Utilisation switch case [Débutant(e)]

C

Discussions similaires

Partager

Partager