Discussion :

[dan34]

Bonjour,

J'ai développé une application qui fonctionne correctement mais qui crashe lors de la fermeture.

Je l'ai réduite au minimum pour faire des tests mais rien n'y fait.
La conf minimum consiste en un QMainWindow auquel j'ajoute une statusBar perso.
Cette classe est appelé par un singleton.

A la fermeture de la fenêtre principale j'ai :
Exeption levée : violation d'accès en lecture
this a été 0xFFFFFFFFFFFFFFFF.

Là je cale, j'ai essayé d'analyser la pile d'appel mais en vain
ça plante sur la ligne 114 du ficher qlist.h alors que je n'ai pas de QList !!!

Code :

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

inline bool isEmpty() const Q_DECL_NOTHROW { return d->end  == d->begin; }

Si quelqu'un peut m'aider, merci d'avance

Je vous met le code utilisé
le .h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#ifndef STATUSBAR_H
#define STATUSBAR_H
 
#include <QStatusBar>
 
class StatusBar : public QStatusBar {
 
	Q_OBJECT
 
public:
	static StatusBar* getInstance(QWidget *parent);
 
private:
	StatusBar(QWidget *parent);
	~StatusBar();
 
	static StatusBar* m_Instance;
	bool m_bInstance = true;
};
 
#endif

le cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
#include "StatusBar.h"
 
StatusBar* StatusBar::m_Instance = (StatusBar*)Q_NULLPTR;
 
StatusBar* StatusBar::getInstance(QWidget *parent) {
	if (m_Instance == (StatusBar*)Q_NULLPTR) {
		m_Instance = new StatusBar(parent);
	}
	return m_Instance;
}
 
 
StatusBar::StatusBar(QWidget *parent) : QStatusBar(parent) {
	qDebug("Je passe par le constructeur");
 
	setStyleSheet("background-color: green; color: white");
	showMessage("La barre de status est bien présente.");
 
}
 
StatusBar::~StatusBar() {
	if (m_bInstance) {
		m_bInstance = false;
		delete m_Instance;
	}
}

Le MainTest.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#ifndef MAINTEST_H
#define MAINTEST_H
 
#include <QtWidgets/QMainWindow>
#include "ui_MainTest.h"
#include "StatusBar.h"
 
class MainTest : public QMainWindow
{
	Q_OBJECT
 
public:
	MainTest(QWidget *parent = Q_NULLPTR);
	~MainTest();
 
private:
	Ui::MainTestClass* ui;
	StatusBar* m_StatusBar;
};
 
#endif

le MainTest.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include "MainTest.h"
 
MainTest::MainTest(QWidget *parent) : QMainWindow(parent), ui( new Ui::MainTestClass() ) {
 
	ui->setupUi(this);
 
	m_StatusBar = StatusBar::getInstance(parent);
	setStatusBar(m_StatusBar);
}
 
MainTest::~MainTest() {
	delete ui;
}

et enfin de main.cpp classique

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include "MainTest.h"
#include <QtWidgets/QApplication>
 
int main(int argc, char *argv[])
{
	QApplication a(argc, argv);
	MainTest w;
	w.show();
	return a.exec();
}

[LittleWhite]

Bonjour,

La pile d'appel peut tracer de nombreux appels dans le code de Qt (et autres), donc, il faut la remonter afin de retrouver le dernier appel lié à votre code.
Cela nous aiderai mieux, si vous pouviez copier l'intégralité de la pile d'appels, du coup.

[dan34]

Merci de m'aider

Voici la pile d'appel

> Qt5Cored.dll!QListData::isEmpty() Ligne 108 C++
Qt5Widgetsd.dll!QList<QStatusBarPrivate::SBItem * __ptr64>::isEmpty() Ligne 185 C++
Qt5Widgetsd.dll!QStatusBar::~QStatusBar() Ligne 245 C++
MainTest.exe!StatusBar::~StatusBar() Ligne 26 C++
[Code externe]
Qt5Cored.dll!QObjectPrivate::deleteChildren() Ligne 1963 C++
Qt5Widgetsd.dll!QWidget::~QWidget() Ligne 1683 C++
Qt5Widgetsd.dll!QMainWindow::~QMainWindow() Ligne 392 C++
MainTest.exe!MainTest::~MainTest() Ligne 16 C++
MainTest.exe!main(int argc, char * * argv) Ligne 9 C++
MainTest.exe!WinMain(HINSTANCE__ * __formal, HINSTANCE__ * __formal, char * __formal, int __formal) Ligne 123 C++
[Code externe]

La ligne 26 du StatusBar.cpp est la parenthèse de sortie de destructeur.
Je ne sais pas trop quoi en déduire.

[LittleWhite]

Je pense que le problème provient du Singleton. En effet, Qt va tenter de détruire la barre des status de lui même (car ça fait partie de ces objets, voir la documentation Qt sur la GodClass (QObject)) et qu'au final, vous l'avez détruite avant lui.
Qui plus est, je ne vois pas pourquoi vous avez besoin d'un Singleton.

[dan34]

J'ai enlevé le "delete m_Instance" du destructeur de la StatusBar.
Et effectivement ça ne plante plus.

Qt va détruire la QStatusBar mais qu'en est il de StatusBar ?

Je me pose donc une question, ne va t'on pas avoir une fuite mémoire ?
On est entrain de fermer l'appli ce ne devrait pas poser de soucis, sauf si on le fait plein de fois

En réponse à votre question, je me suis dis que dans une application, il n'y a qu'une seule StatusBar donc un singleton me paraissait judicieux surtout que je dois y accéder de multiple endroit de l'application.

[LittleWhite]

Votre StatusBar, héritant de QStatusBar sera détruit par Qt (car il hérite d'un objet Qt). Comme je vous l'ai indiqué, il faut lire la documentation .

[dan34]

OK Merci,

Je vais me lire la doc bien que j'ai du mal avec l'anglais.