probleme sur appel fonction menu

Version imprimable

Bonjour à tous,
j'ai codé une petite calculatrice toute simple mais le soucis c'est que mon menu n'apparait pas. Du coup je me retrouve à devoir saisir des chiffres sans avoir choisi le type d'opération.
Voici mon code:

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
 
int menu()
{
    int choix = 0;
 
    while (choix <1 && choix > 4)
    {
        printf("Bonjour et bienvenue sur ma calculatrice \n\n");
        printf("Choisissez une fonction:\n\n");
        printf(".1 addition\n");
        printf(".2 soustraction\n");
        printf(".3 multiplication\n");
        printf(".4 division\n");
        printf("Votre choix :\n\n");
        scanf("%d", &choix);
    }
    return choix;
}
int adittion(int a, int b)
{
    return a + b;
}
 
int soustraction(int a, int b)
{
    return a - b;
}
int multiplication(int a, int b)
{
    return a * b;
}
int division(int a, int b)
{
    return a / b;
}
 
int main(void)
{
    int a = 0, b =0 , continuer = 0, resultats = 0;
 
    printf("nombre 1 :\n");
    scanf("%d", &a);
    printf("nombre 2 :\n");
    scanf("%d", &b);
 
     do
     {
        switch (menu())
         {
         case 1:
            adittion(a, b);
            break;
         case 2:
            soustraction(a, b);
            break;
         case 3:
            multiplication(a, b);
            break;
         case 4:
            division(a, b);
            break;
         default:
            printf("Operation inconnue\n\n");
            break;
         }
 
          printf("resultats :%d", resultats);
          printf("\n");
          printf("souhaiter vous continuer 1/0 ?\n");
          printf("votre choix :\n\n");
          scanf("%d", &continuer);
     }while (continuer == 1);
 
    system("PAUSE");
    return 0;
}

Merci à tous

29/03/2016, 21h32
Matt_Houston

choix <1 && choix > 4 : quand cette condition est-elle vérifiée ?
29/03/2016, 22h01
wil31

oui en plus j'ai inverser les signes pour verifier.
merci
29/03/2016, 22h04
wil31

par contre j'ai du faire une erreur car maintenant mon menu s'affiche,je peux l'operation a effectuer puis entrer les deux nombres et ensuite il repart directement au choix du menu.
je suppose que je doit avoir une erreur au niveau de mes boucles?
30/03/2016, 15h13
dinobogan

Redonne la version modifiée de ton code.
30/03/2016, 15h17
Médinoc

Le code que tu as posté calcule les résultats sans les afficher.

alors voici mon code modifie:

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
int menu()
{
 
 
        printf("\t==========================================\n"
               "\t=Bonjour et bienvenue sur ma calculatrice= \n"
               "\t==========================================\n\n"
               "Choisissez une fonction:\n\n"
               "*********************\n"
               ".1 addition         *\n"
               "*********************\n"
               ".2 soustraction     *\n"
               "*********************\n"
               ".3 multiplication   *\n"
               "*********************\n"
               ".4 division         *\n"
               "*********************\n\n"
               "Votre choix :\n\n");
 
 
 
 
 
    return 0;
    }
int adittion(int a, int b)
{
    return a + b;
}
 
int soustraction(int a, int b)
{
    return a - b;
}
int multiplication(int a, int b)
{
    return a * b;
}
int division(int a, int b)
{
    return a / b;
}
 
 
 
 
 
int main(void)
{
 
    int a , b , continuer, choix = 0, resultats;
 
          do
              {
                   menu();
                   scanf("%d", &choix);
 
                        if (choix < 1 || choix >4)
 
                                 {
                                     printf("Erreur, verifier votre saisie merci !!\n\n");
                                    menu();
                                    scanf("%d", &choix);
 
                                 }
 
              printf("\n");
              printf("nombre 1 :\n"
                     "\n");
              scanf("%d", &a);
              printf("\n");
              printf("nombre 2 :\n"
                     "\n");
              scanf("%d", &b);
 
 
 
 
 
 
        switch (choix)
               {
                   case 1:
                     resultats = adittion(a, b);
                     break;
                   case 2:
                     resultats = soustraction(a, b);
                     break;
                   case 3:
                     resultats = multiplication(a, b);
                     break;
                   case 4:
                     resultats = division(a, b);
                     break;
                   default:
                     printf("Operation inconnue\n\n");
                     break;
 
              }
 
            printf("\n");
            printf("resultats :%d\n", resultats);
            printf("\n");
            printf("souhaiter vous continuer 1/0 ?\n");
            printf("votre choix :\n\n");
            scanf("%d", &continuer);
 
 
    } while (continuer == 1);
 
    getchar();
    return 0;
 
}

par contre j'ai un souci avec la boucle de mon menu au cas ou l'utilisateur entrerait un chiffre autre que ceux attendu.
merci a tous

02/04/2016, 11h52
Matt_Houston

J'ai un souci... oui certes, mais prend l'habitude de décrire précisément ton problème, nous ne sommes pas dans ta tête.

En l'occurrence, renseigne-toi sur le rôle du mot-clef continue.

C'est un problème connu lié au fait que scanf() laisse premier caractère "invalide" lu dans le buffer d'entrée. Ce qui cause des boucles infinies si tu ne fais que des scanf("%d", ) en boucle.
Ce qu'il faut, c'est "vider le buffer d'entrée" (ou plus exactement, finir de lire la ligne) après chaque appel à scanf.

Pour cela, on va faire une fonction qui lit en boucle les caractères jusqu'à avoir lu le '\n' (et qui s'interrompt en cas d'erreur, pour éviter les boucles infinies).
Code:

1 2 3 4 5 6 7 8 //Returns 0 if OK, a negative value if EOF. int fpurge(FILE *f) { int c; while((c=fgetc(f))!=EOF && c!='\n') { } return (c==EOF ? -1 : 0); }
Ainsi, on peut appeler cette fonction après chaque scanf().
On peut même écrire une fonction qui se charge de demander en boucle jusqu'à ce qu'une bonne valeur soit entrée:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 int scanf_nombre(int minimum, int maximum) { int valeur = 0; int nbReconnu; int resultat_fpurge; do { nbReconnu = scanf("%d", &valeur); resultat_fpurge = fpurge(stdin); /*On quitte la boucle en cas d'erreur sur fpurge(), car ça veut dire que scanf() ne réussira jamais*/ if(resultat_fpurge < 0) return -1; } while(nbReconnu != 1 || valeur < minimum || valeur > maximum); return valeur; }

02/04/2016, 13h58
wil31

ok, merci d'avoir prit le temp de me repondre.