Discussion :

[Pfeffer]

Bonjour,

J'aimerai savoir comment faire un delete propre pour supprimer tous les objets de tabResult?
En fait je crée un tableau d'objets, dans lequel j'insère des objets.

Code :

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

1
2
3
4
5
6
 
TCompareResult * tabResult;
TCompareResult tabResult = new TCompareResult[nbCombinaison];
 
result = new TCompareResult();
tabResult[indexCombinaison] = * result;

Ps: Comment faire une allocation propre, c'est à dire savoir si le new a bien fonctionné?

Merci

[Fiquet]

Tout est expliqué dans la FAQ ici : http://c.developpez.com/faq/cpp/?pag...EURS_new_array

Accessoirement ton code est faux je ne vois pas vraiment où tu veux en venir...

[Pfeffer]

D'accord merci mais que se passe t-il si les objets intégrer dans mon tableau n'ont pas de destructeur ?

[jc_isd]

Quand il n'y a pas de destructeur défini, le destructeur par defaut est appellé.

JC

[Pfeffer]

Et que détruit le destructeur dans ce cas la ?

[Laurent Gomila]

Il appelle le destructeur des données membres si elles en ont, et celui de la classe mère s'il y en a une.

[jc_isd]

... et donc si tu as des objets dont des membres sont des pointeurs sur des instances ( via new ), ce sont les pointeurs qui sont detruits par le constructeur par defaut, pas les objets instanciés --> mauvaise gestion mémoire --> toujours bien redefinir un destructeur (virtuel (cfr faq)). Je pense que tu trouveras des infos plus detaillées dans la FAQ.


A++

JC

[loufoque]

Code :

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

1
2
result = new TCompareResult();
tabResult[indexCombinaison] = * result;

Cette portion de code ne sert absolument à rien si ce n'est créer une fuite mémoire et réaliser une initialisation et une copie inutiles.

[Pfeffer]

Pourquoi ce code génèrait une fuite mémoire ?
La je ne comprends pas, le code est propre non ?

[loufoque]

Parce que la mémoire allouée par

Code :

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

result = new TCompareResult();

N'est plus référencée nulle part a priori, il est donc impossible de la libérer.

le code est propre non ?

Oulah, non, loin de là.
C'est du grand n'importe quoi. Et faire du code propre, c'est quelques niveaux au dessus que de faire du code qui ait du sens.

La solution la plus propre est probablement

Code :

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

std::vector<TCompareResult> tabResult(nbCombinaison);

[Jean-Marc.Bourguet]

La solution la plus propre est probablement

Code :

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

std::vector<TCompareResult> tabResult(nbCombinaison);

ou

Code :

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

1
2
3
4
std::vector<TCompareResult> tabResult;
tabResult.reserve(nbCombinaison);
 
tabResult.push_back(....)

ou utiliser une structure qui n'a pas besoin de stocker tout ca et agit sur chaque combinaison tour a tour.

[Pfeffer]

Et si je fais un delete sur la variable result après son utilisation, le code reste impropre? Normalement la mémoire devrait être désallouée non ? je ne comprends pas.

Code :

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

1
2
3
4
5
 
if(result != NULL){
delete result;
result = NULL;
}

Ce code devrait être bon théoriquement non ?

[Jean-Marc.Bourguet]

Si tu veux dire que tu utilises:

Code :

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

1
2
3
4
5
6
result = new TCompareResult();
tabResult[indexCombinaison] = * result;
if(result != NULL){
  delete result;
   result = NULL;
}

Ce code est correct en ce sens qu'il a le resultat auquel tu t'attends vraissemblablement, mais il n'est neanmoins pas bon du tout. Allouer de la memoire dynamique pour y construire un object avec le constructeur par defaut, copier cet objet et ensuite liberer la memoire dynamique est un abus certain de celle-ci.

Code :

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

tabResult[indexCombinaison] = TCompareResult();

(Accessoirement, delete est bien defini si l'argument est NULL).

[tut]

Oulah, non, loin de là.
C'est du grand n'importe quoi. Et faire du code propre, c'est quelques niveaux au dessus que de faire du code qui ait du sens.

La solution la plus propre est probablement

Code :

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

std::vector<TCompareResult> tabResult(nbCombinaison);

pas très sympa...
surtout que ta solution peut coûter cher si TCompareResult est long à copier...
Faut connaitre le contexte avant de juger.

[loufoque]

surtout que ta solution peut coûter cher si TCompareResult est long à copier...

Où tu vois la moindre copie ?
Il n'y aura copie qu'avec des push_back ou autres redimensionnements.
Ou copie du vector lui-même, bien sûr.

[Pfeffer]

Merci de toutes vos réponses, mais je ne comprends pas trop bien alors la notion d'allocation dynamique de mémoire.

Quelle est la différence entre:

Code :

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

1
2
 
TCompareResult * result = new TCompareResult();

et

Code :

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

1
2
 
TCompareResult result;

Dans les deux cas je peux utiliser les méthodes de mes classes. Seulement une fois j'utilise un point (result.test()) et l'autre fois j'utilise une fleche (result->test()).

Merci de m'éclaircir parce que la je ne comprends plus trop!

[loufoque]

Dans un premier cas tu as un pointeur vers un objet dynamiquement alloué que tu dois libérer explicitement et dans le second cas tu as un objet alloué sur la pile.

[Pfeffer]

et liberrer explicitement cela se traduit par un delete non ? Comme je l'ai montré précédemment!

[loufoque]

Précédemment tu ne faisais que libérer une copie.

[Pfeffer]

Donc j'ai suivi vos conseil pour parader ma fuite de mémoire! J'utilise les vector mais maintenant lorsque je veu supprimer son contenu, la mémoire allouée (gestionnaire des tâches) reste la même.

Voici ce que je fais:

Code :

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

1
2
3
4
 
//Création du vecteur
tabResult.reserve(nbCombinaison);
tabResult.push_back(TCompareResult());

Code :

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

1
2
3
4
 
//Suppression du vecteur et du contenu
tabResult.erase(tabResult.begin(),tabResult.end());
tabResult.clear();