Discussion :

[Izem2022]

Bonjour,
Je reviens encore envers vous, vu que je suis autodidacte et j'apprends le langage C comme mon premier langage de programmation , alors je galère , parfois même je me met en doute est ce que c'est le bon choix pour commencer, ceci dit.

Voila je voudrai crée un tableau par l'allocation dynamique de la mémoire puis calculer la somme et la moyenne des éléments du tableau, le problème c'est que j'arrive pas à faire la somme et la moyenne des éléments de ce tableau, car j'ai le résulta der la somme et la moyenne dans la console 0, j'arrive pas à comprendre pourquoi, ou j'ai loupé un truc ?
voici le code :

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>
#include <stdlib.h>
 
int main ()
{
 
int i,n,S;
double M;
printf("entrez le nombre elements du tableau\n ");
scanf("%d",&n);
int *t=malloc(n*sizeof(int));// allocation dynamique de la mémoire pour le tableau t
 
if (t==NULL)
   {
    printf("la mémoire nest pas allouée");
    exit(0);
    }
else
{
    for (i=0;i<n;i++)
    {
        printf("entrez element t[%d]: ",i);
        scanf("%d",(t+i));
    }
 
    S=0;
    for (i=0;i<n;i++);
    {
        S=S+*(t+i);
    }
    M= (double) S/n;
 
    printf("la somme des elements est %d \n",S);
    printf("la moyenne des elements est %.2lf\n",M);
 
   free(t);
}
        return 0;
}

et voici le résulta de output dans la console
entrez le nombre éléments du tableau
4
entrez element t[0]: 12
entrez element t[1]: 12
entrez element t[2]: 13
entrez element t[3]: 15
la somme des elements est 0
la moyenne des elements est 0.00

Process returned 0 (0x0) execution time : 8.986 s
Press any key to continue.

Merci pour votre aide

[dalfab]

Bonjour,

Relis bien ta ligne 27. Le point-virgule!

[Sve@r]

Salut

j'arrive pas à comprendre pourquoi

Tu aurais rajouté un simple printf("t[%d]=%d\n", i, *(t+i)) dans ta boucle de calcul tu aurais trouvé de suite. Le débugging ce n'est pas seulement regarder son code, c'est aussi rajouter des affichages pour vérifier si les valeurs obtenues sont les valeurs attendues.

ou j'ai loupé un truc ?

un simple point-virgule en trop à la fin du for(...). C'est con hein ???

Accessoirement: si tu sors du programme quand le tableau n'est pas alloué, le "else" devient inutile => tu gagnes un niveau d'indentation (remarque que vu la tienne il est clair que tu n'en as pas besoin...).
Et c'est pas parce que tu utilises un pointeur alloué que tu n'as pas le droit d'utiliser la notation tableau. Ecrire t[i] donne exactement le même code que *(t+i) et reste bien plus lisible.

[edgarjacobs]

(....)
un simple point-virgule en trop à la fin du for(...). C'est con hein ??? (...)

Et combien de fois n'a-t-on pas croisé la question "pourquoi le compilateur me sort une erreur sur le else ? J'ai pourtant un if avant."

[Sve@r]

"pourquoi le compilateur me sort une erreur sur le else ? J'ai pourtant un if avant."

Dans ce cas t'as au-moins une erreur de compilation donc un guide. Dans le cas du PO, le truc compile nickel alors pour le trouver... (c'est en mettant le printf() dont je parle et que je n'ai pas vu ressortir à l'exécution que j'ai pu trouver l'erreur...)

[Izem2022]

Bonjour,

Relis bien ta ligne 27. Le point-virgule!

Bonjour,
Merci , c'était ça le problème alors que je cherchais le problème depuis plus de 3 jours lol
je vais jamais oublié ce point virgule lol

Salut

Tu aurais rajouté un simple printf("t[%d]=%d\n", i, *(t+i)) dans ta boucle de calcul tu aurais trouvé de suite. Le débugging ce n'est pas seulement regarder son code, c'est aussi rajouter des affichages pour vérifier si les valeurs obtenues sont les valeurs attendues.


un simple point-virgule en trop à la fin du for(...). C'est con hein ???

Accessoirement: si tu sors du programme quand le tableau n'est pas alloué, le "else" devient inutile => tu gagnes un niveau d'indentation (remarque que vu la tienne il est clair que tu n'en as pas besoin...).
Et c'est pas parce que tu utilises un pointeur alloué que tu n'as pas le droit d'utiliser la notation tableau. Ecrire t[i] donne exactement le même code que *(t+i) et reste bien plus lisible.

Bonjour, merci pour toute votre explication, je n'ai pas vu le point virgule c'était bête de de l'oublier , j'ai essayer de faire un printf pour afficher le contenu du tableau mais je n'ai pas pu voilà , vous avez raison utiliser t[i] est plus lisible que *(t+i), je vais revoir ma syntaxe

[Sve@r]

j'ai essayer de faire un printf pour afficher le contenu du tableau mais je n'ai pas pu voilà

Le fait que le tableau ne s'affiche pas indique que la boucle ne se fait pas. C'est là l'indice qui t'amène à examiner la boucle plus en détail.

[Izem2022]

Le fait que le tableau ne s'affiche pas indique que la boucle ne se fait pas. C'est là l'indice qui t'amène à examiner la boucle plus en détail.

d'accord merci