Discussion :

[Freyrlin]

Bonjour à tous,

je débute complètement en C et je bute depuis des jours sur une erreur qui est sûrement toute bête mais je ne sais plus trop quoi faire. Comme je suis super original je tente de faire un programme pour calculer mes moyennes. Pour l'instant mon programme doit seulement demander le nombre de notes ( ça marche ). Il doit demander ensuite les notes et les afficher et en faire la somme au fur et à mesure. Mais voilà, je n'arrive pas à comprendre pourquoi les 2 résultats affichés sont toujours 0.

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
#include <stdio.h>
#include <stdlib.h>
 
int main(void ) {
	float *liste_notes=NULL,somme = 0.0;
	int nb_de_notes = 0, i;
	printf("Nombre de notes\n");
	scanf("%d", &nb_de_notes);
	liste_notes=malloc(nb_de_notes*sizeof(float));
 
	if(liste_notes == NULL )
		exit(1);
 
	for (i=0; i < nb_de_notes; i++)
{
		printf("Note? ");
		scanf("%f",&liste_notes[i]);
		somme = somme + liste_notes[i];
		printf("[%f]\n",&liste_notes[i]);
		printf("%f\n",&somme);
}
free(liste_notes);
}

[Bousk]

Salut,

Code :

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

1
2
printf("[%f]\n", &liste_notes[i]);
printf("%f\n", &somme);

Tu veux afficher un flottant mais passe un pointeur. Retire les & et tu devrais voir apparaître tes valeurs

[exe2bin]

J'ajoute que la réservation d'un vecteur de nb_de_notes de pointeurs sur des flottants ne m'apparaît pas necessaire .
Pourquoi réserver de l'espace mémoire pour une donnée qui ,une fois utilisée ,disparaît aussitôt ?!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
        float note,somme = 0.0;
 
 
	for (i=0; i < nb_de_notes; i++)
{
		printf("Note? ");
		scanf("%f",&note);
		somme = somme + note;
		printf("[%f]\n",&note);
		printf("%f\n",&somme);
}

me semble amplement suffisant.

[Freyrlin]

Thank you Bousk, it worked !

@exe2bin: elle ne disparaissait que parce que mon programme n'était pas encore fini. Le voici presque fini:

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
#include <stdio.h>
#include <stdlib.h>
 
int main(void ) {
	float *liste_notes=NULL, somme = 0.0, moyenne=0.0;
	int nb_de_notes = 0, i;
	printf("Nombre de notes\n");
	scanf("%d", &nb_de_notes);
	liste_notes=malloc(nb_de_notes*sizeof(float));
 
	if(liste_notes == NULL )
		exit(1);
 
	for (i=0; i < nb_de_notes; i++)
{		
		printf("Note? ");
		scanf("%f",&liste_notes[i]);
		if (liste_notes[i] > 20) 
			printf("Erreur, note non prise en compte\n");
		else if (liste_notes[i] < 0) 
			printf("Erreur, note non prise en compte\n");
		else 
     		somme = somme + liste_notes[i];
}
 
	moyenne = somme / nb_de_notes;
	printf("moyenne:""%f\n",moyenne);
	for (i=0; i < nb_de_notes; i++)
	{printf("%f\n",liste_notes[i]);}
 
free(liste_notes);
 
}

[wolinn]

Si des notes ne sont pas prises en compte, le calcul de moyenne ne sera pas correct.
Il faut diviser la somme par le nombre de notes prises en compte, qui peut être inférieur à nb_de_notes.

[Freyrlin]

Il suffit de décrémenter i dans les 2 cas d'erreur