Discussion :

[timeis]

Bonsoir, je suis entrain de faire un programme contenant une structure de donnée contenant le nom, prenom, age et note des N etudiants d'une classe. Il doit etre capable de calculer la moyenne de la classe, de classer les eleves par age, et d'enregistrer les données ds un fichier.
Voici mon programme :

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
#include <stdio.h>
 
struct etudiant {
		char nom [32] ;
		char prenom [32] ;
		int age ;
		float note ;
 
 
	};
 
	// fonction moyenne
 
 
	float moyenne(int N, struct etudiant Moyennes){
	float moyenne;
	int i;
 
 
	struct etudiant moy[N];
	moy[N]=Moyennes ;
 
 
	for(i=0;i<N;i++) {
		moyenne=moyenne + moy[i].note ;
	}
moyenne=moyenne/N;
return moyenne;
}
 
int main () {
 
	float moyenne;
	int N, i, j, indMin, age;
	struct etudiant aide[1], aide2[1];
	// enregistrement
 
	FILE *myfile;
	myfile=fopen("etu.txt", "w");
 
 
	fprintf(myfile, "saisir le nombre d'etudiant N\n");
	scanf("%d", &N) ;
	struct etudiant tab[N] ;
 
	//remplissage du tableau des N etudiants
 
	for(i=0;i<N;i++) {
		fprintf(myfile, "saisir nom, prenom, age, note de l'etudiant numero [%d]\n", i) ;
		scanf("%s%s%d%f", &tab[i].nom, &tab[i].prenom, &tab[i].age, &tab[i].note) ;
	}
	for(i=0;i<N;i++) {
		fprintf(myfile, "%s, %s, %d, %f\n", tab[i].nom, tab[i].prenom, tab[i].age, tab[i].note) ;
			}
	fclose(myfile);
 
 
	moyenne = moyenne(N,tab);
 
	printf("la moyenne de la classe est %f", moyenne) ;
 
 
 
 
			// tri par selection
 
			for (i = 0; i < N; i++){
				aide[0] = tab[i];
				for (j= 0; j + 1 < N; j++){
          				if (tab[j].age < aide[0].age) and (tab[j+1] > aide[0]){
						aide2[0] = tab[j];
						tab[j] = aide[0];
						aide[0] = aide2[0];}
				}
				tab[i] = aide[0];
       }
 
 
 /* Affichage après le tri : */
 printf("\nContenu apres le tri :\n ");
 for (j = 0; j <N ; j++)
    printf("%d", tab[j].age);
 printf("\n");
 
 
	return 0;
}

Un probleme m'est signalé à la ligne :
moyenne = moyenne(N,tab);
j'ai quand meme essayé d'enlever tout ce qui avait un rapport avec la moyenne, et d'executer le programme, mais rien ne se passe :s
J'aurais donc voulu savoir si quelqu'un sait à quoi est du cela.
Je tiens à préciser que c'est mon premier programme, je ne m'y connais donc pas beaucoup.

Par avance merci.

[souviron34]

d'abord reprend tes cours sur les pointeurs, et les passages de parametres en C...

Apres on en reparlera...

[Thierry Chappuis]

Voici ce que me dit mon compilateur:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
Compiling: main.c
/home/thierry/devel/sdz/main.c:35: attention : function declaration isn"t a prototype
/home/thierry/devel/sdz/main.c: In function «main":
/home/thierry/devel/sdz/main.c:55: attention : format «%s" expects type «char *", but argument 2 has type «char (*)[32]"
/home/thierry/devel/sdz/main.c:55: attention : format «%s" expects type «char *", but argument 3 has type «char (*)[32]"
/home/thierry/devel/sdz/main.c:64: erreur: called object «moyenne" is not a function
/home/thierry/devel/sdz/main.c:78: attention : implicit declaration of function «and"
/home/thierry/devel/sdz/main.c:78: erreur: opérandes invalides pour le binaire >
/home/thierry/devel/sdz/main.c:79: erreur: expected «;" before «{" token
/home/thierry/devel/sdz/main.c:39: attention : unused variable «aide2"
/home/thierry/devel/sdz/main.c:38: attention : unused variable «age"
/home/thierry/devel/sdz/main.c:38: attention : unused variable «indMin"
Process terminated with status 1 (0 minutes, 0 seconds)
10 errors, 0 warnings

En survolant, je vois:

• moyenne est à la fois une variable locale à main() et une fonction (que tu essaies d'appeler dans main). Tu utilises également une variable moyenne dans l'implantation de ta fonction moyenne. C'est pas terrible.
• aide1 et aide2 sont des tableaux de une structure etudiant. Pourquoi pas simplement une variable de type struct etudiant?
• Attention: tab est un tableau à longueur variable. Ce n'est certainement pas ce que tu veux. Pour définir un tableau traditionnel, N doit être une expression constante.
• Tu ne vérifie pas la valeur retournée par fopen().
• Tu passes un VLA en 2ème argument de ta fonction moyenne()

entre autres choses.

Thierry

[timeis]

Merci pour ces aides.
J'ai revu aujourd'hui tout mon programme
J'arrive maintenant a voir mon programme dans un fichier texte, mais à l'execution de la moyenne le programme plante... Je n'arrive pas à comprendre pourquoi, si quelqu'un pourrait m'eclairer... merci d'avance

voici mon code

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
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
145
146
147
148
149
150
151
152
153
154
155
#include <stdio.h>
 
struct etudiant {
        char nom [32] ;
        char prenom [32] ;
        int age ;
        float note ;
};
 
    // fonction moyenne
 
 
float calculMoyenne(int nbEtudiant, struct etudiant Moyennes){
	long total;
	total = 0;
 
	struct etudiant moy[nbEtudiant];
	moy[nbEtudiant] = Moyennes;
 
	for(int i=0;i<nbEtudiant;i++){
		total = total + moy[i].note;
	}
 
	float moyenne;
 
	moyenne = total / nbEtudiant;
 
	return moyenne;
}
 
int main () {
 
    //float moyenne;
    int N, i, j, k, indMin, age;
	float average;
	//float moyenne;
    //struct etudiant aide[1], aide2[1];
    // enregistrement
 
	//declaration de la fonction moyenne
	float moyenne (int N, struct etudiant tab);
 
 
    printf("saisir le nombre d'etudiant \n");
    scanf("%d", &N) ;
    struct etudiant tab[N] ;
 
    //remplissage du tableau des N etudiants
 
	FILE *myfile;
 
	myfile = fopen("etudiant.txt", "w+");
 
    for(i=0;i<N;i++) {
 
		//printf - sortie a l'ecran
		printf("Etudiant numero [%d]\n", i+1);
 
		printf("Saisir Nom: ", i);
		scanf("%s", &tab[i].nom);
 
		printf("Saisir Prenom: ", i);
		scanf("%s", &tab[i].prenom);
 
		printf("Saisir Age: ", i);
		scanf("%d", &tab[i].age);
 
		printf("Saisir Note de l'Etudiant: ", i);
		scanf("%f", &tab[i].note);
 
		//fprintf - ecriture dans le fichier
		fprintf(myfile,"%s, %s, %d, %f\n", tab[i].nom, tab[i].prenom, tab[i].age, tab[i].note) ;
	}
 
	printf("TEST TEST TEST \n");
 
 
    //printf("la moyenne de la classe est %f", moyenne);
 
 
	// tri par selection
 
	// creation d une struct temporaire pour trier les etudiant par age
	//struct etudiant temp[N];
	int temp[N];
	temp[0] = 0;
	//k = N;
 
	if(N>1){
		for(i=0;i<N-1;i++){
			//decale tous les nombre d une case vers le bas tant que le nb trie est inferieur a la 1er case temp
			//for(k=0;k<N-1;k++){
			if(tab[i].age >= temp[0]) {
				//while(tab[i].age >= temp[0]){
				for (k=N-1;k>0;k--)
					temp[k] = temp[k-1];
					//temp[N-1-i] = temp[N-2-i];
					temp[k] = tab[k].age;
					k++;
				//}
			}
			else{
				temp[0] = tab[i].age;
			}
 
			//}
			//temp[0] = tab[i].age;
		}
	}
 
	for(j=0;j<N;j++){
		printf ("%d\n", temp[j]);
	}
 
	fclose(myfile);
 
	average = calculMoyenne(N, tab[N]);
 
	printf ("valeur de la moyenne de la classe : %f\n", average);
 
	//
	//		if(tab[i].age <= tab[i+1].age){
	//			temp[i].age = tab[i].age;
	//		}
 
				// if(tab[i].age < temp[i].age){
 
	//			temp[i+1].age = tab[i]
	//
	//			temp[i+2] = temp[i+1]
	//			temp[i+1] = temp[i]
	//			temp[0] = tab[i]
//
//				temp[i+3] = temp[i+2]
//				temp[i+2] = temp[i+1]
//				temp[i+1] = temp[i]
//
//
//				for(j=1;j<N;i++) {
//					temp[N-j] = temp[N-1-j]
//				}
//
//				temp[0].age = tab[i].age;
//			}
//		}
 
 
 /* Affichage après le tri : */
 //printf("\nContenu apres le tri :\n ");
 //for (j = 0; j <N ; j++)
 //   printf("%d", tab[j].age);
 //printf("\n");
 
    return 0;
}

[Nykoo]

Code :

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

1
2
 
float moyenne (int N, struct etudiant tab);

Une fonction ne s'utilise pas comme ça. Ca c'est la forme d'un prototype de fonction. Quand tu veux utiliser une fonction il faut seulement passer le nom des variables en argument:

Code :

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

1
2
 
moyenne (N,tab);

Et tu n'as pas déclaré la structure tab avec de l'utiliser dans cette fonction.

[timeis]

Une fonction ne s'utilise pas comme ça. Ca c'est la forme d'un prototype de fonction. Quand tu veux utiliser une fonction il faut seulement passer le nom des variables en argument:

Code :

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

1
2
 
moyenne (N,tab);

Et tu n'as pas déclaré la structure tab avec de l'utiliser dans cette fonction.

j'ai fait ce que tu m'as dit, j'ai donc :

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
 
int main () {
 
    //float moyenne;
    int N, i, j, k, indMin, age;
	float average;
	//float moyenne;
    //struct etudiant aide[1], aide2[1];
    // enregistrement
 
	//declaration de la fonction moyenne
	struct etudiant tab[N];
	moyenne (N, etudiant tab);
 
 
    printf("saisir le nombre d'etudiant \n");
    scanf("%d", &N) ;
 
 
    //remplissage du tableau des N etudiants

mais jai plusieurs erreurs pour moyenne (N, etudiant tab); du style "illegal statement termination

[fremen167]

Si tu définis une fonction :

Code :

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

float moyenne (... )

Une instruction valide peut être :

Code :

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

mamoyenne = moyenne(...) ;

car

Code :

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

moyenne(...)

forme une expression mais pas une instruction (statement).