Discussion :

[Flophx]

Bonjour à tous!

Voici mon 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
 
#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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[DaZumba]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
void f(int *a)
{
  *a = 2;
}
...
int b = 3;
f(&b);
/* b vaut maintenant 2 */

[stephl]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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]);

[Médinoc]

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.

[Flophx]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

[Flophx]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

avec ce code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
	}

[Thierry Chappuis]

As-tu essayé avec?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Flophx]

Bien, effectivement en changeant tous les

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
unsigned char

par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char const

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

[stephl]

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.

[Thierry Chappuis]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

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
#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

[Médinoc]

char const ** ?