Double pointeur sur structure, libération et allocation de mémoire

Bonjour,

J'ai un petit problème avec l'allocation et la libération de cette structure:

Code:

---

1 2 3 4 5

typedef struct AudioMegaConvertPlugin { char filtreOpen[size...]; char filtreSave[size...]; /* .... */ } AudioMegaConvertPlugin, * lpAudioMegaConvertPlugin;

---

Enfait, le problème est pendant l'allocation et après pour la libération, je ne suis pas sûr que mon code soit juste.
Je veux allouer autant de structures que j'en ai besoin, donc à chaque boucle, j'en alloue une nouvelle, voila la code:

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

AudioMegaConvertPlugin **amCplugin = {NULL}; /* ou (lpAudioMegaConvertPlugin *amCplugin) */ amCplugin = malloc(sizeof(AudioMegaConvertPlugin*));   /* ======== BOUCLE */ /* Quand je veux une nouvelle structure, je fais */ *amCplugin = malloc(sizeof(AudioMegaConvertPlugin));   /* Puis je veux par exemple remplir le membre 'filtreOpen' et le membre 'filtreSave' (ici par l'appel d'un plugin) */ dllStartV((*amCplugin)->filtreOpen, (*amCplugin)->filtreSave);   /* Maintenant, je veux une nouvelle structure sur le meme   pointeur donc, je fais: (C'EST LA QU'IL Y A UN PROBLEME) */ ++amCplugin; /* Et je recommence comme au dessus, malloc(), etc. */ /* ======== FIN BOUCLE */     /* ======== Puis arrive a la fin du programme, (2eme PROBLEME...)   je veux liberer ce fameux truc...   donc, je fais tant que (*amCplugin != NULL) free(amCplugin);   Mais je ne sais pas "où commence" "amCplugin" (emplacement memoire)   alors j'ai enregistre au debut du programme sa valeur par la variable   PluginAdress, comme cela: (PluginAdress = (int) amCplugin;) */   for(amCplugin = (int) PluginAdress ; *amCplugin != NULL ; free(*amCplugin), ++amCplugin) ; /* Mais mon compilo n'a pas l'air de trouver ca marrant, il me sort:   [Warning] assignment makes pointer from integer without a cast */

---

Si quelqu'un pourrait me dire comment faire une libération correcte et une allocation correcte car le "++amCplugin;" ne me semble pas correct, d'ailleur j'ai testé avec beaucoups de boucles et j'ai eu:

Citation:

---

Runtime Error!

Program : E:\[AudioMega]\scr\AudioMega.exe

This application has requested the Runtime to terminate it in an unsual way.
Please contact the application's support team for more information.

---

quand je faisais le "++amCplugin;"



Merci

J'initialiserai le tableau de pointeur.

Puis j'utiliserai des index.

Donc par exemple :

Code:

---

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

const unsigned nb_element_max = 10; unsigned nb_element = 0;   AudioMegaConvertPlugin **amCplugin = {NULL}; amCplugin = malloc(nb_element_max * sizeof(AudioMegaConvertPlugin*)); memset(amCplugin, 0, nb_element_max * sizeof(AudioMegaConvertPlugin*));   while(nb_element < nb_element_max) { amCplugin[nb_element] = malloc(sizeof(AudioMegaConvertPlugin)); /* test si bien alloue etc */ dllStartV(amCplugin[nb_element]->filtreOpen, amCplugin[nb_element]->filtreSave); nb_element++; }   while(nb_element) { delete amCplugin[nb_element]; nb_element--; } delete amCplugin

---

Tu peux aussi faire un tableau qui se fini toujours par un pointeur NULL. et faire une sorte de tant que pointeur pas null alors delete le pointeur. Mais apres cela dépend de ce qe tu fais avec ton tableau.

Merci mais je compile en C (delete existe en C ?).
<reEdit> Mais le problème sur ton code c'est qu'on est limité sur 'element_max', alors que moi je veux quelque chose que totalement indépendant, qui peux fonctionner même avec 5600560560918456 plugins :-)

n'y a t-il pas moyen 'tout simplement' d'utiliser realloc() (sauf que je n'y arrive pas :P).

Bon je crois qu'il faut reprendre à la base...

1)

Code:

---

1 2 3 4

AudioMegaConvertPlugin **amCplugin = {NULL}; /* ou (lpAudioMegaConvertPlugin *amCplugin) */ amCplugin = malloc(sizeof(AudioMegaConvertPlugin*));

---

ici tu alloues la place pour 1 pointeur de type amCplugin

2)

Code:

---

1 2 3

* Quand je veux une nouvelle structure, je fais */ *amCplugin = malloc(sizeof(AudioMegaConvertPlugin));

---

ici tu ne fais pas ce que tu crois.... Ce n'est pas le CONTENU du pointeur alloué plus haut que tu veux modifier, c'est le premier élément du tableau....


Comme en 1) tu as créé un TABLEAU de pointeurs à 1 élément, L'écriture correcte et conforme est :

Code:

---

1 2 3

* Quand je veux une nouvelle structure, je fais */ amCplugin[0] = malloc(sizeof(AudioMegaConvertPlugin));

---

3)

Code:

---

1 2	++amCplugin;

---

Incrémente l'adresse d'un tableau de pointeurs... :aie:

Tu ne peux pas faire ça.....


Ce que tu veux faire c'est :

• Stocker un nombre d'éléments (par exemple nbPlugin) initialisé à 0.
  
  
• Allouer (ou réallouer, avec dans ce cas une variable intermédiaire) nbPlugin+1 adresses d'éléments (le tableau) ( 1) modifié)
  
  
• Allouer pour l'indice en question la place pour une structure ( 2) modifié)
  
  
• Si tout s'est bien passé incrémenter le nombre d'éléments nbPlugin

Ok, c'est bien ce qu'il me semblait mais je ne savais pas trop comment le "formuler", merci pour ta réponse.

Donc, je dois:
O Allouer amCplugin

Code:

---

1 2 3

amCplugin = malloc(sizeof(AudioMegaConvertPlugin *)); nbPlugins = 0; size_t sizeAmCplugin = 1;

---

(j'ai donc un tableau de 1 case (amCplugin[0])

O Allouer une structure dans cette premiere case du tableau:

Code:

---

amCplugin[nbPlugins] = malloc(sizeof(AudioMegaConvertPlugin));

---

O Remplir mon tableau:

Code:

---

1 2	dllStartV(amCplugin[nbPlugins]->filtreOpen, amCplugin[nbPlugins]->filtreSave); /* .... */

---

O Faire un realloc() sur amCplugin pour avoir un nouvelle case dans le tableau et incrementer 'nbPlugins'; (pas sûr)

Code:

---

1 2	++sizeAmCplugin; realloc(amCplugin, sizeAmCplugin * sizeof(AudioMegaConvertPlugin *))

---

O Allouer une structure dans cette 2eme case:

Code:

---

amCplugin[nbPlugins] = malloc(sizeof(AudioMegaConvertPlugin));

---

O Remplir le tableau
O ETC...

O Pour la libération:

Code:

---

1 2	for(; nbPlugins >= 0 ; free(amCplugin[nbPlugins]) , --nbPlugins) ; free(amCplugin);

---

Correct ? :)

Oui à 2 ou 3 nuances près :

1. Ton free ne libère que le tableau, pas les structures
   
   
2. Vérifier que les retours d'allocation sont corrects, et sinon prévoir des échapatoires
   
   
3. Pour le realloc, passer par un buffer temporaire, car si ça échoue, tu n'auras plus accès au tableau précédemment alloué
   
   
4. Pour la libération, pourquoi ne pas faire une boucle claire ?
   
   
   Code:

   ---

   1 2 3 4

   for ( i = (nbPlugins-1) ; i >= 0 ; i-- ) free (amCplugin[i]);   free(amCplugin);

   ---

<edit> Tout fonctionne, merci énormément ! :-)