Discussion :

[Katian]

mon main.c :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int main(int argc, char *argv[])
{
    long main_tableau[6]=
        {
            2,3,5,5,15,20
        };
    //    long SonTableau[6]={0};
 
    printf("Somme : %ld\n", sommeTableau(main_tableau, 6));
    printf("Moyenne : %lf\n", moyenneTableau(main_tableau, 6));
 
 
    return 0;
}

mon tableau.h :

Code :

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

1
2
3
long sommeTableau(long tableau[], long tailleTableau);
 
double moyenneTableau(long tableau[], long tailleTableau);

mon tableau.c :

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
#include "tableaux.h"
/*
Exercice 1 : créer une fonction sommeTableau qui renvoie la somme des valeurs
 contenues dans le tableau (utilisez un return pour renvoyer la valeur).
*/
long sommeTableau(long tableau[], long tailleTableau)
{
    long i;
    long l_somme = 0;
 
    for (i = 0; i < tailleTableau; i++)
    {
        l_somme+=tableau[i];
    }
 
    return l_somme;
}
/*
Exercice 2 : créer une fonction moyenneTableau  qui calcule et renvoie la
 moyenne des valeurs.
*/
double moyenneTableau(long tableau[], long tailleTableau)
{
    long i;
    double d_somme=0.0, d_moyenne;
 
 
    for(i=0 ; i< tailleTableau;i++)
    {
        d_somme += tableau[i];
    }
 
    d_moyenne = d_somme / tailleTableau;
 
    printf("moyenneTableau : %lf\n", d_moyenne);
 
    return d_moyenne;
}

A l'execution l'obtiens :

Somme : 50
moyenneTableau : 8.333333
Moyenne : 0.000000

Press ENTER to continue.

Pourquoi Moyenne : 0.000000 ???

Merci à vous !!!

[Emmanuel Delahaye]

Pourquoi Moyenne : 0.000000 ???

Ton code parait incomplet (je ne vois pas d'includes).

Regle ton compilateur :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Ensuite, recompile tout (sans exécuter : rebuild all) et poste les message d'erreur...

[Katian]

En tete de mon main.c j'ai bien :

#include <stdio.h>
#include <stdlib.h>

J'ai ajouté ceci dans CodeBlocks :

-Wall -Wextra -O2 -Wwrite-strings -Wstrict-prototypes

Et le resultat de compile :

Project : Console application
Compiler : GNU GCC Compiler (called directly)
Directory : D:\Travail\c-c++\c\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: tableaux.c
tableaux.c: In function `moyenneTableau':
tableaux.c:35: warning: implicit declaration of function `printf'
Compiling: main.c
main.c: In function `main':
main.c:12: warning: implicit declaration of function `sommeTableau'
main.c:12: warning: long int format, int arg (arg 2)
main.c:13: warning: implicit declaration of function `moyenneTableau'
main.c:13: warning: double format, different type arg (arg 2)
main.c: At top level:
main.c:4: warning: unused parameter 'argc'
main.c:4: warning: unused parameter 'argv'
Linking console executable: D:\Travail\c-c++\c\console.exe
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 7 warnings

merci !!!

[crocodilex]

Il suffit de savoir interpreter les warnings du compilateur.
Tu vois bien que tu utilises les fonctions sommeTableau() et moyenneTableau() dans la fonction main() alors qu'elles ne sont même pas déclarées.

[Katian]

Super ca marche, merci !

J'aurai imaginé que le compilateur genere une erreur sur ce genre de probleme !

donc comme je doit conserver mon #include "tableaux.h" dans mon tableaux.h (n'est-ce pas ? )

j'ai ajouté #include "tableaux.h" dans main.c

J'ai ajouter #ifndef ... etc dans le tableaux.h contre les risques d'inclusions multiples

youpi !

[crocodilex]

Super ca marche, merci !

J'aurai imaginé que le compilateur genere une erreur sur ce genre de probleme !

donc comme je doit conserver mon #include "tableaux.h" dans mon tableaux.h (n'est-ce pas ? )

j'ai ajouté #include "tableaux.h" dans main.c

J'ai ajouter #ifndef ... etc dans le tableaux.h contre les risques d'inclusions multiples

youpi !

J'espère que tu as aussi réglé le warning concernant "printf()" !!!

[hegros]

J'espère que tu as aussi réglé le warning concernant "printf()" !!!

Il faudrait qu'il inclus stdio.h dans son tableaux.h

[David.Schris]

Il faudrait qu'il inclus stdio.h dans son tableaux.h

Non, dans "tableau.c".

Un fichier d'en-tête ne doit inclure que le strict nécessaire à son autonomie.

[crocodilex]

Non, dans "tableau.c".

+1

[Emmanuel Delahaye]

Il faudrait qu'il inclus stdio.h dans son tableaux.h

Pourquoi ? "tableaux.h" ne dépend absolument pas de <stdio.h>.
Il fait mettre ce qui est est nécessaire, ni plus ni moins. Si dans un protptype, il il y avait un paramètre de type FILE*, OK, mais là, non.

[hegros]

oiu dans tableau.c plutot

merci pour les précisions