[DEBUTANT] Tableau et pointeur

Version imprimable

Bonjour

Je commence le C et je suis rendu au tableau avec les pointeurs et j'ai un petit souci.

Je regroupe toute les fonctions dans un meme fichier ou on peut choisir laquelle executer avec un menu (par exemple 1 : Calculer la somme du tableau 2: la moyenne etc...)

J'ai un souci au niveau des pointeurs mais je ne vois pas où.
Je vous met mon code

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
 
 
#include <stdio.h>
#include <stdlib.h>
 
static long tailletableau;
static long *pointeurtaille;
static long *pointeurtableau;
 
 
int menu() //menu pour le choix des fonctions a executer sur le tableau
{
    int choix;
    printf("==== MENU ====\n\n");
    printf("1 : Somme des valeurs du tableau\n");
    printf("2 : Moyenne des valeures du tableau\n");
 
    scanf("%d",&choix);
    return choix;
}
 
void init () //initialisation du tableau
{
    long i;
        printf("Rentrez la taille du tableau : ");
    scanf("%ld",&tailletableau);
    long tableau[tailletableau];
        for (i = 0 ; i < tailletableau ; i++)
    {
        printf("Rentrez les valeurs du tableau : ");
        scanf("%ld", &tableau[i]);
    }
    pointeurtableau = &tableau;
    pointeurtaille = &tailletableau;
 
}
 
long sommetableau(long tableau[], long tailletableau)
{
    long somme = 0;
    long i;
 
    for(i=0;i<tailletableau;i++)
    {
        somme = somme + tableau[i];
    }
    return somme;
}
 
void affiche(long *tableau, long tailletableau)
{
    long i;
 
    for (i = 0 ; i < tailletableau ; i++)
    {
        printf("%ld\n", tableau[i]);
    }
}
 
int main()
{
    init();
    //printf("%ld\n",pointeurtableau[0]); en rajoutant ces 2 lignes, ca plante (voir le probleme plus bas)
    //printf("%ld\n",pointeurtaille);
    printf("La somme du tableau vaut %ld", sommetableau(pointeurtableau, *pointeurtaille));
return 0;
}

Quand je compile et execute ce programme, il marche bien (enfin ca a l'air)

Puis je veux mettre en application le menu avec un switch.

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
 
#include <stdio.h>
#include <stdlib.h>
 
static long tailletableau;
static long *pointeurtaille;
static long *pointeurtableau;
 
 
int menu()
{
    int choix;
    printf("==== MENU ====\n\n");
    printf("1 : Somme des valeurs du tableau\n");
    printf("2 : Moyenne des valeures du tableau\n");
 
    scanf("%d",&choix);
    return choix;
}
 
void init ()
{
    long i;
        printf("Rentrez la taille du tableau : ");
    scanf("%ld",&tailletableau);
    long tableau[tailletableau];
        for (i = 0 ; i < tailletableau ; i++)
    {
        printf("Rentrez les valeurs du tableau : ");
        scanf("%ld", &tableau[i]);
    }
    pointeurtableau = &tableau;
    pointeurtaille = &tailletableau;
 
}
 
long sommetableau(long tableau[], long tailletableau)
{
    long somme = 0;
    long i;
 
    for(i=0;i<tailletableau;i++)
    {
        somme = somme + tableau[i];
    }
    return somme;
}
 
void affiche(long *tableau, long tailletableau)
{
    long i;
 
    for (i = 0 ; i < tailletableau ; i++)
    {
        printf("%ld\n", tableau[i]);
    }
}
 
int main()
{
    init();
    switch (menu())
    {
        case 1:
            printf("La somme du tableau vaut %ld", sommetableau(pointeurtableau, *pointeurtaille));
            break;
        default:
            printf("pas bon");
            break;
    }
return 0;
}

Et la, quand j'execute il me rend une valeur totalement fausse (du genre 24564423)
Si quelqu'un a une réponse a mon problème, je lui serai reconnaissant!!!

17/03/2007, 14h40
mchk0123
Quand tu écris :
Citation:
Code:

1 2 long tableau[tailletableau];
tableau est déjà un pointeur sur long. Donc il ne faut pas prendre son adresse dans l'instruction :

Citation:

Code:

pointeurtableau = &tableau;

Ecrit plutôt à la place :

Code:

pointeurtableau = tableau;

Bonjour,
Essayes avec le malloc , ça devrait marcher, voici le code, Et puis y'avait des erreurs dans ton code aprés compilation, Non??

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
 
#include <stdio.h>
#include <stdlib.h>
 
static long tailletableau;
static long *pointeurtaille;
static long *pointeurtableau;
 
 
int menu()
{
    int choix;
    printf("==== MENU ====\n\n");
    printf("1 : Somme des valeurs du tableau\n");
    printf("2 : Moyenne des valeures du tableau\n");
 
    scanf("%d",&choix);
    return choix;
}
 
void init ()
{
    long i;
        printf("Rentrez la taille du tableau : ");
    scanf("%ld",&tailletableau);
    long *tableau;
	tableau= (long*)malloc(tailletableau*sizeof(long));
        for (i = 0 ; i < tailletableau ; i++)
    {
        printf("Rentrez les valeurs du tableau : ");
        scanf("%ld", &tableau[i]);
    }
    pointeurtableau = tableau;
    pointeurtaille = &tailletableau;
 
}
 
long sommetableau(long tableau[], long tailletableau)
{
    long somme = 0;
    long i;
 
    for(i=0;i<tailletableau;i++)
    {
        somme = somme + tableau[i];
    }
    return somme;
}
 
void affiche(long *tableau, long tailletableau)
{
    long i;
 
    for (i = 0 ; i < tailletableau ; i++)
    {
        printf("%ld\n", tableau[i]);
    }
}
 
int main()
{
    init();
    switch (menu())
    {
        case 1:
            printf("La somme du tableau vaut %ld", sommetableau(pointeurtableau, *pointeurtaille));
            break;
        default:
            printf("pas bon");
            break;
    }
return 0;
}

Bonne chance!

17/03/2007, 15h15
odsen.s

Citation:

Envoyé par mchk0123

Quand tu écris :

tableau est déjà un pointeur sur long. Donc il ne faut pas prendre son adresse dans l'instruction :

Ecrit plutôt à la place :

Code:

pointeurtableau = tableau;

Il faudrait préciser à Nicolascrepe pourquoi, sinon ça ne sert à rien.

Code:

pointeurtableau = tableau;

est équivalent à

Code:

pointeurtableau = &tableau[0];

Le pointeur prend comme valeur l'adresse du premier bloc du tableau.
17/03/2007, 16h23
Nicolascrepe

Non il n'y a pas d'erreur pendant la compilation,

J'essaye de faire par une autre technique, je vous tiendrai au courant.
J'ai fait trop de chose a la fois (je suis un vrai debutant, j'ai vu les pointeurs il y a 2 jours)
Je vous tien au courant
17/03/2007, 16h40
abdou karim diagne

pointeurs

le tableau c est une adresse memoire dont on ne connait la ou ces valeurs pointes mais les pointeurs permettent de pointer a l'adresse direct dans la memoire

Theoriquement, lorsqu'on écrit :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
void init () //initialisation du tableau
{
    long i;
    printf("Rentrez la taille du tableau : ");
    scanf("%ld",&tailletableau);
    long tableau[tailletableau];
        for (i = 0 ; i < tailletableau ; i++)
    {
        printf("Rentrez les valeurs du tableau : ");
        scanf("%ld", &tableau[i]);
    }
    pointeurtableau = &tableau;
    pointeurtaille = &tailletableau;
 
}

Le tableau "tableau" est créé en tant que tableau local, donc détruit après l'appel de la fonction. Il faut utiliser un malloc et un free.

Nouveau code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
void init () //initialisation du tableau
{
    long i;
    printf("Rentrez la taille du tableau : ");
    scanf("%ld",&tailletableau);
    long* tableau=malloc(sizeof(long)*tailletableau);
        for (i = 0 ; i < tailletableau ; i++)
    {
        printf("Rentrez les valeurs du tableau : ");
        scanf("%ld", &tableau[i]);
    }
    pointeurtableau = &tableau;
    pointeurtaille = &tailletableau;
 
}

19/03/2007, 13h43
diogene
Nyarlathotep
Code:

1 2 3 4 5 6 7 8 void init () //initialisation du tableau { .... long* tableau=malloc(sizeof(long)*tailletableau); .... pointeurtableau = &tableau; pointeurtaille = &tailletableau; }
est incorrect pointeurtableau contient alors l'adresse d'une variable LOCALE (tableau) qui est détruite en sortie de fonction. On ne doit jamais utiliser l'adresse d'une variable locale en dehors de la fonction.

Par ailleurs, l'utilisation des variables globales est absolument à éviter.
19/03/2007, 13h46
Nyarlathotep
Ah, oui.
Il faudrait donc remplacer :
Code:

1 2 pointeurtableau = &tableau;
par:
Code:

1 2 pointeurtableau = tableau;

En supprimant les variables globales, main devrait ressembler à :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
int main(void)
{
 long tailletableau=0;
 long *tableau = init(&tailletableau);
        // tester si tableau n'est pas NULL. Si OK :
    switch (menu())
    {
        case 1:
            printf("La somme du tableau vaut %ld", sommetableau(tableau,tailletableau));
            break;
        default:
            printf("pas bon");
            break;
    }
   free(tableau);
return 0;
}

Du coup init a l'allure suivante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
long * init (long * dimension) //initialisation du tableau
{
    long i;
    long dim=0;
    printf("Rentrez la taille du tableau : ");
    scanf("%ld",&dim);
   // tester le retour de scanf puis si la valeur entrée est >0 sinon, reposez la question
 
    long* tableau=malloc(sizeof(long)*dim);
    // tester le retour de malloc. Si c'est OK
   *dimension = dim ;
   for (i = 0 ; i < dim ; i++)
    {
        printf("Rentrez les valeurs du tableau : ");
        scanf("%ld", &tableau[i]);
       // tester le retour de scanf
    }
return tableau ; 
}