Discussion :

[dabasarr]

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
#include<stdio.h>
#include<conio.h>
void main ()
{
int n=3,cpt, m=2,note[2], t[3],moy[3],i,j,s,temp;
clrscr();
printf("Donner les codes etudiant et les notes\n");
for(i=0;i<n;i++)
{
printf("\ncode %d: ",i+1);
scanf("%d",&t[i]);
j=0; s=0;
do
{
printf("dommerla %d ieme note :",i);
scanf("%d",&note[j]);
s=s+note[j];
j++;
}
while(j<m);
cpt=j;
moy[i]=(s/cpt);
printf("\nLa moyenne de %d est %d: ",t[i],moy[i]);
printf("\n");
}
for(i=0;i<(n-1);i++)
{
for(j=1;j<n;j++)
{
if(moy[i]<moy[j])
{
temp=moy[i];
moy[i]=moy[j];
moy[j]=temp;
}
}
}
for(i=0;i<n;i++)
{
printf("\nletudiant %d: \n",t[i]);
printf("de est %d avec une moyenne de %d",i+1,moy[i]);
}
getch();
}

[_SamSoft_]

Bonjour, je sais qu'on est sur un forum C de débutant ainsi, je me dois de corriger ce code (avec mes maigres connaissances...)

void est à supprimer. Ce n'est pas standard. On fait :

Code :

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

1
2
3
4
5
6
 
int main(void)
{
/* mon code */
return 0;
}

Tes tableaux ne sont pas initialisés :

on fait :

Code :

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

1
2
3
4
5
 
for(i = 0 ; i < nombre_max ; i++)
{
note[i] = 0;
}

Code :

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

1
2
3
4
5
6
7
8
 
do
{
printf("dommerla %d ieme note :",i);
scanf("%d",&note[j]);
s=s+note[j];
j++;
}

scanf est fortement déconseillé, tu ne sais pas réellement ce que l'utilisateur a entré, on utilisera fgets puis fclean (pour supprimer le '\n') et enfin une fonction qui converti une chaine de caractère en un entier.

conio c'est pas super. PDCurses fait la même chose en standard (je me trompe ?) Et puis pour clrscr et getch utiliser conio c'est comme sortir le bazooka pour écraser une araignée. On peut faire un appel système (je sais c'est affreux mais bon, moi j'ai déjà dit que PDCurses était une solution).

Voilà

Bonne prog !

[Emmanuel Delahaye]

on utilisera fgets puis fclean (pour supprimer le '\n') et enfin une fonction qui converti une chaine de caractère en un entier.

C'est bien de citer fclean(), mais il faut le faire correctement, car ce n'est pas une fonction standard. Il faut le préciser, en indiquant qu'on pourra en trouver le code sur le forum ou ici :

http://delahaye.emmanuel.free.fr/clib/ed/inc/fclean.h
http://delahaye.emmanuel.free.fr/clib/ed/src/fclean.c

[Thierry Chappuis]

conio c'est pas super. PDCurses fait la même chose en standard (je me trompe ?)

Rien de standard ici. PDCurses est simplement beaucoup plus portable que conio.

Thierry

[Thierry Chappuis]

Voici une proposition de correction:

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
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
#include <stdio.h>
#include <stdlib.h>
 
/* -tc- on n'a problement pas besoin d'utiliser conio juste pour effacer l'ecran.
   Voici une autre solution qui a l'avantage d'etre pseudo-portable et legere: */
 
#if defined (WIN32)
#   define CLEAR "cls"
#elif defined (__linux)
#   define CLEAR "clear"
#else
#   error "Not defined for this platform!"
#endif
 
#define CLS() system(CLEAR)
 
#define ETUDIANTS_NB 3
#define NOTES_NB 2
 
/* -tc- Je trouve preferable de representer un etudiant a l'aide d'une structure. */
struct Etudiant
{
    int id;
    double notes[NOTES_NB];
    double somme;
    double moyenne;
};
 
/* -tc- renvoit une valeur negative si e1 a une meilleure moyenne que e2,
   une valeur positive si  e2 a une meilleure moyenne que e1, et une
   0 sinon. */
static int
etudiants_cmp(void const *e1, void const *e2)
{
    int ret = 0;
 
    if (e1 != NULL && e2 != NULL)
    {
        struct Etudiant const *p_etud1 = e1;
        struct Etudiant const *p_etud2 = e2;
        ret = (int)(p_etud2->moyenne * 10) - (int)(p_etud1->moyenne * 10);
    }
 
    return ret;
}
 
static void
fpurge(FILE *fp)
{
    if (fp != NULL)
    {
        int c;
 
        while ((c = fgetc(fp)) != '\n' && c != EOF)
        {
        }
    }
}
 
/* -tc- main() retourne un entier de type int, TOUJOURS! */
int main(void)
{
    /* -tc- C'est une preference personnelle, mais je ne definis que une
       variable par ligne. Beaucoup de variables sont rendues inutiles par
       l'utilisations des structures */
    struct Etudiant etudiants[ETUDIANTS_NB] =
    {
        {
            0, {0.0}, 0.0, 0.0
        }
    };
    int i;
    int j;
    int rv;
 
    printf("Donner les codes etudiant et les notes\n");
    /* -tc- ATTENTION: le C est sensible a la casse. Ainsi, FOR n'est pas
       equivalent a for.*/
    for (i = 0; i < ETUDIANTS_NB; i++)
    {
        CLS();
 
        /* -tc- saisie +/- securisee d'un entier */
        do
        {
            printf("code %d: ", i + 1);
            /* -tc- Pour etre portable, il est necessaire ici de forcer
               l'affichage */
            fflush(stdout);
            /* -tc- Toujours verifier la valeur retournee par scanf() et purger le
               tampon du flux d'entree standard */
            rv = scanf("%d", &etudiants[i].id);
            fpurge(stdin);
        }
        while (rv != 1);
 
        for (j = 0; j < NOTES_NB; j++)
        {
            do
            {
                printf("donner la %d%s note: ", j + 1, (j > 0) ? "eme":"ere");
                fflush(stdout);
                rv = scanf("%lf",&etudiants[i].notes[j]);
                fpurge(stdin);
            }
            while (rv != 1);
            etudiants[i].somme += etudiants[i].notes[j];
        }
 
        etudiants[i].moyenne = etudiants[i].somme / NOTES_NB;
    }
 
    /* -tc- Ici, j'utilise la fonction standard qsort() pour effectuer le tri */
    qsort(etudiants, ETUDIANTS_NB, sizeof *etudiants, etudiants_cmp);
 
    for (i = 0; i < ETUDIANTS_NB; i++)
    {
        printf("Etudiant %d est %d%s avec une moyenne de %f\n",
               etudiants[i].id, i + 1, (i>0)?"eme":"er", etudiants[i].moyenne);
    }
    return 0;
}

Thierry