Discussion :

[Pirodactil]

Je suis un débutant et je voudrais savoir comment trouver vous ce mini programme archi débutant . Les critiques sont les biens venus.

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
#include <stdio.h>
#include <stdlib.h>
 
int menu()
{
int identifier = 0;
 
while (identifier != 37)
{
    printf("\n=== Mot de Passe ===\n\n");
    printf("Identifez-vous : ");
    scanf("%d", &identifier);
}
  int choix = 0;
 
while (choix < 1 || choix > 5)
{
    printf("\n\nOperation :\n\n");
    printf("1. Addition\n");
    printf("2. Soustraction\n");
    printf("3. Multiplication\n");
    printf("4. Division\n");
    printf("5. Quitter\n\n");
    printf("Votre choix ?\n");
    scanf("%d", &choix);
    printf("\n\n");
}
return choix;
}
int main ( int argc, char** argv )
{
 
 
int operation;
int resultat = 0, nombre1 = 0, nombre2 = 0;
int Menu = 1;
 
do
{
 
switch (operation = menu())
 
{
case 1:
printf("Addition :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 + nombre2;
printf("%d + %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 2:
printf("Soustraction :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 - nombre2;
printf("%d - %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 3:
printf("Multiplication :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 * nombre2;
printf("%d * %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 4:
printf("Division :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 / nombre2;
printf("%d / %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 5:
printf("Appuyer sur une touche pour continuer...");
break;
 
default:
printf("Vous n'avez pas rentre un nombre correct.");
break;
}
 
printf("\n\n");
 
 
system("PAUSE");
printf("Revenir au menu?\n");
printf("1-Oui\n");
printf("2-Non\n");
scanf("%d",&Menu);
 
}while(Menu != 0);
 
return 0;
}

Comme vous l'avais vu le mot de passe est 37

ps: Désolé pour les fautes d'ortho

[lilington]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while (choix < 1 || choix > 5)
{
    printf("\n\nOperation :\n\n");
    printf("1. Addition\n");
    printf("2. Soustraction\n");
    printf("3. Multiplication\n");
    printf("4. Division\n");
    printf("5. Quitter\n\n");
    printf("Votre choix ?\n");
    scanf("%d", &choix);
    printf("\n\n");
}

sinon , ton while fait l'inverse de ce que tu veux. en C le while continue tant que l'expression =1. prenons par exemple le cas ou choix = 2 pour une soustraction on a while (2<1 || 2>5) ce qui donne while(0||0) ce qui donne while(0) donc on quite la boucle sans faire la soustraction.
je crois que tu voulais dire:

Code :

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

1
2
3
while(choix>0&&choix<6)  
//ou encore 
while(choix>=1&&choix<=5)

[Médinoc]

^Non non, cette partie du code est correcte: On pose la question tant qu'on n'a pas une réponse valide.

[bpy1401]

Bonjour Pirodactil

Est tu sur que cela compile correctement.

Pour moi, il est interdit de faire une déclaration de variable locale après du code (ligne 14 de ton code). ce n'est pas ANSI C.

Remplace

Code :

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

int menu()

par

Code :

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

int menu(void)

je préfère

Code :

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

1
2
3
 
   operation = menu();
   switch (operation)

au lieu de

Code :

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

1
2
 
  switch (operation = menu())

Car je n'aime pas les affectation dans un test.

Tu peux utiliser getch() au lieu de system("PAUSE");

et pour finir

Code :

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

1
2
3
4
5
6
printf("Revenir au menu?\n");
printf("1-Oui\n");
printf("2-Non\n");
scanf("%d",&Menu);
 
}while(Menu != 0);

0 n'est pas une option de ton menu

Cordialement

[Médinoc]

^ getch() n'est pas standard (bon OK, system("pause") non plus)

[gangsoleil]

Bonjour,

Lire les entrees clavier avec scanf est un probleme : si l'utilisateur rentre une lettre au lieu d'un nombre, le programme plante.

Regarde les tutoriaux sur "comment faire des entrees/sorties"

Il faut aussi que tu indentes ton code.

[Sve@r]

Pour moi, il est interdit de faire une déclaration de variable locale après du code (ligne 14 de ton code). ce n'est pas ANSI C.

Salut

Sisi, c'est maintenant autorisé depuis, je crois, C99...

Remplace

Code :

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

int menu()

par

Code :

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

int menu(void)

Là c'est une notation C++. C'est autorisé aussi de la mettre en C mais ce n'est pas une obligation.

je préfère

Code :

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

1
2
3
 
   operation = menu();
   switch (operation)

au lieu de

Code :

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

1
2
 
  switch (operation = menu())

Car je n'aime pas les affectation dans un test.

Chacun voit midi à sa porte (surtout quand il est midi )

Accessoirement, l'une ou l'autre écriture produisent exactement le même code à la compilation. Donc autant c'est vrai que dans un while ((carac=fct()) != valeur_precise) c'est pratique de mettre l'affectation dans le test, autant c'est vrai que ici ta proposition apporte plus de clarté sans rien enlever en rapidité.

[Obsidian]

Sisi, c'est maintenant autorisé depuis, je crois, C99...

Ça reste mal ! :-)

En C++, je veux bien admettre qu'il y a quelques cas où ça se justifie. En C, c'est beaucoup plus contestable…