Discussion :

[Niak74]

Bonjour,
je vais tenter de vous exposer brièvement mon problème.

J'ai une classe mère (nommée ClasseMere) qui étend de QFrame, et une classe fille (nommée ClasseFille, qui étend donc de ClasseMere).

Les classes filles sont construites selon les directives de la classe mère et possèdent des attributs qu'il est nécessaire de sauvegarder avant fermeture.
Pour se faire, je comptais implémenter la modélisation suivante :

La classe mère :
- étend QFrame
- possède un slot virtuel "void saveAndClose()"
- redirige le signal d'un bouton fermer vers le slot virtuel "void saveAndClose()"

La classe fille :
- étend de la classe mère et propose ses propres attributs (à sauvegarder)
- implémente le slot virtuel "void saveAndClose()" décrivant la manière de sauvegarder les données, et ferme la fenêtre.


Voici à quoi ressemble ma classe mère (.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
 
#include <QtGui>
 
class ClasseMere : public QFrame
{
Q_OBJECT
 
public:
ClasseMere(QWidget *parent=0);
virtual ~ClasseMere();
 
void initialiseFenetre();
 
public slot:
virtual void saveAndClose() = 0;
 
protected:
virtual void initialiseContenuFenetre() = 0;
 
//Contient les pointeurs vers les principaux éléments graphiques (Layouts, Widgets, Boutons, ect...)
}

et son .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
 
ClasseMere::ClasseMere(QWidget *parent):QFrame(parent)
{
initialiseFenetre();
 
//bouton close est un pointeur vers le bouton (QPushButton) permettant la fermeture de la fenêtre
QObject::connect(bouton_close, SIGNAL(clicked()), this, SLOT(saveAndClose());
}
 
ClasseMere::~ClasseMere()
{
}
 
ClaseeMere::initialiseFenetre()
{
//Création des éléments graphiques
 
//...
//...
}

Voici les fichiers de la classe fille : (.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
 
class ClasseFille : public ClasseMere
{
Q_OBJECT
 
ClasseFille(QWidget *parent = 0);
virtual ~ClasseFille();
 
public slot:
void saveAndClose(); //Implémente de la classe mère
 
protected:
void initialiseContenuFenetre(); //Implémente de la classe mère
 
//Contient les pointeurs vers les principaux éléments graphiques de la classe fille (Layouts, Widgets, Boutons, ect...)
}

Et son .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
 
ClasseFille::ClasseFille(QWidget *parent):ClasseMere(parent)
{
initialiseContenuFenetre();
}
 
ClasseFille::~ClasseFille()
{
}
 
ClasseFille::initialiseContenuFenetre()
{
//Construction des éléments graphique de la classe fille
 
//...
//...
}
 
ClasseFille::saveAndClose()
{
//Sauvegarde des infos
 
//...
//...
 
this->close();
}

La compilation se déroule bien, l'erreur arrive à l'exécution :

Object::connect: No such slot QFrame::saveAndClose()

2 soucis :

1) Je ne comprends pas pourquoi on cherche le slot saveAndClose() dans la classe QFrame alors que l'élément présent dans mon connect est bien une instance de la ClasseMere.

Rappel :
QObject::connect(bouton_close, SIGNAL(clicked()), this, SLOT(saveAndClose());

2) Peut-être existe-t-il un autre moyen d'arriver à la fin souhaitée (appeler une méthode perso lors de la fermeture de la fenêtre, j'ai tenté d'exploité le signal "destroyed" de QFrame, mais celui-ci survient après la destruction, donc le contenu à exploité est perdu.


Cette méthode me parait la plus propre pour créer plusieurs Classe filles différentes proposant chacune leurs propres éléments. Ici, il ne suffit que d'implémenter la construction de la fenêtre et la façon de sauvegarder les éléments dans la classe fille.

[Niak74]

En testant divers choses, je me suis rendu compte que le connect marche si il est écrit dans la classe fille.

Le problème viendrait peut être du fait que le slot est virtuel pur, Qt cherche peut être alors ce slot dans la/les classes au dessus de ma classe mère...

Ceci ne répond pas tout à fait à mon problème étant donné que je souhaite factoriser un maximum d'éléments dans ma classe mère, cette connexion comprise...

[Altess]

Bonjour.

Je dit peut être une bêtise, mais ton sujet m'intéresse beaucoup
Dans ton code, tu écris :

Code :

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

QObject::connect(bouton_close, SIGNAL(clicked()), this, SLOT(saveAndClose());

Tu indiques l'objet courant "this", donc la classe mère. Or, comme tu l'indiques, ton slot se traite dans ta classe fille. A tu essayer de pointer non pas sur ta classe mère, mais sur ta classe fille avec le connect ? parce que le fait que ta méthode soit virtuel ne suffit peut être pas
Cordialement,
PKO

[yan]

Salut,
Créé plutôt une fonction virtuel appelé par le slot. Un truc comme

Code :

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

1
2
3
4
5
6
7
8
private
virtual void internalSaveAndClose() = 0;
 
public slot:
void saveAndClose()
{
     internalSaveAndClose()
}

ou un fonction init qui fini l'initialisation : A appeler hors du constructeur


si je me trompe pas ton constructeur ne peut marcher car la class fille n'est pas encore créé lors du connect et this est donc encore une ClasseMere :

ClasseMere::ClasseMere(QWidget *parent):QFrame(parent)
{
initialiseFenetre();

//bouton close est un pointeur vers le bouton (QPushButton) permettant la fermeture de la fenêtre
QObject::connect(bouton_close, SIGNAL(clicked()), this, SLOT(saveAndClose());
}

EN gros ce qui se passe quand tu créé une class fille :
1- la partie class QFrame est créé
2- la partie class mère est créé et tu essaie de faire un connect
3- la partie class fille est créé.

Le connect est appliqué sur la class mère et non la class fille car sa partie n'est pas encore créé. Et comme la class mère n'as pas de slot saveAndClose() il essaie d'utiliser celle de QFrame qui n'existe pas

[Niak74]

J'ai modifié un peu le code de la classe mère :

Code :

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

1
2
3
4
5
6
7
8
9
 
protected:
virtual void saveAndClose() = 0;
 
public slot:
void saveAndCloseSlot()
{
     saveAndClose();
}

Avec création des connections dans le constructeur : même erreur (impossible de trouver le slot QFrame::saveAndCloseSlot()).

En déportant la création des connexion dans une méthode qui est ensuite appelée dans le main() : erreur : impossible de trouver le slot ClasseFille::saveAndCloseSlot().

Je vais tester un petit truc...

[Niak74]

J'ai une réponse partielle au problème :

Classe mère :

- Créer le slot public virtuel pur saveAndClose()
- Créer la méthode public makeConnections(), dans le cpp, établir les liens avec le slot saveAndClose()

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
 
///////////////////////
     Header
///////////////////////
 
public slot :
virtual void saveAndClose() = 0;
 
public: 
void makeConnections()
 
///////////////////////
     Cpp
///////////////////////
 
ClasseMere::makeConnections()
{
QObject::connect(bouton_fermer, SIGNAL(clicked()), this, SLOT(saveAndClose());
}

Classe fille :

- implémenter le slot saveAndClose()

Dans le main :

- créer une instance de ClasseFille
- appeler la méthode makeConnections de cette instance


Cette méthode fonctionne. Cependant, elle me semble partielle car une ligne est à ajouter dans le main pour établir les connexions. Je m'en contenterai si il est impossible de faire mieux.

Merci à vous pour les idées proposées =)

[deebou]

je suis étonné de ne pas avoir entendu parler de moccing dans cette histoire...
Pour que tout ca marche, ne faut-il pas que moc soit appelé sur les classes mères et filles????

Je suis pas un spécialiste, hein. C'est plus une question qu'une piste...

[yan]

je suis étonné de ne pas avoir entendu parler de moccing dans cette histoire...
Pour que tout ca marche, ne faut-il pas que moc soit appelé sur les classes mères et filles????

Je suis pas un spécialiste, hein. C'est plus une question qu'une piste...

Il as bien mis QOBJECT en début de class et na pas d'erreur de compilation donc je ne pense pas que cela soit un problème de moc

Ryo2468 peut tu faire un exemple compilable de ton problème? je regarderais demain

[Niak74]

Voici un exemple simplifié autant que possible :

main.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
 
#include <QtGui/QApplication>
#include "mainwindow.h"
 
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
 
    MainWindow w;
    w.finalizeInit();
    w.showFullScreen();
 
    a.connect( &a, SIGNAL(lastWindowClosed()), &a, SLOT(quit()) );
 
    return a.exec();
}

Classe mère alias ConfigFormPattern, header :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
#ifndef CONFIGFORMPATTERN_H
#define CONFIGFORMPATTERN_H
 
#include <QtGui>
#include <QProcess>
 
class ConfigFormPattern : public QFrame
{
    Q_OBJECT
 
    //------------------------------//
    //     To be used by Childs     //
    //------------------------------//
 
public:
    //Finalization : put configuration form and create connections, to be called AFTER instanciation of a child
    void finalizeInit();
public slots:
    //Describes how to quit configuration application and write settings, to be implemented by childs
    virtual void saveAndClose() = 0;
protected:
    //Configuration form container, to be used by childs
    QWidget* configurationForm;
    //Describes how to build configuration form container, to be implemented by childs
    virtual void initConfigurationForm() = 0;
 
    //--------------------------//
    //     Common elements      //
    //--------------------------//
 
public:
    ConfigFormPattern(QWidget *parent = 0);
    virtual ~ConfigFormPattern();
 
protected:
    //Methods
    void initCommonItems();
 
    //Layouts
    QVBoxLayout * layout_main;
    QHBoxLayout * layout_modules;
    QHBoxLayout * layout_configuration;
    //Buttons
    QPushButton * btn_quit;
};
 
#endif // CONFIGFORMPATTERN_H

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
#include "configformpattern.h"
 
ConfigFormPattern::ConfigFormPattern(QWidget *parent)
    : QFrame(parent)
{
    configurationForm = 0;
 
    //Initializes form
    initCommonItems();
}
 
ConfigFormPattern::~ConfigFormPattern()
{
}
 
void ConfigFormPattern::initCommonItems()
{
    //Set spacings
    layout_main = new QVBoxLayout;
    layout_main->setContentsMargins(5,5,5,5);
    layout_main->setSpacing(4);
 
    layout_configuration = new QHBoxLayout;
    layout_configuration->setContentsMargins(0,0,0,0);
    layout_configuration->setSpacing(0);
 
    layout_modules = new QHBoxLayout;
    layout_modules->setContentsMargins(0,0,0,0);
    layout_modules->setSpacing(2);
 
    //Close button
    btn_quit = new QPushButton("Close");
    btn_quit->setFixedSize(30,30);
 
    layout_modules->addStretch();
    layout_modules->addWidget(btn_quit);
 
    //Set main layout
    layout_main->addLayout(layout_configuration);
    layout_main->addLayout(layout_modules);
 
    this->setLayout(layout_main);
}
 
void ConfigFormPattern::finalizeInit()
{
	//Make connections
    QObject::connect(btn_quit, SIGNAL(clicked()), this, SLOT(saveAndClose()));
 
    //Put configuration form
    if(configurationForm == 0)
        return;
	else
	    layout_configuration->addWidget(this->configurationForm);
}

Classe fille alias MainWindow, header :

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
28
29
 
#ifndef MAINWINDOW_H
#define MAINWINDOW_H
 
#include <QtGui>
#include "configformpattern.h"
 
class MainWindow : public ConfigFormPattern
{
    Q_OBJECT
 
public:
    MainWindow(QWidget *parent = 0);
    virtual ~MainWindow();
 
private:
    void writeConfSettings();
 
    //-----------------//
    //     HERITE     //
    //-----------------//
 
protected:
    void initConfigurationForm();
public slots:
    void saveAndClose();
};
 
#endif // MAINWINDOW_H

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
28
29
30
31
32
33
34
35
36
37
38
 
#include "mainwindow.h"
 
MainWindow::MainWindow(QWidget *parent)
    : ConfigFormPattern(parent)
{
    //Initializes configuration view
    initConfigurationForm();
}
 
MainWindow::~MainWindow()
{
}
 
void MainWindow::writeConfSettings()
{
    qDebug("WRITING CONFIGURATION");
}
 
void MainWindow::saveAndClose()
{
    writeConfSettings();
    this->close();
}
 
void MainWindow::initConfigurationForm()
{
    this->configurationForm = new QWidget;
 
    QHBoxLayout * layout_main = new QHBoxLayout;
 
    QLabel * label_title = new QLabel("Test");
    label_title->setFont(QFont("Arial",12,QFont::Bold));
 
    layout_main->addWidget(label_title);
 
    this->configurationForm->setLayout(layout_main);
}

et le .pro :

Code :

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

1
2
3
4
5
6
7
8
 
TARGET = projetTest
TEMPLATE = app
SOURCES += configformpattern.cpp \
main.cpp \
mainwindow.cpp
HEADERS += configformpattern.h \
mainwindow.h

Dans l'ensemble, ça fonctionne (c'est à dire l'appel à la méthode saveAndClose() défini par la classe fille lors de la fermeture de l'application). Le "petit soucis" reste cette ligne à ajouter dans le main après la construction de l'objet pour établir les connexions :

Code :

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

1
2
3
4
 
MainWindow w;
    w.finalizeInit();
    w.showFullScreen();

Si il y a un moyen de s'en passer, je prends, sinon il faudra que j'insiste particulièrement dans la documentation du code pour que les futurs utilisateurs de mon patron de classe n'ai pas un soucis avec leurs connexions ^^

[IrmatDen]

Sachant que le moc s'appuie pas mal sur des chaînes de caractères, tu as déjà tenté en préfixant la déclaration de ton slot dans MainWindow par "virtual" ?

[Niak74]

Ça marche en ajoutant le mot clé "virtual" devant le saveAndClose de la méthode fille. Je peux alors retirer la ligne en plus du main et faire appel aux connexions dans le constructeur de la classe mère.

Je suis somme toute curieux de comprendre ce qu'il se passe :o

[IrmatDen]

Sachant que le moc s'appuie pas mal sur des chaînes de caractères

[yan]

du coup, ce que j'ai mis ici:
http://www.developpez.net/forums/d69...e/#post4051511

est faux?

[deebou]

ouais...
je serais curieux de voir le contenu des fichiers moc avec et sans le "virtual"...
On pourrait croire que moc attend l'identité complète des prototypes des slots, y compris les préfixes...

On pourrait vérifier cela avec du "inline" par exemple, non?

[Niak74]

Vérifier avec du inline?

Je suis débutant, n'allez pas trop vite