Discussion :

[bosk1000]

Bonjour,

je viens de faire en exercice un programme de calculatrice

le code bug après avoir choisi le mode de calcul en quittant le programme
elle fonctionnait lorsque je passais par un autre système, en créant un "int" pour chaque forme de calcul,
mais en voulant optimiser par un switch, la je bug
merci pour votre aide

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
 
#include <stdio.h>
 
void calc(voi);
void menu(void);
int calcul(int nb1,int nb2,int typecalcul);
 
/*-----------------------------------------------*/
int main(void)
{
    calc();
    return 0;
}
/*-----------------------------------------------*/
void calc(void)
{
    int menuchoice=0;
    int number1=0, number2=0, result=0;
 
    printf("\n---------------------------\n");
    printf("\tMINI CALCULATRICE");
    printf("\n---------------------------\n");
 
 
    do
    {
    //affichage menu
    menu();
 
    printf("\n\n Votre choix :");
    scanf("%d",&menuchoice);
 
 
 
    if (menuchoice==6)
        printf(" A bientot\n\n");
        return 0 ;
 
 
    printf("entrer un premier nombre\n");
    scanf("%d",&number1);
 
 
    printf("entrer un second nombre\n");
    scanf("%d",&number2);
 
    switch (menuchoice)
    {
        case 1:
            result=calcul(number1, number2,menuchoice);
            break;
        case 2:
           result=calcul(number1, number2,menuchoice);
            break;
        case 3:
            result=calcul(number1, number2,menuchoice);
            break;
        case 4:
            result=calcul(number1, number2,menuchoice);
            break;
        case 5:
            result=calcul(number1, number2,menuchoice);
            break;
 
    default:
        printf("Erreur, vous devez taper un chiffre entre 1 et 6");
        break;
    }
    printf ("le resultat du calcul est %d\n",result);
 }
    while(menuchoice!=6);
 
}
/*-----------------------------------------------*/
 
 
void menu(void)
{
    printf("Menu principal :\n\n");
    printf("1. Calculer une somme\n");
    printf("2. Calculer une difference\n");
    printf("3. Calculer un produit\n");
    printf("4. Calculer un quotient\n");
    printf("5. Calculer un modulo\n");
    printf("6. Quitter\n");
}
/*-----------------------------------------------*/
 
 
int calcul(int nb1,int nb2,int typecalcul)
{
    if(typecalcul==1)//somme
        return nb1+nb2;
    else if(typecalcul==2)//diff
        return nb1-nb2;
    else if(typecalcul==3)//produit
        return nb1*nb2;
    else if(typecalcul==4)//quotien
        return nb1/nb2;
    else if(typecalcul==5)//reste
        return nb1%nb2;
    else
        return -1;
}

[foetus]

, python style

Code :

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

1
2
3
    if (menuchoice==6)
        printf(" A bientot\n\n");
        return 0 ;

En théorie ton compilateur doit te dire qu'il y a du code inatteignable

[ternel]

En effet.

Avec un switch, tu pourrais ecrire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
enum calcul_type {calc_addition=1, calc_soustraction=1, calc_produit=3, calc_quotient=4, calc_reste=5};
 
int calculer(int a, int b, enum calcul_type calcul) {
    switch (calcul) {
        case calc_addition:      return a + b;
        case calc_soustraction: return a - b;
        case calc_produit:      return a * b;
        case calc_quotient:     return a / b;
        case calc_reste:        return a % b;
        default: return 0;
    }
}

Cela dit, pour une fois, je pense qu'il serait encore plus prudent de ne pas faire une fonction, et d'inclure ce switch directement dans le main.
ca éviterait d'avoir à gérer le non respect du type de calcul.

[bosk1000]

, python style

Code :

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

1
2
3
    if (menuchoice==6)
        printf(" A bientot\n\n");
        return 0 ;

En théorie ton compilateur doit te dire qu'il y a du code inatteignable

mon compilateur (qtcraetor) ne me dit rien, il lance le programme
si je fait un autre choix que le 6 il me ramene a :
"appuyer sur entrer pour fermer"

ceci dit ton encadré ma allumé une lumière

Code :

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

1
2
3
4
5
6
 
 if (menuchoice==6)
    {
        printf(" A bientot\n\n");
        return 0 ;
}

ça fonctionne mieux ainsi

en fait la question que je me pose c'est pourquoi ça avait fonctionné avant....
enfin solution trouvé
Merci

[bosk1000]

En effet.

Avec un switch, tu pourrais ecrire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
enum calcul_type {calc_addition=1, calc_soustraction=1, calc_produit=3, calc_quotient=4, calc_reste=5};
 
int calculer(int a, int b, enum calcul_type calcul) {
    switch (calcul) {
        case calc_addition:      return a + b;
        case calc_soustraction: return a - b;
        case calc_produit:      return a * b;
        case calc_quotient:     return a / b;
        case calc_reste:        return a % b;
        default: return 0;
    }
}

Cela dit, pour une fois, je pense qu'il serait encore plus prudent de ne pas faire une fonction, et d'inclure ce switch directement dans le main.
ca éviterait d'avoir à gérer le non respect du type de calcul.

Merci
je vais regarder cela
cette exemple me permet de voir une autre manière de coder et de prendre de la reflexion

[ternel]

Personnellement, le compilateur me dirait plutot ceci:

gcc -Wall -Wextra truc.c

truc.c: In function ‘calc’:
truc.c:37: attention : ‘return’ with a value, in function returning void

[bosk1000]

par contre en terme de gestion d'erreur je cherche quel bon message mettre si je tape 0 ou un chiffre supérieur à 7 dans le choix du menu

j'ai mis ceci :

Code :

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

1
2
3
4
5
6
 
    if (menuchoice==0 ||menuchoice>6 )
    {
        printf(" Erreur, vous devez taper un chiffre entre 1 et 6\n\n");
         menu();
    }

et j’obtiens un truc pas terrible car bien qu'il redémarre le menu, il m'indique directement d'entrer un premier nombre et non pas "votre choix"

j'ai bien vue que mon idée de revenir sur menu() est fausse, du coup je ne vois pas comment rebouclé sur choisir le menu

[Sve@r]

j'ai bien vue que mon idée de revenir sur menu() est fausse, du coup je ne vois pas comment rebouclé sur choisir le menu

Bonjour
Il te suffit de faire la gestion de la saisie et de ses erreurs éventuelles dans le menu lui-même. Ainsi quand la fonction se termine, tu es certain que ce qu'elle te renvoie (ben oui, tant qu'à faire autant lui faire renvoyer le choix de l'utilisateur) est une valeur correcte et que tu n'as plus à te préoccuper que de la gérer...

Code c :

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
unsigned short menu(void)
{
    unsigned short n;
    char rep[100];
 
    while (1)
    {
        fputs("Menu principal :\n\n" stdout);
        fputs("1. Calculer une somme\n" stdout);
        fputs("2. Calculer une différence\n" stdout);
        fputs("3. Calculer un produit\n" stdout);
        fputs("4. Calculer un quotient\n" stdout);
        fputs("5. Calculer un modulo\n" stdout);
        fputs("0. Quitter\n", stdout);
        fputs("Votre choix :", stdout);
        fgets(rep, 100, stdin);
        if (sscanf(rep, "%hu", &n) == 1 && rep >= 0 && rep <= 5) break; 
        fputs("Choix incorrect - Recommencez !!!\n", stdout);
    }
    return rep;
}

[bosk1000]

Merci
Je dois du coup changer un peu mon schema de départ car je cherchais juste à améliorer un résultat d'un corrig" d'exo

je vais me pencher sur ce que tu viens de me mettre

[Sve@r]

Quand tu découpes ton travail en fonctions, il vaut mieux essayer d'avoir des fonctions qui font un travail complet. C'était dommage, dans ton code, d'afficher le menu d'un coté puis gérer ses erreurs d'un autre...

[bosk1000]

Je suis parti d'un exercice trouvé sur le net
j'avais vue une vidéo expliquant les pointeurs suite à nos précédents échange

qui donne une bonne introduction aux pointeurs

par la suite j'ai donc suivi l'exercice sur la mini calculatrice et je regardais pour quelques modification pour m’améliorer...