Discussion :

[Johnny P.]

Bonjour,

Je suis entrain de réaliser un programme qui d'après le choix de l' utilisateur choisit 2 entiers et un caractère (parmi +, -, x et: ).
Le programme doit effectuer l’opération choisie par l'utilisateur.

Si l’utilisateur n’a pas saisi de caractère correct, il faut afficher une erreur

voici mon 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
41
42
43
 
 
#include <stdio.h>
 
void main (void) 
{
// déclaration variable
    int nombre1,nombre2;
	char choix;
 
//on demande à l'utilisateur 2 entiers 
	printf("choisir deux entiers \n");
	scanf("%d",&nombre1); /*lire l'entiers */
	scanf("%d",&nombre2);
 
 //choix de l'opération par l'utilisateur
	printf("===choix===\n");
	printf(" + \n");
	printf(" - \n");
	printf(" * \n");
	printf(" / \n");
	scanf("%c",&choix);
 
 // pour chaque cas on effectue l'operation demandé par l'utilisateur      
 
 
switch (choix)
	{
        case 1	:	printf("Resultat : %d",nombre1+nombre2);
				break; 
	case 2	: 	printf("Resultat : %d",nombre1-nombre2;
				break;
	case 3	:	printf("Resultat : %d",nombre1*nombre2);
				break;
	case 4  :   printf("Resultat : %d",nombre1/nombre2);
                break;
	default	:	printf("erreur\n");
 
	}
 
 
 
}

je peux lancer en console le test mais après avoir rentrées les 2 valeurs le menu choix s'affiche avec une erreur directement ?

Je débute en C Merci de votre aide et remarque pour ne plus refaire les erreurs là à l'avenir.

[buggen25]

Bonjour

Dans les cases il faut mettre

Code :

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

1
2
3
4
5
6
7
8
9
 
switch(choix)
{
      case '1' : ....
      break;
      case '2' :
      break;
     ......
}

[Johnny P.]

Bizarre j'ai pourtant vue sans les '' par contre ça ne résout pas le problème il m'affiche toujours erreur ?

[buggen25]

salut:
il t'afiche quel type d'erreur ?

[droggo]

Gia,

Bizarre j'ai pourtant vue sans les '' par contre ça ne résout pas le problème il m'affiche toujours erreur ?

Vu ton code, je suppose que tu entres un des caractères +,-,* ou / pour choisir l'opération ?

Ton switch doit donc tester ces caractères, et pas '1', '2' ...

[Emmanuel Delahaye]

voici mon code :

Code :

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

1
2
 
	case 2	: 	printf("Resultat : %d",nombre1-nombre2;

Il manque une )

main() retourne int. Toujours.

Ensuite, tu as un problème de saisie. le scanf() laisse trainer un '\n' qui est pris directement par le 3 ème scanf(), sans s'arrêter.

Le '\n' est donc pris comme une erreur.

Il faut travailler à la sécurisation des saisies.

Bricolage rapide :

Code :

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

1
2
3
4
5
6
7
8
 
void purge(void)
{
    int c;
   while ((c = getchar()) != '\n' && c != EOF)
   {
   }
}

qu'il faut placer après chaque scanf() si le dernier caractère lu n'est pas `\n' :

Dans le switch-case, 1 2 3 4 n'a aucun sens. C'est '+', '-' etc.

Ne pas autoriser les divisions par 0.

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
 
#include <stdio.h>
 
void purge (void)
{
   int c;
   while ((c = getchar ()) != '\n' && c != EOF)
   {
   }
}
 
int main (void)
{
/* déclaration variable */
   int nombre1, nombre2;
   char choix;
 
/* on demande à l'utilisateur 2 entiers */
   printf ("choisir deux entiers \n");
   scanf ("%d", &nombre1);      /*lire l'entiers */
   purge ();
   scanf ("%d", &nombre2);
   purge ();
 
   /* choix de l'opération par l'utilisateur */
   printf ("===choix===\n");
   printf (" + \n");
   printf (" - \n");
   printf (" * \n");
   printf (" / \n");
   scanf ("%c", &choix);
   if (choix != '\n')
   {
      purge ();
   }
   /* pour chaque cas on effectue l'operation demandé par l'utilisateur */
 
   switch (choix)
   {
   case '+':
      printf ("%d %c %d = %d", nombre1, choix, nombre2, nombre1 + nombre2);
      break;
   case '-':
      printf ("%d %c %d = %d", nombre1, choix, nombre2, nombre1 - nombre2);
      break;
   case '*':
      printf ("%d %c %d = %d", nombre1, choix, nombre2, nombre1 * nombre2);
      break;
   case '/':
      if (nombre2 != 0)
      {
         printf ("%d %c %d : %d", nombre1, choix, nombre2, nombre1 / nombre2);
      }
      else
      {
         printf ("le denominateur doit etre <> 0\n");
      }
      break;
   default:
      printf ("erreur\n");
   }
   return 0;
}

[buggen25]

Gia,

Vu ton code, je suppose que tu entres un des caractères +,-,* ou / pour choisir l'opération ?

Ton switch doit donc tester ces caractères, et pas '1', '2' ...

Oui c'est exactement ça, j'ai pas fait attention!

[Johnny P.]

merci de l'aide je viens de changer mais le problème continue après le choix de mes 2 valeurs il y a le menu de choix et en bas une erreur le programme quitte !

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>
 
void main (void) 
 
 
{
 
// déclaration variable
    int nombre1,nombre2;
	char choix;
 
//on demande à l'utilisateur 2 entiers 
	printf("choisir deux entiers \n");
	scanf("%d",&nombre1); /*lire l'entiers */
	scanf("%d",&nombre2);
 
 //choix de l'opération par l'utilisateur
	printf("==Choisir==\n");
	printf(" + \n");
	printf(" - \n");
	printf(" * \n");
	printf(" / \n");
	scanf("%c",&choix);
 
 // pour chaque cas on effectue l'opération demandé par l'utilisateur      
 
 
switch (choix)
	{
    case '+'	:	printf("Resultat : %d",nombre1+nombre2);
				break; 
	case '-'	: 	printf("Resultat : %d",nombre1-nombre2);
				break;
	case '*'	:	printf("Resultat : %d",nombre1*nombre2);
				break;
	case '/'  :   printf("Resultat : %d",nombre1/nombre2);
                break;
	default	:	printf("erreur\n");
 
	}
 
 
 
}