améliorer mon programme

Version imprimable

Bonjour

je developpe un prog de conversion francs->euros et euros->francs il fonctionne mais je pense que mon code est sale.
Ce qui me derange c'est l'imbrication des 3 switchs : le but c'est que si lors de l'affichage du menu principale l'utilisateur se trompe trois fois de suite, le programme quitte automatiquement.

voici le 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
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
#include<stdio.h>
#include<windows.h>
 
void calcul()
{
    double a = 0;
    double b =0;
    char ta = 0;
    int z = 0;
 
    printf("entrez votre valeur en franc\n");
    scanf("%lf",&a);
    b=a/6.55957;
    printf("%0g F vaut %lf euro(s)\n",a,b);
    printf("saisir c pour continuer\n");
    scanf("%s",&ta);
 
    while(ta!='c')
    {
        z++;
        printf("erreur de saisie\n");
        printf("saisir c pour continuer\n");
        scanf("%s",&ta);
 
        if(z==2)
        {
            ta='c';
            printf("vous aller etre rediriger vers le menu principal");
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
        }
    }
    main();
}
 
void calcul2()
{
    double a = 0;
    double b =0;
    char ta = 0;
    int z=0;
 
    printf("entrez votre valeur en euro\n");
    scanf("%lf",&a);
    b=a*6.55957;
    printf("%0g euro(s) vaut %lf F\n",a,b);
    printf("saisir c pour continuer\n");
    scanf("%s",&ta);
 
    while(ta!='c')
    {
        z++;
        printf("erreur de saisie\n");
        printf("saisir c pour continuer\n");
        scanf("%s",&ta);
 
        if(z==2)
        {
            ta='c';
            printf("vous aller etre rediriger vers le menu principal");
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
            printf(".");
            Sleep(1000);
        }
    }
    main();
}
 
int main()
{
    char to=0;
    int y= 0;
 
    system("cls");
    printf("Bienvenue dans le programme conversion monnaitaire\n");
    printf("conversion franc->euro saisir : f\n");
    printf("conversion euro->franc saisir : e\n");
    printf("quitter le programme          : q\n");
    scanf("%s", &to);
 
    switch(to)
    {
        case 'f':calcul();
        break;
        case 'e':calcul2();
        break;
        case 'q':
        case 'Q':return 0;
        default :printf("erreur de saisie\n");
                 scanf("%s", &to);
                 switch(to)
                    {
                        case 'f':calcul();
                        break;
                        case 'e':calcul2();
                        break;
                        case 'q':
                        case 'Q':return 0;
                        default :printf("erreur de saisie\n");
                                 scanf("%s", &to);
                                 switch(to)
                                 {
                                    case 'f':calcul();
                                    break;
                                    case 'e':calcul2();
                                    break;
                                    case 'q':
                                    case 'Q':return 0;
                                    default :printf("trop erreur de saisie\n");
                                             printf("le programme va quitter automatiquement");
                                             printf(".");
                                             Sleep(1000);
                                             printf(".");
                                             Sleep(1000);
                                             printf(".");
                                             Sleep(1000);
                                             printf(".");
                                             Sleep(1000);
                                             printf(".");
                                             Sleep(1000);
                                             return 0;
                                   break;
                                 }
                       break;
                }
        break;
    }
    return 0;
}

auriez vous une autre alternative a cette imbrication.

En outre si vous avez des idees pour améliorer le code a d'autre niveau n'hésitez pas et si vous avez des idees d'ajoue d'autre fonction nhésitez pas non plus.

merci

02/10/2010, 14h48
vivitron

existe t-il une fonction qui me permette de retourner a une ligne de mon code .

merci
02/10/2010, 16h35
jeroman

En effet. :cry:
Utilise plutôt une boucle, c'est bien plus adapté.
02/10/2010, 19h07
vivitron

justement, je n'arrive pas a la faire cette boucle, l'illumination ne me parviens pas.
02/10/2010, 19h10
Jannus

Un peu de lecture me semble indispensable :fleche: http://c.developpez.com/cours/bernar...gne/node33.php
02/10/2010, 20h40
droggo

Hia,

Citation:

Envoyé par vivitron

justement, je n'arrive pas a la faire cette boucle, l'illumination ne me parviens pas.

Programmer, ce n'est pas attendre l'illumination, mais travailler.

Comme ton problème est basique (eh oui, il faut bien commencer un jour ou l'autre), les cours/bouquins/tutos ne manquent pas. :)
03/10/2010, 13h01
vivitron

La boucle que vous me dite d'utiliser doit-elle remplacer la fonction switch ? ou est-elle à placer apres le default du switch.

j'ai fais des tests de boucle mais rien y fait je ne trouve pas la solution, je suis:cry:
03/10/2010, 13h24
jeroman
Une boucle englobe toute une partie de code qui doit être répétée en fonction de certaines conditions (une ou plusieurs).
"while" exécute le code >=0 fois.
"do/while" exécute le code >=1 fois.

Dans ton cas, ce sera une boucle do/while car l'affichage et la saisie doivent se faire au moins une fois.
Code:

1 2 3 4 5 6 7 8 9 10 11 DEBUT BOUCLE * affichage * saisie FIN BOUCLE (on sort si la/les conditions est/sont respectées, ie : si on a bien saisi f/e/q ou Q) SWITCH case xxx: ... case yyy: ... case ... FIN SWITCH
Après, si tu veux que l'utilisateur puisse faire plusieurs conversions fr/€ ou €/fr sans relancer le programme à chaque fois, il faudra englober toute cette partie dans une autre boucle (do/while) et prévoir une condition pour sa sortie, soit par une valeur "magique" (-1 ou autre) ou par un autre moyen.

Une chose aussi : supprime absolument ces "main();" qu'on trouve un peu partout dans tes fonctions. :calim2: