Structure en C : Allocation et désallocation propre

Bonjour,

Je revisite un peu le langage C, étant plus connaisseur de la programmation objet C#.

je travaille avec les structure en C sur un petit projet de jeu en SDL (rien de bien exceptionnel). cependant je me pose la question de comment bien rendre la mémoire au système pour éviter les "memory leak".

Voici le bout de code que j'utilise pour déclarer, affecter et utiliser ma structure. J'aimerais savoir si cela est correct en terme de gestion de mémoire particulièrement le dernier volet.

Merci de vos commentaires et de toute aide que vous pourrez me donner:zoubi:.

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 35 36 37 38 39 40 41 42 43 44 45

//DECLARATION DE MA STRUCTURE typedef struct SpriteStruct { char NumerOfLife; unsigned short Xlocation; unsigned short Ylocation; char Energy; unsigned short *XframeLocation; unsigned short *YframeLocation; char *Hframe; char *Lframe; SDL_Surface *surface; };   //MISE EN PLACE DES TABLEAUX unsigned short *xframeLocation= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *yframeLocation= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *hframe= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *lframe= (unsigned short *) malloc(10 * sizeof(unsigned short)); heroSurface = SDL_LoadBMP("hero.bmp");   //INSTANCIATIOn struct SpriteStruct hero;   //AFFECTATION hero.NumerOfLife=3; hero.Xlocation=0; hero.Ylocation=320; hero.Energy=255; hero.XframeLocation=xframeLocation; hero.YframeLocation=yframeLocation; hero.Hframe=hframe; hero.Lframe=lframe; hero.surface=heroSurface;   //TRAITEMENTS ......   //DESALOCATION free(hero.XframeLocation); free(hero.YframeLocation); free(hero.Hframe); free(hero.Lframe); SDL_FreeSurface(hero.surface); free(hero);

---

déjà tu as une erreur (énorme) de définiton d'un type :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

typedef struct SpriteStruct { char NumerOfLife; unsigned short Xlocation; unsigned short Ylocation; char Energy; unsigned short *XframeLocation; unsigned short *YframeLocation; char *Hframe; char *Lframe; SDL_Surface *surface; };

---

ne définit pas une structure, ou plutôt elle en défini une, mais non utilisable, car là tout ce que tu as défini est un tag sur une structure...

Code:

---

typedef struct tag { ...} nameofstruct ;

---

Disons, qu'elle déclare un structure accessible comme struct SpriteStruct et que le typedef ne sert à rien.

Merci déjà de cette remarque très pertinente : j'ai corrigé le problème de syntaxe !

Cela nous donne :

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 35 36 37 38 39 40 41 42 43 44 45

//DECLARATION DE MA STRUCTURE typedef struct SpriteTag { char NumerOfLife; unsigned short Xlocation; unsigned short Ylocation; char Energy; unsigned short *XframeLocation; unsigned short *YframeLocation; char *Hframe; char *Lframe; SDL_Surface *surface; } SpriteStruct ;   //MISE EN PLACE DES TABLEAUX unsigned short *xframeLocation= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *yframeLocation= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *hframe= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *lframe= (unsigned short *) malloc(10 * sizeof(unsigned short)); heroSurface = SDL_LoadBMP("hero.bmp");   //INSTANCIATION struct SpriteStruct hero;   //AFFECTATION hero.NumerOfLife=3; hero.Xlocation=0; hero.Ylocation=320; hero.Energy=255; hero.XframeLocation=xframeLocation; hero.YframeLocation=yframeLocation; hero.Hframe=hframe; hero.Lframe=lframe; hero.surface=heroSurface;   //TRAITEMENTS ......   //DESALOCATION free(hero.XframeLocation); free(hero.YframeLocation); free(hero.Hframe); free(hero.Lframe); SDL_FreeSurface(hero.surface); free(hero);

---

1- le code exécutable en C doit être placé dans une fonction. Ce n'est pas le cas ici :

Code:

---

1 2	unsigned short *xframeLocation= (unsigned short *) malloc(10 * sizeof(unsigned short)); // et la suite

---

2- xframeLocation (et les autres qui suivent) n'est pas défini, c'est en tant que membre d'une structure qu'il est défini

3- Pourquoi des unsigned short et pas des char si on suit la définition de la structure

Code:

---

1 2	unsigned short *hframe= (unsigned short *) malloc(10 * sizeof(unsigned short)); unsigned short *lframe= (unsigned short *) malloc(10 * sizeof(unsigned short));

---

3- heroSurface n'est pas défini ici. La variable doit être définie avant d'être utilisée

Code:

---

heroSurface = SDL_LoadBMP("hero.bmp");

---

4-

Code:

---

1 2 3

struct SpriteStruct hero; ... free(hero);

---

hero n'a pas été obtenu par allocation dynamique et ne doit pas être détruite par free()

5 - On peut donc avoir quelque chose du genre :

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

//DECLARATION DE MA STRUCTURE typedef struct SpriteStruct { char NumerOfLife; //..... SDL_Surface *surface; } SpriteStruct;   //... fonction(....) { SpriteStruct hero; hero.NumerOfLife=3; hero.Xlocation=0; hero.Ylocation=320; hero.Energy=255; hero.XframeLocation = malloc(10 * sizeof(unsigned short)); hero.YframeLocation = malloc(10 * sizeof(unsigned short));; hero.Hframe = malloc(10); hero.Lframe = malloc(10); hero.surface = SDL_LoadBMP("hero.bmp"); // vérifier que toutes ces allocations ont réussi // Traitement si tout est OK.... free(hero.XframeLocation); free(hero.YframeLocation); free(hero.Hframe); free(hero.Lframe); SDL_FreeSurface(hero.surface); }

---

Merci beaucoup, ça m'éclaire énormément.

Une dernière chose avant de clore cette discussion.

Il me reste a savoir comment les allocations se sont bien faites.

Visiblement, les pointeurs référencent une zone mémoire où avant allocation il pourrait y avoir n'importe quoi. Comment discerner que ce qui est présent dans l'espace mémoire après allocation est correct ?

Merci

Citation:

---

Il me reste a savoir comment les allocations se sont bien faites.

---

En cas d'échec de l'allocation, les fonctions d'allocation retournent NULL.

Citation:

---

Visiblement, les pointeurs référencent une zone mémoire où avant allocation il pourrait y avoir n'importe quoi. Comment discerner que ce qui est présent dans l'espace mémoire après allocation est correct ?

---

Après une allocation (réussie) avec malloc(), la zone allouée contient ... n'importe quoi. Si on utilise calloc(), la zone allouée contient des 0.

il y a 2 élements :

• Primo s'assurer que l'allocation s'est bien passée
  
  Dans ce qu'a cité diogene, il faut ajouter la vérification :
  
  
  Code:

  ---

  1 2 3 4 5

  if ( (hero.XframeLocation = malloc(10 * sizeof(unsigned short))) == NULL ) { fprintf ( stderr, "Erreur d'allocation !!\n"); .... }

  ---

  
  
• Secondo, ce que fait l'alloction est simplement réserver une certaine zone à partir d'une certaine adresse.
  
  Donc avant l'utilisation il y a n'importe quoi à cet emplacement.
  
  Si ce sont des variables typées normalement (integer, double, float, structures, etc, ), une simple assignation assignera tous les bits correspondants.
  
  
  Code:

  ---

  1 2 3 4 5

  int i = 4, j ; struct MaStruct = {0}, MaStruct1;   j = i ; MaStruct1 = MaStruct ;

  ---

  Dans le cas d'une chaîne (un tableau de char), une manière sûre est soit d'intialiser manuellement chacun des élements, soit (ce qu'on fait en général) d'initialiser simplement la chaîne (strcpy ( chaine, "");). Si cependant une utilisation caractère par caractère de la chaîne est faite, sans être sûr de passer par des fonction de copie de chaîne (strcpy, strup, sprintf, etc), alors une assignation caractère par cractère sera nécessaire.
  

Note :

Je rajouterais effectivement suite à la dernière remarque de digoene que c'est une raison pour laquelle je n'utilise jamais malloc mais plutôt calloc..

Donc nous avons comme version finale quelquechose qui se rapproche de ceci :

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 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51

//DECLARATION DE MA STRUCTURE typedef struct SpriteStruct { char NumerOfLife; unsigned short Xlocation; unsigned short Ylocation; char Energy; char *XframeLocation; char *YframeLocation; char *Hframe; char *Lframe; SDL_Surface *surface; } SpriteStruct;   //... MaFonction(....) { SpriteStruct hero;   hero.NumerOfLife=3; hero.Xlocation=0; hero.Ylocation=320; hero.Energy=255;     if ( (hero.XframeLocation = malloc(10)) == NULL ) fprintf (stderr, "Erreur d'allocation XframeLocation !!\n");   if ( (hero.YframeLocation = malloc(10)) == NULL ) fprintf (stderr, "Erreur d'allocation YframeLocation!!\n");   if ( (hero.Hframe = malloc(10)) == NULL ) fprintf (stderr, "Erreur d'allocation Hframe !!\n");   if ( (hero.Lframe= malloc(10)) == NULL ) fprintf (stderr, "Erreur d'allocation Lframe !!\n");   if ( (hero.surface = SDL_LoadBMP("hero.bmp")) == NULL ) fprintf (stderr, "Erreur d'allocation surface !!\n");     // Traitement si tout est OK....   //LIBERATION   free(hero.XframeLocation); free(hero.YframeLocation); free(hero.Hframe); free(hero.Lframe); SDL_FreeSurface(hero.surface);   }

---

Avec le défaut que même en cas d'erreur d'allocation tu effectues le traitement et qu'alors cela n'a pas de sens.
Par contre, les free() ne poseront pas problèmes même en cas d'erreur d'allocation puisque free(NULL) est possible (et ne fait rien). Pour SDL_FreeSurface(NULL) je ne sais pas.

SDL_FreeSurface(NULL) fonctionne sans problème :)