utilisation de delete

Bonjour

Je possede une variable static défini ainsi dans le .h:

Code:

---

1 2	private: static CMemBuffer* listPreBuffer[3000];

---

et initialisé comme ça au début du .cpp

Code:

---

CMemBuffer* CMemBuffer::listPreBuffer[3000];

---

Dans une de mes méthode, j'ai l'instruction suivante:

Code:

---

1 2	for (i = 0; i < MAXBUFFER; i ++) listPreBuffer[i] = new CMemBuffer(1000000);

---

Et une fois que que j'ai utilisé ce dont j'avais besoin, je fais un

Code:

---

delete [] listPreBuffer;

---

Le compilateur me donne l'avertissement C4154 : deletion of an array expression; conversion to pointer supplied
Si j'exécute malgré tout mon programme, l'espace mémoire n'est pas libéré.

Pouvez-vous me dire quel est mon problème ?

Je vous remercie par avance

Ce ne serait pas plutôt comme ceci?

Code:

---

1 2 3

for(i = 0; i < MAXBUFFER; ++i) delete listPreBuffer[i];

---

Pourquoi n'utilises-tu pas un vecteur ?

Là, si ton système arrive à court de mémoire en cours de boucle, il y a des chances que ton exécution n'arrive jamais à ton instruction delete[], et tu auras alors des fuites de mémoire.

Citation:

---

Envoyé par oodini

Pourquoi n'utilises-tu pas un vecteur ?

Là, si ton système arrive à court de mémoire en cours de boucle, il y a des chances que ton exécution n'arrive jamais à ton instruction delete[], et tu auras alors des fuites de mémoire.

---

S'il n'y a plus de mémoire pour l'allocation, c'est plus le système en entier qui risque de planter à la place de l'application.

Tout dépend du coût de la construction de l'objet...

+1 pour l'utilisation de vector.
Mais bon, ça c'est de manière générale quand on parle de tableau, ici il y a un autre détail que je n'aime pas : tu stockes des pointeurs, donc tu dois forcément parcourir ton vector pour faire des delete dessus. Et ça, comme tout ce qui touche à la destruction des objets, ça devrait être géré par RAII (exception safety powaaa!).

Les solutions sont multiples:
- utiliser un vector de shared_ptr
- utiliser un boost::ptr_vector
- mettre le vector dans un objet qui va gérer correctement son cycle de vie

il faut faire un delete de ce qu'on a alloué avec "new"

en l'occurence, tu n'as pas fait new sur ton tableau donc le delete ne doit pas etre appelé... par contre tu dois faire le delete sur tous les elements de ton tableau (puisque tu as utilisé "new")

oui ton tableau prend 3000 * sizeof(CMemBuffer*) (ou * la taille d'un pointeur).

A la fermeture du programme, cette taille sera libéré, mais la mémoire allouée par les new sera toujours là.

Il faut donc parcourir les éléments et les supprimer un à un.



Mais oui utilise un vector, si tu fais un reserve(3000) au début, ça sera aussi performant qu'un tableau, et pour que ça soit propre à la suppression des éléments :
http://cpp.developpez.com/faq/cpp/?p...elete_sequence