Discussion :

[toto08]

Bonsoir
J'ai fait une structure qui doit représenter un entier grace a sa valeur sous forme de chaine de caractère et le nombre de chiffres de cet entier.
Dans une main, je l'initialise en appelant une autre fonction, puis je modifie cet entier en "concaténant" 1 a la fin de celui-ci.
Mais voila, rien ne marche
Peu importe la valeur de l'initialisation, les paramètres de la structure restent a 0.
Voici la partie du code concernée (répartie en fait dans plein de fichiers)

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
35
36
37
 
typedef struct {
	char *valeur;
	int nbchiffres;
} ValeurCourante;
 
int main(int argc, char **argv) {
 
	ValeurCourante *d;
	d=malloc(sizeof(ValeurCourante)); 
 
	InitValeurCourante(*d);
	printf("Initialisation -> valeur : %d, nbchiffres : %d \n", (d->valeur), d->nbchiffres);
	init_display(argc, argv, d);
	MainLoop();
	free(d);
	return 0;
}
 
void InitValeurCourante(ValeurCourante vc) {
	vc.valeur="999";/* vc.valeur un pointeur vers la valeur 999*/
	vc.nbchiffres=12;
}
 
void ajouter1(ValeurCourante vc) {
	vc.valeur="1";
	vc.nbchiffres++;
}
 
void ajouterChiffre1(Widget w, void *d) {
	/* d désigne la ValeurCourante */
	/* ajouter 1 a la ValeurCourante */
	ValeurCourante *t;
	t = (ValeurCourante *) d;
	ajouter1(*t);
	printf("valeur : %d, nbchiffres : %d \n", (int)t->valeur, t->nbchiffres);
}

J'espere avoir été assez clair
Je ne vois pas pourquoi tout ca reste constant.Si quelqu'un pouvait m'aider?

[Emmanuel Delahaye]

Voici la partie du code concernée (répartie en fait dans plein de fichiers)

Ton code est incomplet :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:10: warning: implicit declaration of function `malloc'
main.c:12: warning: implicit declaration of function `InitValeurCourante'
main.c:13: warning: implicit declaration of function `printf'
main.c:13: warning: int format, pointer arg (arg 2)
main.c:14: warning: implicit declaration of function `init_display'
main.c:15: warning: implicit declaration of function `MainLoop'
main.c:16: warning: implicit declaration of function `free'
main.c: At top level:
main.c:20: error: conflicting types for 'InitValeurCourante'
main.c:12: error: previous implicit declaration of 'InitValeurCourante' was here
main.c: In function `InitValeurCourante':
main.c:21: warning: assignment discards qualifiers from pointer target type
main.c: In function `ajouter1':
main.c:26: warning: assignment discards qualifiers from pointer target type
main.c: At top level:
main.c:30: error: syntax error before "w"
main.c:30: warning: function declaration isn't a prototype
main.c: In function `ajouterChiffre1':
main.c:34: error: `d' undeclared (first use in this function)
main.c:34: error: (Each undeclared identifier is reported only once
main.c:34: error: for each function it appears in.)
Process terminated with status 1 (0 minutes, 3 seconds)
6 errors, 10 warnings

Sinon, d'une façon générale, pour fonction modifie une variable, il faut lui fournir l'adresse de la variable via un pointeur du même type et utiliser le pointeur dans la fonction pour accéder à la variable via l'opérateur de déréférencement '*'.

Code :

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

1
2
3
4
5
6
7
8
 
   int x = 2;
 
   /* x vaut 3 */
 
   f(&x);
 
   /* x vaut 3 */

avec

Code :

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

1
2
3
4
5
 
void f (int *p)
{
   *p = 3;
}

Je te laisse transposer ça aux structures...

[toto08]

Bon en effet en changeant le type des paramètres de certaines fonctions ca marche beaucoup mieux (au moins les valeurs changent).
Mais j'ai l'impression que quand je fait:

Code :

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

1
2
3
4
5
 
void ajouter1(ValeurCourante * vc) {
	vc->valeur++;
	vc->nbchiffres++;
}

ca m'incremente le pointeur et non la valeur pointée (avec *(vc->valeur)++ c'est la meme chose).
Enfin, lorsque je change mon code comme ceci:

Code :

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

1
2
3
4
5
 
void ajouter1(ValeurCourante * vc) {
	strcat(vc->valeur, "1");
	vc->nbchiffres++;
}

j'obtient un bus error que je ne vois pas comment resoudre.
Help!!

[Emmanuel Delahaye]

Mais j'ai l'impression que quand je fait:

Code :

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

1
2
3
4
5
 
void ajouter1(ValeurCourante * vc) {
	vc->valeur++;
	vc->nbchiffres++;
}

ca m'incremente le pointeur et non la valeur pointée (avec *(vc->valeur)++ c'est la meme chose).

Je rappelle que la structure est :

Code :

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

1
2
3
4
5
 
typedef struct {
	char *valeur;
	int nbchiffres;
} ValeurCourante;

Donc, c'est sûr que vc->valeur++; modifie le pointeur.

Je pense qu'il y a un grave problème de conception.

vc->valeur est censé pointer sur quoi ?

[toto08]

Et bien il est censé pointer sur la valeur courante représentée comme chaine de caractères.
Pour la conception, je suis "obligé" d'avoir une structure de ce genre car en fait c'est un tp.

edit

ton pointeur valeur ne pointe pas vers une zone allouée

Mais le malloc lors de la creation d'une valeur courante n'est pas suffisant pour allouer de la memoire? (voir premier post)

[Emmanuel Delahaye]

Et bien il est censé pointer sur la valeur courante représentée comme chaine de caractères.

OK. Et en appliquant un ++ à ce pointeur, tu avais quoi en tête ? Tu t'attendais à quel résultat ?

[telliam]

Mais le malloc lors de la creation d'une valeur courante n'est pas suffisant pour allouer de la memoire? (voir premier post)

non ta premiere allocation alloue une zone mémoire pour ta structure valeurcourante il faut maintenant faire de mm pour le pointeur valeur de cette structure

[telliam]

Code :

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

1
2
3
4
5
 
void ajouter1(ValeurCourante * vc) {
	vc->valeur++;
	vc->nbchiffres++;
}

ca m'incremente le pointeur et non la valeur pointée (avec *(vc->valeur)++ c'est la meme chose).

effectivement valeur est un pointeur donc avec ton code tu incrémentes ton pointeur.

Enfin, lorsque je change mon code comme ceci:

Code :

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

1
2
3
4
5
 
void ajouter1(ValeurCourante * vc) {
	strcat(vc->valeur, "1");
	vc->nbchiffres++;
}

j'obtient un bus error que je ne vois pas comment resoudre.
Help!!

ton pointeur valeur ne pointe pas sur une zone alloué.