mauvaises valeurs en sortie

**emprex** · 05/02/2008, 22h07

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 :

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
#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 :

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

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.

**emprex** · 05/02/2008, 22h36

j'ai trouvé une solution.

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
/* 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;
}

**emprex** · 05/02/2008, 23h28

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

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
#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.

**corentin59** · 06/02/2008, 09h11

Envoyé par emprex

Code :

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

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 :

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

void la_somme(double *somme, int T, int tab[])

Envoyé par emprex

Code :

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

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 :

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

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

**emprex** · 06/02/2008, 10h52

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 :

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
#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;
}

**corentin59** · 06/02/2008, 11h26

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.

**emprex** · 06/02/2008, 12h24

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 :

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
#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;
}

**corentin59** · 06/02/2008, 13h35

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 :

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
#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;
}

**emprex** · 06/02/2008, 14h55

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.

**Emmanuel Delahaye** · 07/02/2008, 19h25

Envoyé par emprex

Code :

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

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 :

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

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 :

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
 
#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;
}

mauvaises valeurs en sortie

C

Discussions similaires

Partager

Partager