Fonction et pointeur

Bonjour à tous!

Voici 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

#include <stdio.h>   #define NB_PARAM 3 int i; unsigned char *parametres[NB_PARAM]={"height", "width", "duree"};   unsigned char commande[255]="";   /* Cette fontion permet de vérifier qu'il y a bien le nombre de paramètres nécessaires à l'initialisation de la vidéo */ void test_parametres(char *a, char *b){ if(b==NULL){ printf("Erreur de chargement sur %s\n", a);} else{ printf("passage de la valeur\n"); printf("a:%s\n",a); printf("b:%s\n",b); a=b; } }   /* Fonction main qui permet de créer un fichier vidéo */ int main(int argc,char *argv[]){   for(i=0; i<NB_PARAM; i++){ test_parametres(parametres[i],argv[i+1]); printf("parametre %d: %s\n", i, parametres[i]); } }

---

Ma question est assez simple, lorsque je faisais dans le main ceci

Code:

---

1 2 3

parametres[0]=argv[2]; printf("hauteur: %s \n", parametres[0]);

---

J'obtenais la valeur du paramètre rentré sans aucun souci.
J'ai voulu créer une fonction (afin de rendre mon code plus claire et plus facilement testable), lorsque j'exécute le programme, je n'obtiens plus la valeur rentrée en paramètre,
mais la valeur initiale (par exemple "height").

Je ne comprends pas parce que j'ai l'impression (fausse...) que je fais exactement la même chose..

Plutôt que de bidouiller, je voudrais comprendre où est mon erreur..
Merci
F.

Citation:

---

Envoyé par Flophx

J'ai voulu créer une fonction (afin de rendre mon code plus claire et plus facilement testable), lorsque j'exécute le programme, je n'obtiens plus la valeur rentrée en paramètre, mais la valeur initiale (par exemple "height").

---

La saisie de la nouvelle valeur se fait dans la fonction ? N'oublie pas que le C passe les arguments de fonction par valeur (i.e. copie de la variable passee en argument). Si tu veux modifier la valeur d'une variable a l'interieur d'une fonction, il faut passer son adresse.
Mauvais:

Code:

---

1 2 3 4 5 6 7 8 9

void f(int a) { a = 2; } ... int b = 3; f(b); /* b vaut toujours 3 */

---

Bon:

Code:

---

1 2 3 4 5 6 7 8 9

void f(int *a) { *a = 2; } ... int b = 3; f(&b); /* b vaut maintenant 2 */

---

Tel que vous l'avez écrit, c'est le pointeur local à la fonction test_parametres() qui est modifié, pas celui de main(). Il faut changer de la manière suivante:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

void test_parametres(char **a,char *b) { if (b==NULL) printf("Erreur de chargement sur %s\n",*a); else { /*...*/ *a=b; } }     dans main: test_parametres(&parametres[i],argv[i+1]);

---

1. Euh... C'est dans ton a=b que tu espérais faire le changement ?
2. Je pense que tu devrais utiliser des const char *, et sans doute aussi éviter de jouer avec leur signe.

Citation:

---

Envoyé par Flophx

Code:

---

1 2 3 4 5 6 7 8 9 10

void test_parametres(char *a, char *b){ if(b==NULL){ printf("Erreur de chargement sur %s\n", a);} else{ printf("passage de la valeur\n"); printf("a:%s\n",a); printf("b:%s\n",b); a=b; } }

---

---

Lorsque tu écris:

Code:

---

a = b;

---

Tu fais pointer a sur la même adresse que b. Mais n'oublie pas qu'en C, le passage des arguments se fait toujours par valeur, c'est à dire que a est initialisé avec une copie de la valeur contenue dans parametres[i] et b est initialisé avec une copie de la valeur contenue dans argv[i+1].

Lorsque tu modifie la valeur de a, tu ne modifie pas la valeur de parametres[i], car tu travaille sur une copie. Pour résoudre ton problème, il te faut un niveau d'indirection supplémentaire:

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

#include <stdio.h> #define NB_PARAM 3   void test_parametres(char **a, char *b){ if (a != NULL) { if(b == NULL){ printf("Erreur de chargement sur %s\n", *a); } else { printf("passage de la valeur\n"); printf("*a:%s\n", *a); printf("*b:%s\n", b); *a = b; } } }   int main(int argc,char *argv[]){ int i; char const*parametres[NB_PARAM] = {"height", "width", "duree"}; char commande[255] = "";   for(i = 0; i < NB_PARAM; i++){ test_parametres(parametres + i, argv[i+1]); printf("parametre %d: %s\n", i, parametres[i]); } return EXIT_SUCCESS; }

---

Thierry

Citation:

---

Euh... C'est dans ton a=b que tu espérais faire le changement ?

---

Oui et non, dans la mesure où j'avais bien vu que c'était la valeur qui était effectivement passée en paramètre, mais en passant l'adresse de la variable à modifier je me susi retrouvé face à cette erreur:

Code:

---

1 2	passing argument 1 of ‘test_parametres’ from incompatible pointer type

---

Et là, ne sachant pas ce qui cloche, je demande!

char const ** ?

Et de nouveau, en utilisant une indirection supplémentaire, j'ai:

Code:

---

1 2	passing argument 1 of ‘test_parametres’ from incompatible pointer type

---

avec ce code:

Code:

---

1 2	test_parametres(&parametres[i],argv[i+1]);

---

et

Code:

---

1 2 3 4 5 6 7 8 9 10

void test_parametres(char **a, char *b){ if(b==NULL){ printf("Erreur de chargement sur %s\n", *a);} else{ printf("passage de la valeur\n"); printf("a:%s\n",*a); printf("b:%s\n",b); *a=b; }

---

As-tu essayé avec?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

void test_parametres(char const **a, char const *b){ if (a != NULL) { if(b == NULL){ printf("Erreur de chargement sur %s\n", *a); } else { printf("passage de la valeur\n"); printf("*a:%s\n", *a); printf("*b:%s\n", b); *a = b; } } }

---

Thierry

Bien, effectivement en changeant tous les

Code:

---

1 2	unsigned char

---

par

Code:

---

1 2	char const

---

Ca marche, mais je comprends pas pourquoi l'erreur provenait en partie de là....

Normalement, ce sont juste des avertissements (warning) pour avoir mélangé pointeurs sur char, unsigned char, et const char. On peut passer outre avec un cast, mais le mieux est de se décider entre char et unsigned char.

Effectivement ce sont ces Warning, mais c'est assez étrange comme erreur pour moi, dans la mesure où je croyais que le C était moins typé... A moins que ca ne provienne du pointeur, qui ne sont pas "substituables". Je me trompe?

Les pointeurs sont en effet plus fortement typés que le reste en C.

Ce qui explique mes soucis...

Merci de toutes vos explications, un problème de plus resolu!

Citation:

---

Envoyé par Flophx

Effectivement ce sont ces Warning, mais c'est assez étrange comme erreur pour moi, dans la mesure où je croyais que le C était moins typé... A moins que ca ne provienne du pointeur, qui ne sont pas "substituables". Je me trompe?

---

On peut substituer les pointeurs. C'est juste un avertissement pour prévenir le programmeur qu'il fait peut-être une erreur. C'est si vite arrivé avec les pointeurs!