Discussion :

[Invité]

Bonjour.

J'ai un plantage très bizarre sur une fenêtre de mon application utilisant des QLineEdit.

Ma classe de fenêtre est déclarée comme suit :

EditAlerts.h

Code :

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

1
2
3
4
5
6
7
8
class EditAlerts : public QDialog
{
     public:
          EditAlerts();
 
     private:
          QGridLayout m_layout;
}

EditAlerts.cpp

Code :

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

1
2
3
4
5
6
7
EditAlerts::EditAlerts()
{
     this->setLayout(&m_layout);
     m_layout.addWidget(new QLabel("Edit alerts", this), 0, 0, 1, 10);
 
     /* Instructions de création, je n'utilise pas QDesigner */
}

Juste là tout fonctionne très bien, mais si j'ai le malheur de déclarer un QLineEdit en membre de la classe :

EditAlerts.h

Code :

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

1
2
private:
     QLineEdit m_min;

Dans ce cas, alors mon application va crasher à la fermeture de la fenêtre (qui n'est pas le fenêtre principale). Et ce que j'utilise ou non ces QLineEdit.

Et ce problème reste inchangé lorsque j'utilise des pointeurs plutôt que des objets en membres de la classe.

[koala01]

Salut,

En deux mots, c'est normal...

Il faut savoir que chaque fois que tu utilise une fonction qui a pour but de placer un objet dans un autre, l'objet "récipiendaire" devient propriétaire de l'objet qu'on lui donne.

En tant que propriétaire, il devient alors responsable de la durée de vie de l'objet qu'on lui a donné, qui ne pourra en aucun cas excéder celle du propriétaire.

Or, Qt se base sur le fait que les objets que l'on transmet ont été créés de manière dynamique (avec new ), et considère donc qu'il faut impérativement appeler delete sur chaque objet.

Seulement, m_layout (pour le premier cas) et m_min (dans le second) n'ont pas étés alloués de manière dynamique, et l'appel de delete sur un pointeur pointant sur un objet qui n'a pas été alloué dynamiquement (avec new) provoque un comportement indéfini, qui prend souvent la forme d'une erreur de segmentation.

La bonne manière de travailler est donc d'utiliser des pointeurs sur des objets alloués de manière dynamique:
EditAlerts.h

Code :

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

1
2
3
4
5
6
7
8
9
class EditAlerts : public QDialog
{
     public:
          EditAlerts();
 
     private:
          QGridLayout * m_layout;
          QLineEdit     * m_min;
}

EditAlerts.cpp

Code :

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

1
2
3
4
5
6
7
8
9
EditAlerts::EditAlerts()
{
     m_min = new QLineEdit;
     m_layout = new QGridLayout;
     this->setLayout(m_layout);
     m_layout->addWidget(new QLabel("Edit alerts", this), 0, 0, 1, 10);
     m_layout->addWidget(m_min, /* ... */);
     /* Instructions de création, je n'utilise pas QDesigner */
}

[Invité]

J'ai avais pensé, et j'avais déjà essayé, c'est ce que j'entendais par :

Et ce problème reste inchangé lorsque j'utilise des pointeurs plutôt que des objets en membres de la classe.

Maintenant, le problème s'est résolut lorsque mon membre de classe est passé de

Code :

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

QLineEdit m_min;

à

Code :

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

QVector<QLineEdit*> m_min

Il semblerait donc que je m'y soit mal pris la première fois.

Mais j'ai donc plusieurs question, concernant les delete.
Si j'ai un QGridLayout* m_layout, et que je l'ajoute à la fenetre avec setLayout, dois-je appeler delete dessus.
Et lorsque j'ajoute les QLineEdit* au layout avec addWidget(), dois-je aussi les delete ensuite ?

[Troudhyl]

http://qt-project.org/doc/qt-4.8/qwidget.html#setLayout

The QWidget will take ownership of layout.

http://qt-project.org/doc/qt-4.8/qlayout.html#addWidget
http://qt-project.org/doc/qt-4.8/qlayout.html#addItem

Note: The ownership of item is transferred to the layout, and it's the layout's responsibility to delete it.

Facile, en lisant la doc

[koala01]

J'ai avais pensé, et j'avais déjà essayé, c'est ce que j'entendais par :

Et ce problème reste inchangé lorsque j'utilise des pointeurs plutôt que des objets en membres de la classe.

Le seul fait d'utiliser des pointeurs n'est pas suffisant: un pointeur n'est jamais qu'une valeur numérique non signée qui représente "l'adresse mémoire à laquelle on trouvera un objet du type indiqué".

Ce qu'il faut surtout, c'est faire en sorte que l'adresse en question corresponde à l'adresse mémoire à laquelle on trouvera un objet créé de manière dynamique (avec new).

L'allocation dynamique de la mémoire pour ton objet (qu'il s'agisse du layout ou du QLineEdit) est indispensable pour s'assurer que Qt fonctionne correctement

Le code

Code :

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

QVector<QLineEdit*> m_min;

n'a de sens que si tu veux pouvoir disposer de plusieurs QLineEdit, mais il doit alors etre mis en relation avec un code proche de

Code :

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

m_min.push_back(new QLineEdit(/*...*/));

dans la (ou les) fonction(s) qui s'occupe(nt) de créer plusieurs champs d'éditions

Mais j'ai donc plusieurs question, concernant les delete.
Si j'ai un QGridLayout* m_layout, et que je l'ajoute à la fenetre avec setLayout, dois-je appeler delete dessus.
Et lorsque j'ajoute les QLineEdit* au layout avec addWidget(), dois-je aussi les delete ensuite ?

Surtout pas, malheureux...

Quand tu invoque setLayout, le widget au départ duquel tu invoques cette fonction va:

• correctement détruire le widget qu'il utilisait avant
• devenir propriétaire du layout que tu lui transmet

Lorsque le widget sera détruit, il détruira correctement le layout dont il est propriétaire (donc, celui que tu lui auras donné en appelant la fonction setLayout).

De même, lorsque tu invoque la fonction addWidget (ou addLayout) d'un layout, le layout devient propriétaire du widget (ou du layout) que tu lui as ajouté.

En tant que layout propriétaire, il veillera à détruire les objets dont il est propriétaire au moment où il sera détruit

Avec ce système, lorsque la fenêtre principale (ou toute autre fenetre / boite de dialogue / ...) est détruite, elle détruit le layout dont elle est propriétaire, et la destruction du layout occasionne la destruction de tout ce qu'il contient, le tout de manière récursive jusqu'à ce que tous les éléments graphiques contenus par la fenêtre / la boite de dialogue soient détruits