Discussion :

[damdam78]

Bonjour

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

Code :

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

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

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

Code :

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

CMemBuffer* CMemBuffer::listPreBuffer[3000];

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

Code :

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

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 :

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

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

[zabibof]

Ce ne serait pas plutôt comme ceci?

Code C++ :

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

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

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

[Davidbrcz]

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.

[loufoque]

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

[zais_ethael]

+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