mauvaises valeurs en sortie

Version imprimable

Bonsoir,
je fais u programme qui demande le nombre de valeurs à entrer dans un tableau, qui demande ensuite d'entrer ces valeurs puis qui les additionne et qui sort enfin le pourcentage.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 #include <stdio.h> #define TMIN 1 /* nb tranche d'age */ #define TMAX 100 #define PMIN 1 /* pop par tranche d'age */ #define PMAX 10000000 int main(void) { double tab[TMAX]; int i; int T = 0; double somme = 0; while ( T<TMIN || T>TMAX) scanf("%d",&T); for (i=0; i<T; i++){ scanf("%lf", tab); if (tab[i] >= PMIN && tab[i] <= PMAX) continue; somme += tab[i]; } for (i=0; i<T; i++) printf("%lf\n", (tab[i]*100)/somme); /* affiche le pourcentage par tranche d'age */ return 0; }
Mais les valeurs de sortie sont étranges. Je pense que le problème se situe au niveau de
Code:

1 2 3 4 for (i=0; i<T; i++){ scanf("%lf", tab); if (tab[i] >= PMIN && tab[i] <= PMAX) continue; somme += tab[i];
Pouvez vous m'aider svp.

j'ai trouvé une solution.

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
/* tranche d'age strohecker guillaume */
#include <stdio.h>
#define TMIN 1 /* nb tranche d'age */
#define TMAX 100
#define PMIN 1  /* pop par tranche d'age */
#define PMAX 10000000 
int main(void)
{
	double tab[TMAX];
	int i;
	int T = 0;
	double somme;
	while ( T<TMIN || T>TMAX)
	scanf("%d",&T);
	for (i=0; i<T; i++){
		scanf("%lf", &tab[i]);
		if (tab[i] >= PMIN && tab[i] <= PMAX) continue;
	}
	for (somme=0, i=0; i<T; i++)
		somme += tab[i];
	for (i=0; i<T; i++)
		printf("%lf\n", tab[i]*100/somme);  /* affiche le pourcentage par tranche d'age */
	return 0;
}

maintenant je voudrais le décomposer en fonction.
le probleme semble un peu plus complexe car a chaque compilation j'ai des erreurs.

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
#include <stdio.h>
#define TMIN 1 /* nb tranche d'age */
#define TMAX 100
#define PMIN 1  /* pop par tranche d'age */
#define PMAX 10000000 
 /* calcul de la somme */
 la_somme(double *somme, int T, int tab[])
{
	int i;
	for (somme=0, i=0; i< T; i++)
		somme += tab[i];
}
 
int main(void)
{
	double tab[TMAX];
	int i;
	int T = 0;
	double somme;
	while ( T<TMIN || T>TMAX)
	scanf("%d",&T);
	for (i=0; i<T; i++){
		scanf("%lf", &tab[i]);
		if (tab[i] >= PMIN && tab[i] <= PMAX) continue;
	}
	/*for (somme=0, i=0; i<T; i++)
		somme += tab[i];*/
	for (i=0; i<T; i++)
		printf("%lf\n", tab[i]*100.0/la_somme(&somme, T, tab));  /* affiche le pourcentage par tranche d'age */
	return 0;
}

peut-on me donner des conseilles svp.

06/02/2008, 09h11
corentin59
Citation:
Envoyé par emprex

Code:

1 2 3 4 5 6 7 /* calcul de la somme */ la_somme(double *somme, int T, int tab[]) { int i; for (somme=0, i=0; i< T; i++) somme += tab[i]; }
il faut que tu dises quel est le type de retour de ta fonction. Ici elle ne retourne rien, alors tu mets void :

Code:

void la_somme(double *somme, int T, int tab[])
Citation:
Envoyé par emprex

Code:

1 2 for (somme=0, i=0; i< T; i++) somme += tab[i];
je pense que c'est la valeur pointée par somme que tu veux changer, alors tu mets *somme :
Code:

1 2 3 4 *somme=0; for (i=0;i<T;i++) { *somme += tab[i]; }

merci,
j'ai corrigé ma fontion calul du résultat,
maintenant j'ai un problème sur une fonction affichage du résultat. Le compilateur me lance deux warnings à propos du pointeur *double et me parle d'un **double.

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
#include <stdio.h>
#define TMIN 1 /* nb tranche d'age */
#define TMAX 100
#define PMIN 1  /* pop par tranche d'age */
#define PMAX 10000000 
 /* calcul de la somme */
double la_somme(double *somme, int T, double tab[])
{
	int i;
	*somme=0;
	for (i=0; i< T; i++)
		*somme += tab[i];
	return *somme;
}
 
/* affichage de la somme */
double affiche_le_resultat(double *somme, int T, double tab[])
{
	int i;
	for (i=0; i<T; i++)
		printf("%lf\n", tab[i]*100.0/la_somme(&somme, T, tab));/* affiche le pourcentage par tranche d'age */
	return i;
}
 
int main(void)
{
	double tab[TMAX];
	int i;
	int T = 0;
	double somme = 0;
	while ( T<TMIN || T>TMAX)
	scanf("%d",&T);
	for (i=0; i<T; i++){
		scanf("%lf", &tab[i]);
		if (tab[i] >= PMIN && tab[i] <= PMAX) continue;
	}
	affiche_le_resultat(&somme, T, tab);
	return 0;
}

06/02/2008, 11h26
corentin59

Il y a qq confusions.

Dans ta fonction affiche_le_resultat, somme désigne un pointeur sur un double. Donc, quand tu appelles la fonction la_somme, ce n'est pas la peine de mettre &, sinon tu lui donnes l'adresse de ton pointeur (d'où le **double dans le message).

Ceci dit, il me semble inutile de donner un pointeur sur somme en paramètre de ta fonction la_somme, puisque maintenant elle te renvoie le résultat.

Enfin, affiche_le_resultat est censée renvoyer un double alors que tu mets un return i; où i est un int.

Alors encore merci. Je continue sur les fonctions et cette fois le compilateur ne n'annonce aucune erreur et ni warning.
le programme s'exécute et me demande la valeur de T, que je donne, et se coupe.
pourquoi ?

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
#include <stdio.h>
#define TMIN 1 /* nb tranche d'age */
#define TMAX 100
#define PMIN 1  /* pop par tranche d'age */
#define PMAX 10000000 
 /* calcul de la somme */
double la_somme(double somme, int T, double tab[])
{
	int i;
	somme=0;
	for (i=0; i< T; i++)
		somme += tab[i];
	return somme;
}
 
/* affichage de la somme */
double affiche_le_resultat(double somme, int T, double tab[])
{
	int i;
	for (i=0; i<T; i++)
		printf("%lf\n", tab[i]*100.0/la_somme(somme, T, tab));/* affiche le pourcentage par tranche d'age */
	return somme;
}
 
/* nombre de valeur à entrer dans le tableau */
int nb_de_valeur(int T)
{
	while ( T<TMIN || T>TMAX)
		scanf("%d",&T);
	return T;
}
 
/* remplissage du tableau */
double le_tableau(int T, double tab[])
{
	int i;
	nb_de_valeur(T);
	for (i=0; i<T; i++){
		scanf("%lf", &tab[i]);
		if (tab[i] >= PMIN && tab[i] <= PMAX) continue;
	}
	return tab[i];
}
 
 
/* fonction main */
int main(void)
{
	double tab[TMAX];
	int T = 0;
	double somme = 0;
	le_tableau(T,tab);
	affiche_le_resultat(somme, T, tab);
	return 0;
}

quand tu utilises des fonctions, fais attention aux paramètres en entrées et à la valeur retournée par la fonction. Tu fais parfois des choses redondantes et inutiles.

Voici un exemple de correction de ton code. exécutes-le et regardes s'il se comporte comme tu le souhaites. Puis regarde plus en détail le code et poses des questions s'il y a des choses que tu ne comprends pas.

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
#include <stdio.h>
#include <stdlib.h>
#define TMIN 1 /* nb tranche d'age */
#define TMAX 100
#define PMIN 1  /* pop par tranche d'age */
#define PMAX 10000000
 
 /* calcul de la somme */
double la_somme(int T,double tab[]) {
	int i;
	double somme=0;
 
	for (i=0;i<T;i++) {
	    somme += tab[i];
	}
 
	return somme;
}
 
/* affichage de la somme */
void affiche_le_resultat(double somme,int T,double tab[]) {
	int i;
 
	for (i=0;i<T;i++) {
	    printf("tab[%d] : %.2lf %%\n",i,tab[i]*100.0/somme);/* affiche le pourcentage par tranche d'age */
	}
}
 
/* nombre de valeur à entrer dans le tableau */
int nb_de_valeur(void) {
    int T=0;
 
    printf("Veuillez entrer la valeur de T : ");
    while ( T<TMIN || T>TMAX) {
        scanf("%d",&T);
    }
 
	return T;
}
 
/* remplissage du tableau */
void le_tableau(int T,double tab[]) {
	int i;
	double val;
 
	for (i=0;i<T;i++) {
	    val = 0;
        while ( val < PMIN || val > PMAX ) {
            printf("Veuillez entrer la valeur de tab[%d] : ",i);
            scanf("%lf",&val);
        }
        tab[i] = val;
	}
}
 
/* fonction main */
int main(void) {
	int T;
	double somme;
	double tab[TMAX];
 
	T = nb_de_valeur();
	le_tableau(T,tab);
	somme = la_somme(T,tab);
	affiche_le_resultat(somme,T,tab);
 
	return 0;
}

06/02/2008, 14h55
emprex

Merci, beaucoup pour ta collaboration. le code correspond à ce que je veux.
Je vais me poser sur le probleme du passage en paramètre et des valeurs de retour, car ca me pose pas mal de problèmes. Lorsque j'avais le tout dans un main le programme tournait bien.
Merci encore.

Citation:

Envoyé par emprex

Code:

1
2
3
 
	for (i=0; i<T; i++){
		scanf("%lf", tab);

Tu passes toujours la même adresse à scanf()...

Je conseille d'aider l'utilisateur à faire des saisies correctes.

Après correction :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Entrer le nombre d'elements entre 1 et 100
3
Entrer un element entre 1 et 10000000
12
Entrer un element entre 1 et 10000000
34
Entrer un element entre 1 et 10000000
56
11.764706
33.333333
54.901961
 
Press ENTER to continue.

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
 
#include <stdio.h>
 
#define TMIN 1                  /* nb tranche d'age */
#define TMAX 100
#define PMIN 1                  /* pop par tranche d'age */
#define PMAX 10000000
 
int main (void)
{
   double tab[TMAX];
   int i;
   int T = 0;
   double somme = 0;
   do
   {
      printf ("Entrer le nombre d'elements entre %d et %d\n", TMIN, TMAX);
      scanf ("%d", &T);
   }
   while (T < TMIN || T > TMAX);
 
   for (i = 0; i < T; i++)
   {
      do
      {
         printf ("Entrer un element entre %d et %d\n", PMIN, PMAX);
         scanf ("%lf", tab + i);
      }
      while (tab[i] < PMIN || tab[i] > PMAX);
      somme += tab[i];
   }
 
   for (i = 0; i < T; i++)
   {
      /* affiche le pourcentage par tranche d'age */
      printf ("%f\n", (tab[i] * 100) / somme); 
   }
   return 0;
}