Demande d'aide sur mon code en C

Version imprimable

tout d'abord bonjour,

Alors voila j'ai apris les bases theoriques du language C, je passe donc a la pratique.

Voici mon code qui consiste a créer un tableau de 4 valeurs puis d'aditionner les valeurs de ce tableau.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 #include <stdio.h> #include <stdlib.h> int sommeTableau(int tableau[], int tailleTableau); int main(int argc, char *argv[]) { int tableau[4]={15, 15, 15, 15}; sommeTableau(tableau, 4); return 0; } int sommeTableau(int tableau[], int tailleTableau) { int sommeTableau=tableau[0]+tableau[1]+tableau[2]+tableau[3]; printf("Le resultat est %d\n", sommeTableau); }
Ce petit programme marche mais j'aimerais avoir votre avis sur celui ci

Merci d'avance

28/12/2009, 17h47
Pouet_forever

Ton code fonctionne avec un tableau de 4 cases, mais si je te donne un tableau de 500 cases comment tu vas faire ? ;)
Tu as passé l'argument tailleTableau à ta fonction, hé bien il faut t'en servir pour faire une boucle et additionner tous les éléments de ton tableau ;)

Bonne chance !

PS : Il y a des balises de code. Le symbole # juste au dessus de là où tu écris ;)
28/12/2009, 18h16
tekos22

Merci beaucoup pour ta reponce je vais reflechir a ca.
30/12/2009, 11h23
dogua

Salut,

Le C est un exellent language à apprendre (Il est très ludique).

Si tu veux, tu pourrais déjà faire un sorte d'entrée les valeurs manuellement( Scanf)...

Tu pourrais essayer de faire une allocation dynamique. C'est à dire pouvoir encoder 3, 4, 20, 50 nombres et par après faire la somme de ceux ci via une boucle...

Enfin, voilà je te propose quelque idée qui te permettrons d'utilisé la puissance des tableaus :D

Citation:

Envoyé par tekos22

tout d'abord bonjour,

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <stdio.h>
#include <stdlib.h>
 
int sommeTableau(int tableau[], int tailleTableau);
 
int main(int argc, char *argv[])
 
{
    int tableau[4]={15, 15, 15, 15};
 
    sommeTableau(tableau, 4);
 
   return 0;
}
 
    int sommeTableau(int tableau[], int tailleTableau)
    {
        int sommeTableau=tableau[0]+tableau[1]+tableau[2]+tableau[3];
 
        printf("Le resultat est %d\n", sommeTableau);
    }

Ce petit programme marche mais j'aimerais avoir votre avis sur celui ci

Merci d'avance

Je relève quand même quelques erreur...

Ton programme se compile?? Car tu déclare ta fonction"SommeTableau" de type Integer, seulement tu ne retourne aucune valeur à la fin de ta fonction...

Pour ce type d'utilisation, tu devrais soit utiliser le type void ou alors voir ton code comme ceci :

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
35
 
#include <stdio.h>
#include <stdlib.h>
 
int sommeTableau(int tableau[], int tailleTableau);
 
int main(int argc, char *argv[])
 
{
    int tableau[4];
    int Resultat = 0;
 
    Printf("\n Entrer le premier nombre : ");
    scanf("%ld", &tableau[0]);
    Printf("\n Entrer le deuxieme nombre : ");
    scanf("%ld", &tableau[1]);
    Printf("\n Entrer le troisieme nombre : ");
    scanf("%ld", &tableau[2]);
    Printf("\n Entrer le quatrieme nombre : ");
    scanf("%ld", &tableau[3]);
 
    Resultat = sommeTableau(tableau); /* J'envoie le resultat dans une variable */
 
    printf("La somme des nombre est : %ld", Resultat); /* J'affiche le résultat */
    system("pause");
 
   return 0;
}
 
    int sommeTableau(int tableau[])
    {
        int Resultat=tableau[0]+tableau[1]+tableau[2]+tableau[3];
 
        return Resultat;
    }

PS: Je n'ai pas tester la compilation, il peut y avoir des erreur...
Je travaille maintenant en c# et sa fait pas loin de 1 an que je n'ai plus toucher de C ^^

30/12/2009, 14h55
SofEvans

Citation:

Envoyé par dogua

Je relève quand même quelques erreur...

Ton programme se compile?? Car tu déclare ta fonction"SommeTableau" de type Integer, seulement tu ne retourne aucune valeur à la fin de ta fonction...

Oui, ca se compile parfaitement, c'est juste qu'il y a quelque warning. En gros, le compilateur insere un return "quelquechose" et l'avertie.

Mais bon, si le compilateur n'est pas bien parametrer, il se peut qu'il ne voit pas les warning. OU alors, il les ignore carrement.
31/12/2009, 00h19
cagne

getch();
bonjour,
à mon avis ce style d'écriture n'est pas très catholique...
mais ça marche ... il faut simplement rajouter après le printf un getch();
en effet le prog n'a pas le temps d'afficher qu'il est déjà fini.

pas catho pourquoi ?
1) les arguments à la fonction main ne sont pas nécessaire
2) lorsque tu rappel le nom de la fonction pour la variable de travail, on a l'impression qu'on va s'offrir un voyage recurcif
tu devrais utiliser un autre nom de variable

4) l'autre warning est l'argument "taille" qui n'est pas utilisé, à ce sujet, dogua a raison, tu pourrais utiliser une boucle pour faire l'addition

je te propose la fonction ci-dessous
Code:

1 2 3 4 5 6 7 8 9 10 int sommeTableau (int tableau[], int taille) { int x, somme=0; for (x=0; x<taille; x++) somme += tableau[x]; printf ("Le resultat est %d\n",somme); getch(); }

Bonjour.

Citation:

Envoyé par cagne

un getch();
en effet le prog n'a pas le temps d'afficher qu'il est déjà fini.

Autant faire du standard (et du portable) et utiliser getchar();

@ cagne :
- Pour une bonne présentation sur le forum, il faut poster le code entre balises codes : comme sur cet émoticône :tagcode:.
- ta fonction sommeTableau () que tu vient de poster est de type int, elle doit donc retourner un int et non se comporter comme une procédure (void). Il faut bien faire la distinction entre fonctions (qui retournent une valeur) et procédures (dont on ne se soucie pas de la valeur retournée).

Pour t'aider à voir la différence :

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
35
36
37
38
39
40
41
#include <stdio.h>
#include <stdlib.h>
#define SIZE 4
 
int sommeTab(int*, int);
void affichSomme (int*, int);
void pause (void);
 
int main (void) /* Tu n'utilises pas les paramètres donc void */
{
    int tableau [SIZE]={15, 15, 15, 15}, somme=0 ;
    somme = sommeTab(tableau, SIZE); /* On affecte la valeur retournée par sommeTab à somme */
    printf ("resultat envoye par la fonction sommeTableau : %d\n", somme);
    /* Note : sans l'affectation, il était possible d'afficher sommeTab (tableau, SIZE) directement dans le printf */
    printf ("utilisation de la procedure :\n");
    affichSomme (tableau, SIZE); /* Appel de la procédure affichSomme */
    pause (); /* Appel de la procédure pause */
    return 0;
}
 
int sommeTab(int tableau[], int tailleTableau)
{
    int sommeTableau = 0, i=0;
    for (i=0 ; i<tailleTableau ; i++) /* Pour i de 0 jusqu'à la taille du tableau (avec un pas de 1), faire ... */
        sommeTableau += tableau[i]; /* sommeTableau = sommeTableau + tableau [i] */
    return sommeTableau; /* On retourne la valeur sommeTableau de type int */
}
 
void affichSomme (int tableau[], int tailleTableau)
{
    int sommeTableau = 0, i=0; /* évitez de donner les mêmes noms aux variables et aux fonctions */
    for (i=0 ; i<tailleTableau ; i++) /* Pour i de 0 jusqu'à la taille du tableau (avec un pas de 1), faire ... */
        sommeTableau += tableau[i]; /* sommeTableau = sommeTableau + tableau [i] */
    printf ("La somme est de : %d\n", sommeTableau);
}
 
void pause (void) /* procédure portable pour faire une pause */
{
    printf ("Appuyez sur ENTER pour continuer ...\n");
    getchar ();
}