Discussion :

[kipgon]

Salut tout le monde,

je me tourne vers vous pour essayer de comprendre quelque chose concernant la gestion de la mémoire avec Qt.

J'ai vu cet exemple sur le forum

Code :

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

1
2
3
4
5
6
7
8
9
10
void showMessage()
{
    QDialog dialog;
    QVBoxLayout *layout = new QVBoxLayout(&dialog);
    QLabel *message = new QLabel("Cliquez sur le bouton");
    layout->addWidget(message);
    QButton *button = new QButton("Cliquez ici");
    layout->addWidget(button);
    dialog.exec();
}

http://qt.developpez.com/faq/?page=generalites-memoire

Il est dit que ce code fonctionne et c'est juste mais quand on regarde dans le gestionnaire de tâche on voit que toute la mémoire c'est pas libéré !

J'aimerais bien comprendre pourquoi ? Et comment faire pour récupérer cette mémoire ?

Merci d'avance

[chrtophe]

Je ne pense pas que la RAM soit libérée car ton QDialog n'est pas détruit. Il appartient à ta QApplication et sera détruite automatiquement à la fin de ta QPApplication.

un delete dessus devrait libérer sa RAM ( du QDialog ) et celle de ses enfants.

Après je suis débutant, donc ce que je dis est à prendre avec précautions.

[kipgon]

Mais on ne peut pas faire de delete puisque le QDialog n'est pas déclaré avec new ! non ?

[gbdivers]

Oui, pour le delete de dialog, il ne faut pas le mettre.
exec() est une fonction bloquante, donc ton dialogue sera affiché jusqu'à temps que l'utilisateur le valide ou l'annule. Donc l'exécution du programme ne sort pas de la fonction tant que le dialogue est ouvert : tout va bien.
dialog est un variable locale sur la pile, elle sera détruite à la fin de la fonction (comportement normal des variables locales en C++).
Le QLayout, le QButton et le QLabel sont créé dynamiquement avec new et sont attachés entre eux : lors de la destruction de QLayout, le QButton et le QLabel seront détruit (système de destruction des hiérarchies d'objets de Qt).
Par contre, ton QLayout n'est pas lié à un autre objet Qt, n'est pas local (donc pas détruit à la fin de la fonction) et n'est pas détruit par delete. Tu as donc bien une fuite mémoire (en plus du fait que ces objets ne servent à rien actuellement)
J'imagine que tu voulais mettre ton QLayout dans ton dialog ? Si tu fais cela, le QDialog détruira les autres objets Qt. Si ce n'est pas ce que tu veux faire, il vaut voir ce que tu veut faire de ton QLayout

[kipgon]

Le fait de créer le layout en lui indiquant que son parent est dialog par une référence

Code :

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

QVBoxLayout *layout = new QVBoxLayout(&dialog);

ne le lie pas à la fenêtre dialog ?

[gbdivers]

Si, j'avais pas vu
Cela permet de mettre le QLayout comme enfant et donc de le détruire automatiquement
(par contre, cela ne permet pas de mettre le layout dans le dialogue, il faudrait utiliser setLayout)
Donc il devrait pas y avoir de fuite mémoire

[kipgon]

Bah ouais je ne comprends pas trop ce qu'il se passe

[chrtophe]

Je présumes que la RAM ayant été allouée est prise dans le tas. Le peut être étendu mais ne rapetisse pas, par contre la RAM n'étant plus utilisée est dispo pour ton processus. Il y a peut-être 1 fonction pour le réduire le tas. ( je parle de fonction système, hors Qt )

[koala01]

Salut

Le fait de créer le layout en lui indiquant que son parent est dialog par une référence

Code :

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

QVBoxLayout *layout = new QVBoxLayout(&dialog);

Attention, tu ne lui indique pas dialog par référence, mais par pointeur!

En fait, le & de &dialog est l'opérateur "address of" (ou si tu préfères, l'opérateur "prend l'adresse de") qui permet d'obtenir... un pointeur vers l'objet en question, ce qui est, justement, attendu par le constructeur de QVBoxLayout (tout comme la plupart des constructeur des composants visuels, d'ailleurs, qui attendent un pointeur sur le composant visuel "propriétaire" )

[Troudhyl]

(par contre, cela ne permet pas de mettre le layout dans le dialogue, il faudrait utiliser setLayout)

Sisi, cf le code de Qt (qlayout.cpp) :

Code :

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

1
2
3
4
5
6
7
QLayout::QLayout(QWidget *parent)
    : QObject(*new QLayoutPrivate, parent)
{
    if (!parent)
        return;
    parent->setLayout(this);
}

[gbdivers]

Ok, si je dis autant de bêtises en aussi peu de messages, ça va plus...