Une section private recalcitrante

Version imprimable

Bonjour,

Lors de la déclaration de ma classe dans le .h, je rencontre un drôle de problème. Dans la section private, je ne peux déclarer que un seul objet privé :roll:.

Si je déclare plusieurs objets, ca compile sans soucis, par contre l'application plante à son lancement (Le programme a généré des erreurs e tsera fermé par Windows... blablabla... voir le journal... blablabla).

Voici ma classe Veilleur.h :
Code:

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 28 29 30 31 32 33 34 35 36 37 38 39 40 #ifndef Veilleur_h #define Veilleur_h #include "Configuration.h" #include "MailChecker.h" #include "MC.h" #include <QTextStream> #include <QSettings> #include <QObject> #include <QTimer> #include <iostream> #include <QWidget> class QAction; class QMenu; class Veilleur : public QObject { Q_OBJECT public: Veilleur(); void Demarrer(); void Arreter(); void Suspendre(int duree); void Changer_frequence(int nouvelle_frequence); private slots: void Essai_timer(); private: MailChecker mChecker; QAction *Act_Tic; //MC mon_mc; //Configuration ma_config; }; #endif
Il s'agit d'une application QT4.2.2 compilée sous Windows avec MinGW.
Dans la section private, si je laisse "QAction *Act_Tic;", alors l'application plantera.
Il en va de même si je décommente la déclaration de "mon_mc" ou bien de "ma_config".

J'avoue ne rien comprendre et ca me frustre...

Pour plus d'informations, voici l'arborescence de mon programme :
Dans le main je crée une Icone (classe personnelle) :
Icone mon_icone;

Ma classe Icone déclare un Veilleur (en private) :
Veilleur mon_veilleur;

Ma classe Veilleur est celle ci-dessus, et elle déclare un MailChecker (en private) :
MailChecker mChecker;

Ma classe MailChecker va utiliser une socket TCP :
QTcpSocket *socket_r;

Voili voilou... Je ne comprends rien du tout :(.
Merci pour votre aide ;)

PS : D'ailleurs ca fait pareil si j'essaie d'ajouter quelque chose en public :yaisse2:

08/03/2007, 15h29
Laurent Gomila

Ca plante à quel moment ?
08/03/2007, 15h30
Nyphel

Au lancement de l'application, immédiatement.
08/03/2007, 15h31
Médinoc
1. Que signifie "slots" ?
2. Es-tu sûr que tu n'as pas une erreur dans les constructeurs de tes objets privés ?
08/03/2007, 15h33
Nyphel

Les slots sont gérés par le pré-compilateur de Qt, ils permettent de gérer des évènements entre objets.

Mes constructeurs sont complètement vides, et à vrai dire je trouve étrange que ca me fasse ca maintenant... Quand j'ai construit la classe ca buggait pas :s
08/03/2007, 15h33
Patriarch24

Citation:

PS : D'ailleurs ca fait pareil si j'essaie d'ajouter quelque chose en public

Le problème n'est pas un problème de droit d'accès. Je parie sur une classe mal initialisée.

Solution => Mode debug, et c'est parti !
08/03/2007, 15h36
Ulmo

Tu n'oublierais pas d'initialiser ton

Code:

QAction *Act_Tic;

des fois ?
08/03/2007, 15h37
Nyphel

J'ai pas de mode debug :aie:.
Non Ulmo je l'initialise dans le .cpp
08/03/2007, 15h53
Patriarch24

Citation:

J'ai pas de mode debug

Ca m'étonnerais... Tous les compilateurs que je connais peuvent générer des informations de débogage. A moins que je sois passé à côté ?
:bug:
08/03/2007, 23h07
olive_le_malin

pour ne pas poser trop de questions "à l'aveugle" pourrait-on voir le code cpp ?

Le voici, en version réduite :)

main.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
#include <QApplication>
#include <QtGui>
 
#include "Icone.h"
 
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);  
			Icone mon_icone;
    return app.exec();
}

Icone.h

Code:

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
28
29
30
31
32
33
34
35
#ifndef ICONE_H
#define ICONE_H
 
 #include "Veilleur.h"
 
 #include <iostream>
 #include <QSystemTrayIcon>
 #include <QWidget>
 
 class QAction;
 class QMenu;
 
 class Icone : public QWidget
 {
     Q_OBJECT
 
 public:
     Icone();
 
 protected:
     [...]
 
 private slots:
     [...]
 
 private:
     // QActions and QMenus
     // [...]
 
     QSystemTrayIcon *trayIcon;
 
     Veilleur mon_veilleur;
 };
 
#endif

Icone.cpp

Code:

1
2
3
4
5
6
7
8
9
10
	#include <QtGui>
 
	 #include "Icone.h"
 
	 Icone::Icone()
	 {
	     Creer_actions();  // Initialise the QActions
	     Creer_menu(); // Initialise the QMenus
	     Creer_TrayIcon(); // Initialise the trayicon and bind it the QMenus
	 }

Veilleur.h

Code:

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
28
29
30
31
32
33
#ifndef Veilleur_h
#define Veilleur_h
 
#include "MailChecker.h"
 
#include <QTextStream>
#include <QObject>
#include <QTimer>
 
 // Added in order to try to add a QAction
 #include <iostream>
 #include <QWidget> 
 class QAction;
 class QMenu;
 
class Veilleur : public QObject
{
	Q_OBJECT	
 
public:
    Veilleur();
	[...]
 
private slots:
	[...]
 
private:
	MailChecker mChecker; 
	//MailChecker mCheckersecond;  // Crash the application if uncommented
	//QAction *Act_Tic; // Crash the application if uncommented
	//QAction *Act_Ticlol; // Crash the application if uncommented
};
#endif

Veilleur.cpp

Code:

1
2
3
4
5
6
7
8
#include <iostream>
#include <QApplication>
 
#include "Veilleur.h"
 
Veilleur::Veilleur() : QObject()
{	
}

MailChecker.h

Code:

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
28
29
#ifndef MailChecker_h
#define MailChecker_h
 
#include <QThread>
#include <QTextStream>
#include <QSettings>
#include <QObject>
#include <QTcpSocket>
 
class QTcpSocket;
 
class MailChecker : public QObject
{
	Q_OBJECT	
 
public:
    MailChecker();
    [...]
 
private slots:
    void afficher_erreur_socket(QAbstractSocket::SocketError socketError);
 
private:
    [...]
 
    QTcpSocket *socket_r;
};
 
#endif

MailChecker.cpp

Code:

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
28
29
30
31
#include <iostream>
 
#include <QApplication>
#include <QHostAddress>
#include <QTcpSocket>
#include <QStringList>
 
#include "MailChecker.h"
 
 
MailChecker::MailChecker() : QObject()
{	
	connect(socket_r, SIGNAL(error(QAbstractSocket::SocketError)), this, SLOT(afficher_erreur_socket(QAbstractSocket::SocketError)));
}
 
void MailChecker::afficher_erreur_socket(QAbstractSocket::SocketError socketError)
{	
      switch (socketError) 
      {
        case QAbstractSocket::RemoteHostClosedError:
            break;
        case QAbstractSocket::HostNotFoundError:
            std::cout << "\n ERREUR SOCKET : Host not found" << std::endl;
            break;
        case QAbstractSocket::ConnectionRefusedError:
            std::cout << "\n ERREUR SOCKET : Connection refused by peer" << std::endl;
            break;
        default:
            std::cout << "\n ERREUR SOCKET : \n" << socket_r->errorString().toStdString() << std::endl;
        }
}

11/03/2007, 12h40
Nyphel

Petit UP, car je ne trouve pas de solution :( ...
Si quelqu'un a une diée, je suis preneur :)
11/03/2007, 13h10
Twindruff

Il faudrait que tu détermines exactement à quelle ligne ton programme plante. Et pour ça un debugger peut grandement t'aider.
11/03/2007, 13h24
Dudule-le-poisson

Bonjour Nyphel

Je viens de tester un peu ton code et j'arrive à me débarrasser de la segfault en initialisant le QTcpSocket *socket_r dans le constructeur de la classe MailChecker.
J'ai également pu décommenté les QAction * sans devoir les initialiser mais c'est parce que ton code réduit ne les utilise pas. Je ne sais pas si c'est suffisant pour ôter toute segfault de ton programme (personnellement, j'aurais préféré tester le code source non réduit) mais le principe suivant devrait t'aider : lorsqu'un de tes attributs n'est qu'un pointeur vers un objet d'un type donné, tu dois l'initialiser dans le constructeur de ton objet avant toute utilisation.
11/03/2007, 19h51
Nyphel

D'accord Dudule, je teste ca demain matin à la première heure :).

En fait ce que je faisait c'est que j'initialise la socket dans une fonction qui :
- Initialise la socket ;)
- L'utilise pour se connecter au serveur
- Emmet une requête précise
- Ecoute la réponse
- Ferme puis détruit la socket (deleteLater())

Je vais donc tenter de faire comme tu me le conseilles, en l'initialisant dans la constructeur :).

Dans tous les cas, merci pour le conseil sur l'initialisation des pointeurs.
J'avoue manquer d'expérience et de connaissances sur le C++, malheureusement je suis contraint dans le temps et ne peut pas vraiment m'attarder sur la consolidation de mes acquis :?

PS : Je crois ne pas disposer de debugger parce que je développe avec QT4.2.2 sur Windows, avec le compilateur MSYS/MinGW. J'édite le code avec Eclipse et son plug-in C++.
Je n'utilise pas le plug-in QtClipse qui permet d'avoir un IDE de développement Qt sous Eclipse car il semblait mal fonctionner sur ma plate-forme.
MinGW dispose (du moins je crois) d'un débugger, mais lorsque j'utilise l'option -d (donnée par le man) sur mon make, rien de particulier ne semble se produire :oops:.
11/03/2007, 22h36
Dudule-le-poisson

Citation:

Envoyé par Nyphel

Je vais donc tenter de faire comme tu me le conseilles, en l'initialisant dans la constructeur :).

Dans tous les cas, merci pour le conseil sur l'initialisation des pointeurs..

En fait, ce n'est nullement une obligation en soi. Ici c'est simplement nécessaire car le constructeur de MailChecker fait :

Code:

connect(socket_r, SIGNAL(error(QAbstractSocket::SocketError)), this, SLOT(afficher_erreur_socket(QAbstractSocket::SocketError)));

... ce qui va utiliser l'objet pointé par socket_r (ouais, en Qt, les slots/signaux sont gérés en partie à l'éxécution). Si tu ne l'initialises pas, non seulement tu ne donnes pas vraiment d'objet "QTcpSocket" mais en plus, tu demandes au programme d'aller voir ce qu'il y a à une adresse mémoire farfelue, ton pointeur n'ayant pas été initialisé. Le résultat de cette demande, c'est bien sûr une segfault :)
12/03/2007, 09h59
Nyphel

Bonjour,

Et bien merci beaucoup Dudule, cela fonctionne maintenant correctement et je comprends mieux quelle était mon erreur :).