probleme avec C

Version imprimable

11/12/2007, 16h50
lea87

probleme avec C

salut,
j'ai un probléme avec un exercice que je fais en C. et voila un petit extrai de mon exercice:
(une compagnie aérien desire faire des reductions,Sur les vols de destination d'Asie, 50% pour l'age inférieur à 20ans ou -25ans et qui ont etudiants.....
je travail avec un menu ou bien un if imbriquée??
11/12/2007, 16h51
Gf6HqmTW

Avec n'importe qui qui t'aiderait à formuler precisement ton probleme ...
Que tu utilise un menu ou un if imbriqué ... On ne sait pas pour quoi tu en as besoins ... soit claire ...
11/12/2007, 16h52
blackhorus

Il faut aussi dire, que t'as pas trop de choix dans ton exercice (2 ou 3 cas, je crois)... Pas trop de différence, je pense à implémenter la solution avec IF ou un CASE
11/12/2007, 16h56
Gf6HqmTW

Je ne suis toujours pas sure de comprendre ton probleme mais je dirais Case ...
Une petite fonction qui transforme d'abord l'age en une categorie (<20 ans : cat=0 ; 20 a 25 : cat=1 etc ...) et tu peut faire un case sur le resultat ...
Sinon une methode avec un tableau peut etre avantageuse ... Les colonnes representent des tranches d'ages et les lignes des destinations ...
Il y a plusieurs methodes ...
11/12/2007, 17h05
lea87

oui merci , mais normalent dans le sours on a pas encor fait ni menu, ni tableau c'est çela qui me gene
11/12/2007, 17h12
Thierry Chappuis

Je ne vois aucun besoin de tableau ni de menu. Voici comme je vois les choses:

Tu demandes son âge à l'utilisateur, puis tu fais une saisie de la valeur entrée. Ensuite, tu calcules le montant des réductions à l'aide d'une structure conditionnelle (if ... else if ... else... ou switch ... case ...).

Poste ton code et pose des questions précises si tu rencontres des difficultés.

Thierry
11/12/2007, 17h12
blackhorus

Citation:

Envoyé par lea87

oui merci , mais normalent dans le sours on a pas encor fait ni menu, ni tableau c'est çela qui me gene

Si tu veux jouer dans la cours des grands, apprends-les dès maintenant, et frime à mort ! Sinon, suis le troupeau et fais l'exo avec des IF

:salut:
11/12/2007, 17h14
BainE

Citation:

Bonjour,

moi ce qui me gene c est qu un menu c'est graphique et qu un tableau c est une maniere d organiser, gerer la memoire, et un if d organiser et conditionner son algo.

Je ne comprends pas vraiment.
11/12/2007, 17h55
Emmanuel Delahaye

Citation:

Envoyé par lea87

oui merci , mais normalent dans le sours on a pas encor fait ni menu, ni tableau c'est çela qui me gene

T'écris avec les pieds ?
11/12/2007, 18h11
blackhorus

Citation:

Envoyé par Emmanuel Delahaye

T'écris avec les pieds ?

Toujours intraitable avec le langage SMS et compagnie :aie:

Citation:

Envoyé par GuJman

Je ne suis toujours pas sure de comprendre ton probleme mais je dirais Case ...
Une petite fonction qui transforme d'abord l'age en une categorie (<20 ans : cat=0 ; 20 a 25 : cat=1 etc ...) et tu peut faire un case sur le resultat ...

Attention, on n'est pas loin d'une bourde...

- Le switch .. case est très limité en C. Pas d'intervalle, juste un empilement de quelques case, et pas d'intevalle ouvert du tout. En fait, c'est parfait pour fonctionner conjointement à une enum, que la liste des cas porte ou non ce nom. En revanche, dans ce cas, ce sera toujours un peu plus performant que les suites if .. else if ..else. Et le temps ne dépendra peut-être pas du résultat du test, à cause de l'adressage paramétré vers les case et default (ce sont des label). Il y a contrôle de cohérence (pas deux fois le même case) par le compilateur.

- La séquence bien rigoureuse "if -- else if -- else if ...... else" est facile et sûre à écrire, et à lire. A condition d'indenter par exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 if condition1{ /* ... */ } else if condition2{ /* ... */ } else if condition3{ /* ... */ } else if condition4{ /* ... */ } else { /* ... */ }
et en espérant que l'éditeur ou l'outil de formatage respectera.
En revanche, même dans les mêmes conditions, avec les mêmes "cas", ce n'est pas le même algorithme que le switch .. case. Le traitement des conditions est séquentiel. Dans le cas où on tombe sur default / else final "presque toujours", ce sera peu performant. Mais ça permettra tous les intervalles et autres comparaisons évoluées, avec des % (modulo) par exemple.

A titre personnel, j'évite le switch .. case sauf de façon presque automatique lié à une enum, et encore.

Ah, la bourde: c'est de bien montrer les limitations du switch .. case en le faisant précéder par une "catégorisation" qui se fera à base de ... if -- else if ... :mouarf:

Citation:

Sinon une methode avec un tableau peut etre avantageuse ... Les colonnes representent des tranches d'ages et les lignes des destinations ...
Il y a plusieurs methodes ...

Ou un truc bien crade genre:

Code:

reduc = 5.0 * (age <= 20) + 5.0 * (age <=25) + 10 * (age <= 30);

20% jusqu'à 20ans, 15% entre 20 et 25 ans et 10% entre 25 et 30 ans

12/12/2007, 07h17
Gf6HqmTW

Perso je voyais bien ma fonction de discretisation codé à la bien crade histoire apres de faire un truc clair à lire pour le case...
Le but étant pour la demoiselle (léa87), qui fait un exercice, d'aborder le principe du case, sa diference d'avec les if et de voir un tas de methodes ...
Histoire de s'ouvrir l'esprit et pas de tomber dans un code immediatement lourd et imbitable parcequ'il est plus performant ou qu'il fait plus pro ^^

Et apres je l'avoue egalement et même si ca peut être compré à un handicap, je suis traumatisé du IF, tant que je peu les eviter je le fais ...
12/12/2007, 08h43
nolofinwe

bonjour,

d'accord avec toi GuJman... Un bon case je trouve ca plus propre... mais personellement je pense que j'associerai une variable a chaque tranche d'age. par exemple une variable qui prendrai la valeur 0 pour une tranche d'age , 1 pour une deuxieme , etc... il n'y aurai qu'a "switcher" cette variable...

a +
12/12/2007, 11h06
Gf6HqmTW

Du coup tu peut me dire l'interet du case ? Et meme ne serait-ce que comment tu l'utilise ?
Employer X variables ou discretiser en X valeures sur une seule variable il n'y a même pas de choix à faire ...

Ceci dit je pense que j'ai mal compris ta methodes (j'espere) car elle me semble vraiment trop absurde ...

c'est vrai que la méthode peut paraitre absurde mais je trouve qu'elle éclairci le code... mais bon ca fais que 3 mois que je fais du c alors je ne me considere pas comme expert :aie:

voila un exemple:
Code:

1 2 3 4 5 6 7 8 9 10 int t=2; if (age<20){t=0;} else if (age<25){t=1;} switch(t){ case 0 : //calcul de la reduc et traitement break; case 1 : //calcul de la reduc et traitement break; ...........
PS : n'hésites pas a la critiquer il y a que comme ca que j'apprendrai ;)

12/12/2007, 11h42
Gf6HqmTW

Oula oula oulaaaaaaaaa !
Apparement tu fais une confusion entre variable et valeur ... C'est comme si tu me parlais d'un verre en me disant que tu va prendre plusieurs verre et que soit tu mettra de l'eau dans l'un soit du jus d'orange dans le suivant etc ...

Quant au switch/case on parle de la même chose qu'on dise l'un ou l'autre ...

Et dans ton exemple tu fais la grosse bourde dont parlait Pierre Maurette tu fais un if pour definir une valeure que tu traite ensuite avec un case ... à ce compte autant faire ton traitmenet dans le IF ...

Au passage j'ai compris de quoi parlait Léa (il serait temps) quand elle parlait de menu ...
Si je ne m'abuse il s'agit au lieu de demander l'age de la personne sous forme de valeur entiere (ou réelle ... ce serait marrant le mec qui réponds 21.257983452 ... useless mais marrant) est invité par un "menu" à choisir parmis les differentes catégories d'age possible... Donnant une valeur justifiant de fait pleinement l'usage du case ...
12/12/2007, 11h50
nolofinwe

lol j'ai bien dit que ca pouvais avoir l'air absurde...

pour etre franc je trouve que réaliser le traitement a l'interieur d'un if imbriqué peut devenir vite dangereux... pour peu que le traitement en question soit un peu lourd le programme peut vite devenir illisible (surtout pour un neophyte comme moi :aie:) c'est pour ca que j'utilise cette méthode c'est surtout pour une histoire de clarté...
J'ai pas encore trouvé comment faire un code qui soit simple a relire avec ce genre de structure autrement que comme ca...

Si tu a quelque conseil a me donner ca serai sympa:mrgreen:
12/12/2007, 11h58
Gf6HqmTW

En indentant ton code, en utilisant des editeurs de textes bien ficelés (NotePad++ est tres bien il permet de developper ou reduire des portions de code ...)

Il y a pas de secret ...

L'emploi regulier de fonction/procedures clairement identifiées codées inline (si elles sont courtes) peut egalement aerer ton code ...

^^ ceci dit c'est en forgant des épées lasers qu'on devient Jedi !
12/12/2007, 13h12
nolofinwe

Citation:

Envoyé par GuJman

^^ ceci dit c'est en forgant des épées lasers qu'on devient Jedi !

je suis tout a fait d'accord avec toi... le probleme c'est que quand on forge seul on voit pas forcement les défaults des épées lasers...

C'est ca mon probleme tant qu'on ne me dis pas que c'est pas bien j'aurai du mal a m'en rendre compte (surtout a mon niveau)

c'est vrai que les fonctions sont une bonne alternative j'y penserai désormais... merci pour tous ces conseils:king:
12/12/2007, 13h15
Pierre Maurette

Citation:

Envoyé par GuJman

^^ ceci dit c'est en forgant des épées lasers qu'on devient Jedi !

Et c'est en sciant que Léonard devint scie...
12/12/2007, 13h24
Gf6HqmTW

Elle est fine elle est fine ^^
D'ailleurs est-ce en jouant du tuba qu'on devient fanfaron ?
12/12/2007, 13h59
lea87

1 pièce(s) jointe(s)

mon code

mater et dites moi ,,,
12/12/2007, 14h06
Gf6HqmTW

Citation:

Envoyé par lea87

mater et dites moi ,,,
12/12/2007, 14h06
nolofinwe

salut,

heu ca m'a l'air pas mal mais deux choses me chiffonent :

1- tu as oublié d'ouvrir l'acolade de ton deuxieme if ton imbrication ne fonctionnera donc pas correctement.
2- 'mois' n'est jamais initialisé.

voila mais sinon la structure ne me choque pas particulierement...

a +

EDIT ==> sorry j'ai mal lu tu n'a pas besoin d'acolade vu que tu n'execute qu'une instruction dans ton if :aie:
12/12/2007, 14h13
Gf6HqmTW

Si, il y a plus d'une chose qui choc ...
Bon passons sur Asi qui s'écrit avec un E
Sinon tu ne demandes l'age que dans le cas ou ton utilisateur va en Asie donc dans les autres cas ta variable "age" ne sera pas initialisée ...
//if(strcmp(pays,"asie"){ printf("%s",&pays); n'a pas de sens :koi:
Le "mois" n'est initialisé nul part ?
A quoi sert-il ?
Il te manque des accolades un peu partout qui, même si elles ne sont pas obligatoires pour une instruction seule, rendent toujours le code plus clair !

Et penses aux balises CODE (c'est le # dans ton editeur de messages)

Voici quelques corrections et mes commentaires:

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
#include <stdio.h>
/* -tc- pas standard et inutile
#include <conio.h>
*/
 
/* -tc- permet de faire le menage de maniere portable sur le flux
   d'entree standard */
void purger(void)
{
    int c;
 
    while ((c = fgetc(stdin)) != '\n' && c != EOF)
    {
    }
}
 
/* -tc- main() retourne un entier de type int, toujours */
int main(void)
{
    int choix;
    int age;
    int status;
    /* non utilisee
    int mois;
    */
    int rv;
 
    /* -tc- on peut securiser la saisie de cette maniere */
    do
    {
        printf("choisir le pays de destination du client:\n");
        printf("1-[asie]\n2-[afrique]\n3-[amerique du sud]\n");
        rv = scanf("%d", &choix);
        purger();
    }
    while (rv != 1 || choix < 1 || choix > 3);
 
    switch (choix)
    {
    case 1:
        printf("Asie\n");
        break;
    case 2:
        printf("Afrique\n");
        break;
    case 3:
        printf("Amerique du sud\n");
        break;
    }
 
    /* if(strcmp(pays,"asie"){ printf("%s",&pays); */
    if (choix == 1)
    {
        /* -tc- on se protege contre les petits malins qui auraient la bonne
           idee d'entrer des caracteres autres que des chiffres */
        do
        {
            printf("entrez l'age du client: ");
            fflush(stdout);
            rv = scanf("%d",&age);
            purger();
        }
        while (rv != 1);
 
        if (age < 20)
        {
            /* -tc- pour afficher le caractere '%', on utilise %% dans la
               chaine de format*/
            printf("reduction de 50%%\n");
        }
 
        do
        {
            printf("entrez le statut du client:\n");
            printf("1-[etudiant]\n2-[autre]\n");
            rv = scanf("%d", &status);
            purger();
        }
        while (rv != 1 || status < 1 || status > 2);
 
        if (status == 1)
        {
            if (age < 25 )
            {
                printf("reduction de 50%%\n");
            }
            else if (age < 30)
            {
                printf("reduction de 30%%\n");
            }
        }
    }
    else if ((choix == 2) && (age < 30))
    {
        printf("reduction de 30%%\n");
    }
    /* -tc- Attention: mois n'a pas ete initialise et 6 <mois < 10 n'est une
       syntaxe valide en C
    else if ((age < 23) && (6<mois<10))
    */
    /* -tc- ??? condition au hasard - je ne comprends pas vraiment ce que tu
       veux ici...*/
    else if (choix == 3 && age < 23)
    {
        printf("reduction de 30%%\n");
    }
 
    /* -tc- pas standard et inutile
    getch();
    */
 
    /* -tc- 0, EXIT_SUCCESS et EXIT_FAILURE sont les 3 valeurs portables que
        peut retourner main() */
    return 0;
}

Thierry

Citation:

Envoyé par Thierry Chappuis

Voici quelques corrections et mes commentaires:

Je m'autorise l'ajout d'une enum, qui à mon avis apporte beaucoup en lisibilité et maintenabilité. J'ai vu sur un autre fil de ce forum un bout de code avec la boucle de message de Windows, exemple de l'utilisation conjointe d'un switch .. case et d'une enum (ou de #define).

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
#include <stdio.h>
/* -tc- pas standard et inutile
#include <conio.h>
*/
 
/* -tc- permet de faire le menage de maniere portable sur le flux
   d'entree standard */
void purger(void)
{
    int c;
 
    while ((c = fgetc(stdin)) != '\n' && c != EOF)
    {
    }
}
 
enum choix_destination {
    choix_asie
  , choix_afrique
  , choix_amerique_sud
};
 
const char* DESTINATIONS[] = {
                               "asie"
                             , "afrique"
                             , "amerique du sud"
};
 
/* -tc- main() retourne un entier de type int, toujours */
int main(void)
{
    enum choix_destination destination_choisie;
    int choix;
    int age;
    int status;
    /* non utilisee
    int mois;
    */
    int rv;
    int i;
 
    /* -tc- on peut securiser la saisie de cette maniere */
    do
    {
        printf("choisir le pays de destination du client:\n");
        for(i = 0; i < sizeof DESTINATIONS / sizeof *DESTINATIONS; i++)
        {
            printf("%d-[%s]\n", i + 1, DESTINATIONS[i]);
        }
        rv = scanf("%d", &choix);
        purger();
    }
    while (rv != 1 || choix < 1 || choix > 3);
 
    destination_choisie = choix - 1;
 
    switch (destination_choisie)
    {
    case choix_asie:
        printf("Asie\n");
        break;
    case choix_afrique:
        printf("Afrique\n");
        break;
    case choix_amerique_sud:
        printf("Amerique du sud\n");
        break;
    }
 
    /* if(strcmp(pays,"asie"){ printf("%s",&pays); */
    if (destination_choisie == choix_asie)
    {
        /* -tc- on se protege contre les petits malins qui auraient la bonne
           idee d'entrer des caracteres autres que des chiffres */
        do
        {
            printf("entrez l'age du client: ");
            fflush(stdout);
            rv = scanf("%d",&age);
            purger();
        }
        while (rv != 1);
 
        if (age < 20)
        {
            /* -tc- pour afficher le caractere '%', on utilise %% dans la
               chaine de format*/
            printf("reduction de 50%%\n");
        }
 
        do
        {
            printf("entrez le statut du client:\n");
            printf("1-[etudiant]\n2-[autre]\n");
            rv = scanf("%d", &status);
            purger();
        }
        while (rv != 1 || status < 1 || status > 2);
 
        if (status == 1)
        {
            if (age < 25 )
            {
                printf("reduction de 50%%\n");
            }
            else if (age < 30)
            {
                printf("reduction de 30%%\n");
            }
        }
    }
    else if ((destination_choisie == choix_afrique) && (age < 30))
    {
        printf("reduction de 30%%\n");
    }
    /* -tc- Attention: mois n'a pas ete initialise et 6 <mois < 10 n'est une
       syntaxe valide en C
    else if ((age < 23) && (6<mois<10))
    */
    /* -tc- ??? condition au hasard - je ne comprends pas vraiment ce que tu
       veux ici...*/
    else if (destination_choisie == choix_amerique_sud && age < 23)
    {
        printf("reduction de 30%%\n");
    }
 
    /* -tc- pas standard et inutile
    getch();
    */
 
    /* -tc- 0, EXIT_SUCCESS et EXIT_FAILURE sont les 3 valeurs portables que
        peut retourner main() */
    return 0;
}

Dans ce cas, autant aller jusqu'au bout. On peut éliminer:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
switch (destination_choisie)
    {
    case choix_asie:
        printf("Asie\n");
        break;
    case choix_afrique:
        printf("Afrique\n");
        break;
    case choix_amerique_sud:
        printf("Amerique du sud\n");
        break;
    }

au profit de:

Code:

printf("%s\n", DESTINATIONS[destination_choisie]);

Thierry

Citation:
Envoyé par Thierry Chappuis
Dans ce cas, autant aller jusqu'au bout. On peut éliminer:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 switch (destination_choisie) { case choix_asie: printf("Asie\n"); break; case choix_afrique: printf("Afrique\n"); break; case choix_amerique_sud: printf("Amerique du sud\n"); break; }
au profit de:

Code:

printf("%s\n", DESTINATIONS[destination_choisie]);

Thierry
J'attendais cette "information", mais plutôt d'un autre que vous. Je m'étais demandé s'il fallait préciser que le code dans les case était factice. Alors je précise également que le status reste à traiter.

12/12/2007, 15h47
Thierry Chappuis

Citation:

Envoyé par Pierre Maurette

J'attendais cette "information", mais plutôt d'un autre que vous.

Oups, pardon :oops:. Je n'avais pas compris l'intention pédagogique.

Thierry