problème de tableaux

Version imprimable

salut a tous j'ai un problem avec ce code
j'ai entrain de faire la somme de 4 cases d mais en resultat il affiche une grand nombre (4009944)
et en rincipe afichie le nombre 44
aide moi

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>
#include <stdlib.h>
 
 
int sommeTableau(int tableau[], int tailleTableau);
 
 
int main(int argc, char *argv[])
{
    int tableaux[4];
     tableaux[0]=2;
     tableaux[1]=3;
     tableaux[2]=4;
     tableaux[3]=5;
 
    return sommeTableau(tableaux,4);
 
 
   }
 
int sommeTableau(int *tableau, int tailleTableau)
{
    int i=0;
    for(i=0;i<tailleTableau;i++)
    {
        tableau[i+1]=tableau[i]+tableau[i+1];
    }
 
    return tableau[i+1];
 
       }

merci

07/10/2010, 00h43
LittleWhite

Bonjour et bienvenue,

Sachez qu'il y a une balise [ C O D E ] et [ / C O D E ] ( sans les espaces ) sur ce forum ( et ce n'est pas de la science fiction :aie: ) qui permette de présenter le code de manière très jolie et très colorée. Un raccourci est de juste cliquer sur le bouton qui ressemble à '#' c'est tout :)
Pensez y la prochaine fois ;)

Sinon pour votre problème, en fait, vous avez un bug, dans le sens, que votre programme devrait crasher ( erreur de segmentation ).

Effectivement, vous construisez un tableau de 4 cases.
Dans la fonction sommeTableau() vous faites la somme des valeurs des quatre cases, certes ( bien que je trouve la méthode assez bizarre ), sauf que votre boucle emmène i de 0 à 3.
Dans la boucle vous avez un 'i+1' qui veut dire, qu'à un certain point, i sera égale à 4. Un PC compte en commançant par zero, et que 4 equivaut à accéder à la 5 ième case du tableau.
Cette cinquième cases du tableau, vous ne l'avez jamais déclaré, du coup, ici il devrait avoir un bug ( erreur de segmentation ).
Il n'y en a pas, c'est un hasard / fait. Sauf que cette case à une valeur qui n'est pas déterminé non plus ( souvent c'est la dernière chose qui a été en mémoire, à cette emplacement, mais bref passons, la valeur n'est pas connue ). Du coup, vous ajouté votre résultat à quelque chose d'inconnu et vous le retourner au main. Ce résultat est inconnu est dans la majorité des cas faux.
Solution possible: ( je ne dis pas que c'est la meilleure, je vous laisse le privilège de faire une recherche pour savoir ce qui est possible )
- Définir un tableau de 5 cases ( la cinquième contiendra le résultat, comme vous semblez l'avoir prévu )
- à l'initialisation la cinquième case, doit contenir 0. ( Ceci est un indice très important )

Voilà, faites votre code, et revenez nous voir pour dire ce qu'il en ai :).

( Note: Il n'y a aucun piège ;) Vraiment :) ;) )
07/10/2010, 09h12
jeroman

Citation:

et en rincipe afichie le nombre 44

Comment arriver à un résultat de 44 ? Je ne comprends pas ce que tu veux faire. Donne-nous plus de précisions.

Outre ton problème de débordement de tableau, ton algorithme est bizarre (à moins que ce soit voulu) : tu additionnes le premier élément et le second pour mettre le résultat dans ce second élément, puis tu additionnes le second (dont la valeur a été écrasée) et le troisième pour mettre le résultat dans le troisième, etc. Au final, ton tableau contient ces valeurs : 2 5 9 14. Il n'y a de 44 nulle part.
07/10/2010, 20h19
fethi1987

Citation:

Envoyé par jeroman

Comment arriver à un résultat de 44 ? Je ne comprends pas ce que tu veux faire. Donne-nous plus de précisions.

Outre ton problème de débordement de tableau, ton algorithme est bizarre (à moins que ce soit voulu) : tu additionnes le premier élément et le second pour mettre le résultat dans ce second élément, puis tu additionnes le second (dont la valeur a été écrasée) et le troisième pour mettre le résultat dans le troisième, etc. Au final, ton tableau contient ces valeurs : 2 5 9 14. Il n'y a de 44 nulle part.

wi 14 desoli:? faut de frap

mais j'ai arrive pas
07/10/2010, 20h27
LittleWhite

Pourtant je pensais tout bien expliquer.

Avez vous suivi mes conseils ?

Qu'est ce que vous n'arrivais pas à faire ?
07/10/2010, 21h02
fethi1987

Citation:

Envoyé par LittleWhite

Pourtant je pensais tout bien expliquer.

Avez vous suivi mes conseils ?

Qu'est ce que vous n'arrivais pas à faire ?

normalment y afficher 14 mais il tjrs afficher des gros nombre

alors commet faire poue connait la somme d'un tableaux
07/10/2010, 21h06
LittleWhite

Avez vous faite les choses suivantes:

- Définir un tableau de 5 cases ( la cinquième contiendra le résultat, comme vous semblez l'avoir prévu )
- à l'initialisation la cinquième case, doit contenir 0. ( Ceci est un indice très important )

Car si vous ne l'avez pas fait, vous pouvez relire ma première réponse :evil:

Indice supplémentaires: Vous savez définir un tableau de 4 cases, alors pourquoi pas 5?
09/10/2010, 18h53
BRK__

Bonjour,
je ne sais pas si je peux me permettre mais:

Code:

return sommeTableau(tableaux,4);

ce ne serait pas mieu de mettre un printf() plutot qu'un return ?

voila 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
25
26
27
28
29
30
31
32
33
34
#include <stdio.h>
#include <stdlib.h>
 
// Prototype
 
int sommeTableau(int tableau[], int tailleTableau);
 
// Fonctions
 
int main(int argc, char *argv[])
{
    int tableaux[5] = {0};
        tableaux[0] = 2;
        tableaux[1] = 3;
        tableaux[2] = 4;
        tableaux[3] = 5;
 
    printf("La somme du tableau est %d.\n",sommeTableau(tableaux,4));
 
    return 0;
 
}
 
int sommeTableau(int *tableau, int tailleTableau)
{
    int i = 0, somme = 0;
 
    for(i = 0 ; i<tailleTableau ; i++)
    {
        somme = tableau[i] + somme;
    }
 
    return tableau[tailleTableau] = somme;
}

Je ne sais pas si c'est exactement ça que tu voulais faire.

09/10/2010, 19h36
Schopenhauer

Bonjour fethi1987,

J'aurais effectivement écris ce code a la manière de BRK_, il me semble plus instinctif en revanche je ne comprend pas pourquoi BRK_ ne fais pas un simple

Code:

return somme;

à la fin de sa fonction.

C'est juste une remarque pour ne pas compliquer les choses.

Edit : je viens de comprendre c'est pour conserver la valeur dans la dernière case du tableau, d'accord, j'avais lus un peu vite.

Edit2 : (voir post suivant)

Citation:

Envoyé par Pierre Dolez

Croyez-vous qu'ils vont se munir d'une cinquième tirelire? surement pas.
Donc la somme cherchée est la somme des 4 cases du tableau, renvoyée par le mot-clé return.

Je suis plutôt d'accord avec cela qui me semble plus instinctif.
09/10/2010, 19h39
Invité

Bonjour, BRK__
Je ne sais pas si je peux me permettre?
Notre ami a LA (?) solution, bon, très bien, il aura donc une bonne note.
Par contre il serait intéressant d'expliquer pourquoi vous mettez le résultat en cinquième position. Je suppose que ce n'est pas parce qu'il a été dit plus haut que cette opération provoquait un écrasement de mémoire, et qu'il fallait donc une case plus.
A quoi peut servir ce programme? Une famille de 4 enfants veut faire un cadeau à leurs parents. Ils se cotisent et regardent combien ils ont chacun dans leur tirelire, et doivent calculer la somme.
Croyez-vous qu'ils vont se munir d'une cinquième tirelire? surement pas.
Donc la somme cherchée est la somme des 4 cases du tableau, renvoyée par le mot-clé return.
10/10/2010, 00h08
fethi1987

Citation:

Envoyé par BRK__

Bonjour,
je ne sais pas si je peux me permettre mais:

Code:

return sommeTableau(tableaux,4);

ce ne serait pas mieu de mettre un printf() plutot qu'un return ?

oui j'ai essayé de faire ca mais donné moi la même valeur

Citation:

Envoyé par BRK__

voila 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
25
26
27
28
29
30
31
32
33
34
#include <stdio.h>
#include <stdlib.h>
 
// Prototype
 
int sommeTableau(int tableau[], int tailleTableau);
 
// Fonctions
 
int main(int argc, char *argv[])
{
    int tableaux[5] = {0};
        tableaux[0] = 2;
        tableaux[1] = 3;
        tableaux[2] = 4;
        tableaux[3] = 5;
 
    printf("La somme du tableau est %d.\n",sommeTableau(tableaux,4));
 
    return 0;
 
}
 
int sommeTableau(int *tableau, int tailleTableau)
{
    int i = 0, somme = 0;
 
    for(i = 0 ; i<tailleTableau ; i++)
    {
        somme = tableau[i] + somme;
    }
 
    return tableau[tailleTableau] = somme;
}

Je ne sais pas si c'est exactement ça que tu voulais faire.

merci mon ami et merci pour tous les autre ami

10/10/2010, 03h51
BRK__

fethi1987,
je ne sais pas si c'est réellement ce que tu voulais faire, dans la version que j'ai donné, les chiffres qui sont dans le tableau ne bougent pas et ne sont jamais remplacés.

La boucle ne fait juste qu'additionner les chiffres, et mettre le résultat dans la variable "somme".

Ensuite return tableau[tailleTableau] = somme; ajoute la valeur de la variable "somme" dans la dernière case du tableau.

Et donc comme l'on dit Schopenhauer, Pierre Dolez, LittleWhite et jeroman je ne sais pas si c'était le but recherché, de mettre cette somme dans la dernière case du tableau, car sinon il y a bien plus simple pour renvoyer la somme.

Pour finir, fais attention, j'ai donné une solution qui apporte le résultat que tu souhaitais, mais la manière est peut-être pas celle que tu voulais appliquer au départ.
10/10/2010, 19h28
fethi1987

Citation:

Envoyé par BRK__

fethi1987,
je ne sais pas si c'est réellement ce que tu voulais faire, dans la version que j'ai donné, les chiffres qui sont dans le tableau ne bougent pas et ne sont jamais remplacés.

La boucle ne fait juste qu'additionner les chiffres, et mettre le résultat dans la variable "somme".

Ensuite return tableau[tailleTableau] = somme; ajoute la valeur de la variable "somme" dans la dernière case du tableau.

Et donc comme l'on dit Schopenhauer, Pierre Dolez, LittleWhite et jeroman je ne sais pas si c'était le but recherché, de mettre cette somme dans la dernière case du tableau, car sinon il y a bien plus simple pour renvoyer la somme.

Pour finir, fais attention, j'ai donné une solution qui apporte le résultat que tu souhaitais, mais la manière est peut-être pas celle que tu voulais appliquer au départ.

merci beaucoup j'ai trouver la solution