Modification tableau de pointeurs au sein d'une structure

Bonjour,
voici un soucis sur lequel je planche depuis à mon gout trop longtemps x]
J'espère que vous pourrez me filer un coup de pouce :)

==> Je dispose d'une structure joueur que je désire "initialiser"
=> Aucun soucis pour rentrer les coordonnées du joueur qui sont "pos_x" et "pos_y".

=> BITMAP* marche[7] est censé contenir les 7 images correspondant au mouvement "marche" de mon personnage, mais je n'arrive pas à rentrer les images dans le tableau (pas de soucis à la compilation) et j'obtiens un beau segment_fault avec le code que voici:

jeu.c

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

#include <stdio.h> #include <allegro.h> #include <string.h> #include <stdlib.h> #include "fenetre.h" #include "joueur.h"   typedef struct _Joueur{ char* name; double pos_x; double pos_y; BITMAP* marche[7]; BITMAP* poing[9]; BITMAP* saut[11]; }Joueur, *pJoueur ;   Joueur joueur1; joueur1.name="Silver";   joueur_init(&joueur1);

---

joueur.h

Code:

---

1 2	void joueur_init();

---

joueur.c

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

#include <stdio.h> #include <allegro.h> #include <string.h> #include <stdlib.h>   typedef struct _Joueur{ char* name; double pos_x; double pos_y; BITMAP* marche[7]; BITMAP* poing[9]; BITMAP* saut[11]; }Joueur, *pJoueur ;   void joueur_init(pJoueur J){   J->pos_x=300; J->pos_y=300;   J->marche[0]=load_bitmap("Perso1.bmp",NULL); }

---

Merci d'avance pour votre aide :)

bonjour

voici quelques idées :

dans le header vus avez :joueur_init(mettre un type)
car la fonction est utilisé : joueur_init(&joueur1); void joueur_init(pJoueur J)
et mettre un extern devant le prototype

si j'ai bien compris:
BITMAP* marche[7]; est peut etre a modifier
marche[7] est un tableau se type BITMAP.
il faut un pointeur de type BITMAP qui est initialisé &marche[0]
est un pointeur sur un

boon courage & bonne journée
PY

1- C'est étonnant que jeu.c compile :

Code:

---

1 2	joueur1.name="Silver"; joueur_init(&joueur1);

---

est un code hors de toute fonction.

2- Il ne faut pas dupliquer les définitions : les .h sont faits pour ça.

3- void joueur_init();n'est pas un prototype. Un prototype de fonction DOIT avoir la déclaration des types des paramètres.

4- Il est de bon ton pour un .c d'inclure le .h associé. Ceci permet de vérifier la cohérence des déclarations entre les deux.

joueur.h :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include <allegro.h> // pour avoir la déclaration de BITMAP typedef struct _Joueur { char* name; double pos_x; double pos_y; BITMAP* marche[7]; BITMAP* poing[9]; BITMAP* saut[11]; }Joueur ; // *pJoueur : ce n'est pas astucieux de cacher un pointeur dans un typedef void joueur_init(Joueur * joueur);

---

joueur.c :

Code:

---

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

#include <stdio.h> // si besoin dans joueur.c #include <string.h> // si besoin dans joueur.c #include <stdlib.h> #include "joueur.h" void joueur_init(Joueur* J) { if(J!=NULL) { J->pos_x=300; J->pos_y=300; J->marche[0]=load_bitmap("Perso1.bmp",NULL); } }

---

jeu.c :

Code:

---

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

#include <stdio.h> // si besoin dans jeu.c #include <string.h> // si besoin dans jeu.c #include <stdlib.h> // si besoin dans jeu.c #include "fenetre.h" // si besoin dans jeu.c #include "joueur.h" int main(void) //par exemple { Joueur joueur1; joueur1.name="Silver"; joueur_init(&joueur1); return 0; }

---

Merci diogene pour ton aide.
Evidemment j'avais oublié le main() dans jeu.c :)

J'ai effectué les modifications que tu as indiquées.
Malheureusement toujours le même résultat, à savoir:

=> la modification J->pos_x est bien modifié
=> la modification J->marche[0]=load_bitmap("Perso1.bmp",NULL) semble provoquer un segment fault :/

Pourtant l'adresse de l'image est correcte :/

EDIT: En fait je faisais l'initialisation d'allegro ( fenêtres etc) après celle du joueur d'où le segment fault ... Une inversion et tout était réglé.
Merci quand même pour vos réponses au moins les structures me paraissent plus clairs dorénavant =)