Libération de la mémoire allouée pour les widgets enfants à leur destruction sans détruire le parent

Bonjour à tous,
voilà j'ai compris que Qt se charge de libérer la mémoire allouée aux widgets enfants quand on détruit le parent.

cela me pose un petit soucis, mais peut être que j'ai pas tout capté...

Pour vous expliquer simplement mon problème :

- j'ai une classe FenetreParent qui est dérivée de QMainWindow
- dans cette classe j'ai un bouton qui me permet d'ouvrir plusieurs FenetreFille (qui sont dérivées aussi de QMainWindow)
pour les ouvrir le QPushButton renvoie avec une fonction membre de FenetreParent.
Dans cette fonction je créé une FenetreFille de cette façon :

Code:

---

FenetreFille *fille = new FenetreFille(this);

---

Jusque là tout est normal.
Je précise que chaque FenetreFille me permet d'afficher de gros fichiers csv.

Grace à cette fonction membre de création de FenetreFille par new, je peux obtenir de multiples fenetres à l'écran et c'est ce que je cherche à faire.
Jusque là tout va bien

Sauf que lorsque je ferme une FenetreFille, la mémoire allouée n'est pas libérée, et mon ordinateur rame de plus en plus.
Comment puis je libérer la mémoire allouée à une FenetreFille à sa fermeture, si je ne détruit pas le parent.

Je précise qu'à la fermeture du parent toutes les FenetreFille sont bien détruites.
Quelqu'un saurait m'aider?
merci

Je propose une solution mais j'aimerai savoir si c'est correct dans la phylosophie de Qt de procéder ainsi?

En gros, je vais faire un delete this lors de la fermeture (closeEvent) des FenetreFille comme ceci:

Code:

---

1 2 3 4

void FenetreFille::closeEvent(QCloseEvent *event) { delete this; }

---

Le destructeur est ainsi bien appelé et donc je pense que la mémoire allouée est libérée.
Mais est ce politiquement correcte de coder comme cela en Qt?

Heuuu... j suis tout seul sur ma planète? mdr

Bon trêve de plaisanterie, je pense avoir trouvé la bonne solution, du moins celle qui semble correspondre à l'esprit Qt

En fait il suffit de rajouter un attribut à chaque FenetreFille juste après leur création comme ceci :

Code:

---

1 2	FenetreFille *fille = new FenetreFille(this); fille->setAttribute(Qt::WA_DeleteOnClose); // permet de faire un delete automatiquement lors d'un QCloseEvent de fille

---

Et voilà le tour est joué!
J'espère que ça sera utile à d'autres.
A bientôt pour de nouvelles questions et peut être réponses :P:mouarf::lol:
;)

Bonjour,

Concernant la première solution, il est risqué d'utilisé delete directement dans le corps d'une fonction de l'objet qui est détrui. Je pense que cela va engendrer un crash du programme. Par contre il y a une autre façon de faire avec Qt, qui permet de détruire un objet en "différé":

Code:

---

1 2 3 4

void FenetreFille::closeEvent(QCloseEvent *event) { this->deleterLater(); }

---

Bien sûre la solution utilisant

Code:

---

setAttribute()

---

est tout aussi valide. Par contre il serait encore plus simple de placer cet appel dans le constructeur de FenetreFille afin de rendre cela "automatique", enfin c'est du chipotage, le résultat sera le même:

Code:

---

1 2 3 4 5 6

FenetreFille::FenetreFille(QWidget *parent) : QWidget(parent) { //Code d'initialisation de classe this->setAttribute(Qt::WA_DeleteOnClose); }

---

Sinon, je vais faire mon chiant, mais il serait bien d'appliquer ce qui est dis dans votre signature ;)

Merci pour la réponse,
Pour ma signature, je l'ai faite il y a plus de 8 ans je pense, et depuis j'ai un peu laissé tomber le forum donc j'ai perdu cette bonne habitude.
Mais cette fois c'est volontaire, effectivement je n'ai pas mis le sujet en résolu parce que personne n'avait encore confirmé mes hypothèses.

C'est chose faite donc je valide.

Merci Gojir4.;)

Citation:

---

Envoyé par katanaenmousse

Mais cette fois c'est volontaire, effectivement je n'ai pas mis le sujet en résolu parce que personne n'avait encore confirmé mes hypothèses.

---

Autant pour moi :aie:

Salut,

Le lien parent/enfant ne devrait être utilisé que quand un élément ne peut effectivement pas exister si son parent cesse d'exister (par exemple : un bouton dans une fenêtre).

Dans ta situation, à moins de garder une trace de toutes les fenêtre ouvertes dans une collection quelconque (mais, est-ce que cela a vraiment du sens :question:), les fenêtre créées lors du clique sur le bouton ont une durée de vie limitée à leur durée d'exécution qui, soit dit en passant, empêchera sans doute ta "fenêtre principale" (celle qui contient le bouton provoquant l'ouverture de cette fenêtre supplémentaire) de faire quoi que ce soit d'autre en attendant.

Dés lors, le plus facile est -- peut-être -- tout simplement de n'avoir pas recours à l'allocation dynamique de la mémoire pour créer cette fenetre "secondaire", sous une forme qui pourrait être proche de

Code:

---

1 2 3 4

void FenetrePrincipale::trucMuch(){ FenetreFille fille; fille.exec(); // lance l'exécution de la fenêtre fille } // la fenetre fille est détruite ici, merci beaucoup

---

Bonsoir
En fait je suis obligé d utiliser l allocation dynamique vu que ma fenêtre principale et son bouton permettent d ouvrir un nombre indéfini de fenêtres filles
Et pour poursuivre ma fenêtre principale reste évidemment complètement fonctionnelle même avec plusieurs fenêtres filles ouvertes
En tout cas comme je l ai indiqué le sujet est résolu et tout fonctionne
Mais merci pour ta contribution tout de même :-)