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 ?

[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é.

[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 ?

[toto08]

Et bien justement quand je me suis rendu compte de ca j'ai essayé de remplacer par *(vc->valeur)++ mais apparement ca incrémente aussi la valeur du pointeur non?

[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

[Emmanuel Delahaye]

Et bien justement quand je me suis rendu compte de ca j'ai essayé de remplacer par *(vc->valeur)++ mais apparement ca incrémente aussi la valeur du pointeur non?

Je te parle de spécification et tu me réponds implémentation. Ca ne va pas.

Peut importe ce que ça fait, tant que je ne sais pas ce que tu veux faire, on ne va pas avancer.

Si tu veux de l'aide véritable (pas du bricolage, OK ?) réponds à cette question :

Quelle modification veux tu apporter au pointeur ou à la chaine de caractères et pourquoi ?

On verra ensuite les moyens de le faire.

[toto08]

Je te parle de spécification et tu me réponds implémentation. Ca ne va pas.

Peut importe ce que ça fait, tant que je ne sais pas ce que tu veux faire, on ne va pas avancer.

Si tu veux de l'aide véritable (pas du bricolage, OK ?) réponds à cette question :

Quelle modification veux tu apporter au pointeur ou à la chaine de caractères et pourquoi ?

On verra ensuite les moyens de le faire.

Désolé.
La valeur courante est un entier. Le champ valeur de la structure est une représentation sous forme de chaine de caractères de cet entier et lorsque j'appelle la fonction ajouter1, et bien je voudrais concaténer cette chaine de caractères avec "1".
Dans le code posté avant, je testais juste en incrémentant de 1 mais avec la fonction strcat le probleme est le meme.

[Emmanuel Delahaye]

La valeur courante est un entier. Le champ valeur de la structure est une représentation sous forme de chaine de caractères de cet entier

C'est à dire ? Un identificateur quelconque ou une chaine genre 123 = "cent vingt-trois" ?

et lorsque j'appelle la fonction ajouter1, et bien je voudrais concaténer cette chaine de caractères avec "1".

Pour concaténer une chaine à une autre chaine, oui on utilise strcat(). Mais il y a 2 conditions pour que l'opération se fasse correctement :

• La chaine de destination doit être modifiable
• La chaine de destination doit être suffisamment grande.

Ces critères sont-ils vérifiés ?

[toto08]

Tout simplement, 123 = "123".
Quant au strcat, et bien la chaine de destination est bien modifiable mais il doit y avoir un problème au niveau de la taille de celle ci.
J'ai essayé d'allouer de la memoire

Code :

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

1
2
3
4
 
ValeurCourante *d;
d=malloc(sizeof(ValeurCourante));
d->valeur=malloc(10*sizeof(char));

mais cela ne resoud pas le probleme

[telliam]

Tout simplement, 123 = "123".
Quant au strcat, et bien la chaine de destination est bien modifiable mais il doit y avoir un problème au niveau de la taille de celle ci.
J'ai essayé d'allouer de la memoire

Code :

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

1
2
3
4
 
ValeurCourante *d;
d=malloc(sizeof(ValeurCourante));
d->valeur=malloc(10*sizeof(char));

mais cela ne resoud pas le probleme

il faut egalement que tu initialises la zone allouée avec le caractere de terminaison de chaine de caractere.Sinon ton strcat va potentiellement planter.
référe toi a la doc de strcat et a ton cours sur les chaines de caracteres en C.

[Emmanuel Delahaye]

Tout simplement, 123 = "123".<...>

Je crois qu'avant de te lancer dans des structures avec des pointeurs, tu devrais faire une 'maquette' de ce que tu veux faire dans un petit programme séparé qui montre très précisément les opérations une par une :

Si j'ai bien compris :

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
 
int main (void)
{
   int i;
   char *s;
 
  /* initialisation avec 123 */
  i = 123;
  char *s = malloc (sizeof "123");
  strcpy (s, "123");
 
  /* ensuite, c'est pas tres clair, mais voici ce que j'ai compris : */
  i++;
  char *s = realloc (s, sizeof "1231");
  strcat (s, "1");
 
}

Je te laisse compléter le code...

[toto08]

Voila ce que j'ai maintenant

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/*main*/
ValeurCourante *d;
d=malloc(sizeof(ValeurCourante));
InitValeurCourante(d);
...
/*initValeurCourante*/
void InitValeurCourante(ValeurCourante * vc) {
	vc->valeur=malloc(10*sizeof(char));
	(vc->valeur)='\0';
	vc->nbchiffres=0;
}

mais cela me donne tout de meme un bus error lorsque j'utilise le strcat, par la suite, alors qu'il y a moins de 10 caractères et je ne vois pas pourquoi...

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/*main*/
ValeurCourante *d;
d=malloc(sizeof(ValeurCourante));
InitValeurCourante(d);
...
/*initValeurCourante*/
void InitValeurCourante(ValeurCourante * vc) {
	vc->valeur=malloc(10*sizeof(char));
	(vc->valeur)='\0';
	vc->nbchiffres=0;
}

Si tu n'as pas compris ce que j'ai écrit, pose des questions, mais ne fait pas n'importe quoi comme ça au hasard...

[telliam]

Voila ce que j'ai maintenant

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/*main*/
ValeurCourante *d;
d=malloc(sizeof(ValeurCourante));
InitValeurCourante(d);
...
/*initValeurCourante*/
void InitValeurCourante(ValeurCourante * vc) {
	vc->valeur=malloc(10*sizeof(char));
	(vc->valeur)='\0';
	vc->nbchiffres=0;
}

mais cela me donne tout de meme un bus error lorsque j'utilise le strcat, par la suite, alors qu'il y a moins de 10 caractères et je ne vois pas pourquoi...

Code :

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

1
2
 
(vc->valeur)='\0';

là c'est le pointeur que tu initialises pas la zone pointée.
tu as le choix entre :

Code :

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

1
2
 
*(vc->valeur)='\0';

dans ce cas là tu initialises que le premier charactere sur les 10 que tu as alloués
soit

Code :

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

1
2
 
memset(vc->valeur,'\0',10);

là tu initialises les 10 caracteres.

[toto08]

Si tu n'as pas compris ce que j'ai écrit, pose des questions, mais ne fait pas n'importe quoi comme ça au hasard...

Arrf j'ai posté en meme temps je n'avais pas vu le post.
Menfin maintenant ca marche nikel grace a ce code.
Merci beaucoup de m'avoir aidé!!!!