Bonjour à tous

Voilà j'ai un problème dans l'exercice 2 du chapitre sur les tableaux.

L'énoncé :

Citation : M@teo21

Exercice 2 : créer une fonction moyenneTableau qui calcule et renvoie la moyenne des valeurs.

Je vous met mon code source :



Et ca me donne ça :



Je vous explique ma logique (illogique donc puisque ça marche pas ) : j'additionne les 4 valeurs du tableau que je divise par la taille du tableau (4) pour avoir la moyenne que je place dans moyenneTableau.

Ma question : pourquoi ça marche pas ? Et comment faire pour que ça marche ?

Merci d'avance pour votre aide

Citation : carrieres

double moyenneTableau = (*tableau+*(tableau+1)+*(tableau+2)+*(tableau+3))/tailleTableau;

*tableau pointe bien sur la première case du tableau, mais pas *(tableau+1)...
pourquoi pas simplement tableau[1]??

En plus la ta moyenne ne fonctionne que pour 4 cases, fait une boucle

Tu veux dire comme ça ?




Ca me donne le même résultat ...

for(i=0; i<tailleTableau; i++)
{
somme += tableau[i];
}
moyenne = somme/tailleTableau;

Ca devrait marcher non?

EDIT: euh ton code marche pas parce que tu affiches pas le résultat... mais l'adresse de la fonction! (enfin je crois)
Il te faut une variable résultat = moyenneTableau(tableau, longueur) dans ton main!

Le programme me met que la moyenne = 0

Rappel : un %ld va avec un long int, tandis qu'un %lf va avec un long float (double) :

Si je remplace le %ld par %lf, j'obtiens "-1.#QNANO" comme résultat

Ne faut-il pas également caster chaque valeur du tableau en tant que flottant pour le calcul de la moyenne ?

Citation : carrieres

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

double moyenneTableau(long tableau[], long tailleTableau)

{
    int i;
    double somme, moyenne;
   
    for(i=0; i<tailleTableau; i++)
    {
          somme += tableau[i];
    }
     
moyenne = somme/tailleTableau;

return moyenne;

   
}   
     

Bonjour :-)
Premièrement quand tu initialises somme je te conseille de le mettre à 0 avec:
double somme = 0;
Sinon il me semble que "somme" contiendra la valeur actuelle de ce qu'il y a dans la pile à l'endroit ou est créé la variable. Et donc il peut contenir quelque chose de différent de 0. Ce qui fait que dans ces cas la somme sera faussée vu que tu prends "somme" et tu lui additionnes tous les éléments. Au lieu d'avoir tableau[0] + tableau[1]... Tu as "valeur initiale de somme" + tableau[0] + tableau[1]...
Si quelqu'un peut confirmer ?

Et ensuite comme il a été dit plus haut, float semble plus approprié pour une moyenne, vu qu'elle peut contenir des virgules :-)

(Ah, aussi, tu utilises stdlib.h en include alors que tu n'en as pas besoin)

Et j'ai remplacé la nombre d'élement dans ton tableau qui était en dur par: sizeof(tableau)/sizeof(long) Je sais pas si c'est recommandé ou non, mais comme ca tu peux modifier les éléments de ton tableau (en ajouter, en supprimer...) sans avoir a toucher au reste du code :-)

Voila ce que j'ai fait:


et ca donne:
$ gcc -ansi -std=c99 -pedantic test.c
$ ./a.out
Valeur 0 : 23
Valeur 1 : 34
Valeur 2 : 75
Valeur 3 : 89
La moyenne du tableau est 55.250000

Citation : RiftAndChips

Sinon il me semble que "somme" contiendra la valeur actuelle de ce qu'il y a dans la pile à l'endroit ou est créé la variable. Et donc il peut contenir quelque chose de différent de 0. Ce qui fait que dans ces cas la somme sera faussée vu que tu prends "somme" et tu lui additionnes tous les éléments. Au lieu d'avoir tableau[0] + tableau[1]... Tu as "valeur initiale de somme" + tableau[0] + tableau[1]...
Si quelqu'un peut confirmer ?

Je confirme.