QObject::connect ne fonctionne pas

Version imprimable

Bonjour,

J'ai le code suivant, dans le constructeur de l'une de mes classes (simplifié):
Code:

1 2 3 4 5 QPushButton entrer = new QPushButton; QComboBox langues = new QComboBox; QObject::connect(entrer, SIGNAL(clicked()), this, SLOT(addToInputStack())); QObject::connect(langues, SIGNAL(currentIndexChanged(int)), this, SLOT(changeLang(int)));
Et à l'exécution, j'ai ces erreurs:

Citation:

Object::connect: No such slot QObject::addToInputStack() in console.cpp:18
Object::connect: No such slot QObject::changeLang(int) in console.cpp:24

Or, ceci est dans une classe dérivée de QObject, implémentant les deux fonctions.

Pourquoi la méthode connect va-t-elle chercher les fonctions dans QObject et pas dans ma classe? (nommée Console)

Merci d'avance,
ProgVal

EDIT: et d'après le débogueur, à la ligne où est présent connect, this pointe bien vers un objet de type Console

23/08/2009, 12h31
Alp

Déjà, pas la peine de préfixer par QObject:: si tu te trouves dans une classe qui en hérite (si elle hérite de QWidget, elle hérite forcément de QObject).

Ensuite, tu as bien passé ton fichier .h à la moulinette du moc ?
23/08/2009, 12h35
ProgVal

Citation:

Envoyé par Alp

Déjà, pas la peine de préfixer par QObject:: si tu te trouves dans une classe qui en hérite (si elle hérite de QWidget, elle hérite forcément de QObject).

Ok. Ma classe hérite directement de QObject.

Citation:

Envoyé par Alp

Ensuite, tu as bien passé ton fichier .h à la moulinette du moc ?

C'est à dire? (Google n'est pas très explicite)
23/08/2009, 12h36
dourouc05

Citation:

Envoyé par ProgVal

C'est à dire? (Google n'est pas très explicite)

Le moc est un outil qui permet de faire accepter à ton compilateur C++ les nouveautés de Qt : les signaux, les slots...

Si tu utilises qmake pour la compilation, alors tu utilises forcément le moc. Sinon, passe à qmake !
23/08/2009, 12h38
Alp

Pour les signaux/slots et qmake/moc :
http://matthieu-brucher.developpez.c...t/signal-slot/
http://qt.developpez.com/faq/?page=qt4Core#signalslot
23/08/2009, 12h44
ProgVal

J'utilise le compilateur de Qt Creator (ça devrait être le qmake, normalement ^^)
J'ai vérifié en lançant qmake à partir de la console => aucun changement

EDIT: j'avais mis mes slots en temps que public; je les ai mis en public slots, mais aucun changement.
23/08/2009, 12h49
Alp

Citation:

Envoyé par ProgVal

J'utilise le compilateur de Qt Creator (ça devrait être le qmake, normalement ^^)
J'ai vérifié en lançant qmake à partir de la console => aucun changement

EDIT: j'avais mis mes slots en temps que public; je les ai mis en public slots, mais aucun changement.

Est-ce que tu as bien :

Code:

Q_OBJECT

au tout début de la déclaration de ta classe ?

EDIT : lire ceci pourrait t'aider.
23/08/2009, 12h50
ProgVal

J'ai oublié de précisé que je l'avais ajouté en même temps que public slot, donc oui, il y est bien.
23/08/2009, 12h51
Alp

Bon. Tu peux nous c/c ton .h avec donc la déclaration de ta classe ? Ainsi que les commandes que tu exécutes pour compiler.

Déclaration de la classe:

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
#ifndef CONSOLE_H
 #define CONSOLE_H
  class Console : QObject
 {
     Q_OBJECT
     public:
         Console(QApplication*, QStringList, QStringList);
         int push(QString);
         QString getInputStack();
     public slots:
         int changeLang(int);
         QString addToInputStack();
     private:
         QWidget fenetre;
         QGridLayout *layout;
         QPlainTextEdit *sorties;
         QLineEdit *entree;
         QPushButton *entrer;
          QString inputStack;
         QComboBox *langues;
         QStringList idLangues;
         QTranslator translator;
         QApplication* app;
 };
  #endif // CONSOLE_H

Et pour les commandes... Ctrl + R (équivalent au bouton "run")

23/08/2009, 12h58
Alp

Rajoute public devant QObject, juste après "class Console :".

Il se peut que ça joue.

Comme ça?

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
#ifndef CONSOLE_H
#define CONSOLE_H
 
class Console : QObject
{
    public:
        Q_OBJECT
        Console(QApplication*, QStringList, QStringList);
        int push(QString);
        QString getInputStack();
    public slots:
        int changeLang(int);
        QString addToInputStack();
    private:
        QWidget fenetre;
        QGridLayout *layout;
        QPlainTextEdit *sorties;
        QLineEdit *entree;
        QPushButton *entrer;
 
        QString inputStack;
        QComboBox *langues;
        QStringList idLangues;
        QTranslator translator;
        QApplication* app;
};
 
#endif // CONSOLE_H

Aucun changement

23/08/2009, 13h01
Alp

Non non.
Tranformer ça :

Code:

class Console : QObject

en

Code:

class Console : public QObject
23/08/2009, 13h02
ProgVal

Aucun changement.
23/08/2009, 13h06
Alp

Tant qu'on y est, mets aussi le .cpp (et laisse le public devant QObject).

Ca doit être le connect qui déraille :koi:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <QtGui>
 #include "console.h"
 Console::Console(QApplication *application, QStringList shortIdLanguesDisponibles, QStringList nomLanguesDisponibles) : QObject(), app(application)
 {
     // ...
    entrer = new QPushButton;
     entrer->setText(tr("Send", "Send the typed text to the input stack"));
     entrer->setToolTip(tr("Send the typed text to the input stack", "ToolTip of the 'Send' button"));
      // ...
    QObject::connect(entrer, SIGNAL(clicked()), this, SLOT(addToInputStack()));
      langues = new QComboBox;
     langues->addItems(nomLanguesDisponibles);
     // ...
     QObject::connect(langues, SIGNAL(currentIndexChanged(int)), this, SLOT(changeLang(int)));
      // ...
}

23/08/2009, 13h13
Alp

Et en mettant

Code:

connect(/* etc */)

à la place de

Code:

QObject::connect(/* etc */)

?

Et si ça ne marche toujours pas, fais un "Clean All" puis "Build All" ou alors "Rebuild All".
23/08/2009, 13h17
ProgVal

Citation:

p, li { white-space: pre-wrap; } debug/moc_console.cpp:12:2: erreur: #error "The header file 'console.h' doesn't include <QObject>."
In file included from debug/moc_console.cpp:10:
debug/../console.h:5: erreur: expected class-name before ‘{’ token
debug/../console.h:6: erreur: ISO C++ forbids declaration of ‘Q_OBJECT’ with no type
debug/../console.h:7: erreur: expected ‘;’ before ‘public’
debug/../console.h:9: erreur: ‘QString’ has not been declared
debug/../console.h:10: erreur: ‘QString’ does not name a type
debug/../console.h:11: erreur: expected `:' before ‘slots’
debug/../console.h:12: erreur: expected primary-expression before ‘int’
debug/../console.h:12: erreur: ISO C++ forbids declaration of ‘slots’ with no type
debug/../console.h:12: erreur: expected ‘;’ before ‘int’
debug/../console.h:13: erreur: ‘QString’ does not name a type
debug/../console.h:15: erreur: ‘QWidget’ does not name a type
debug/../console.h:16: erreur: ISO C++ forbids declaration of ‘QGridLayout’ with no type
debug/../console.h:16: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:17: erreur: ISO C++ forbids declaration of ‘QPlainTextEdit’ with no type
debug/../console.h:17: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:18: erreur: ISO C++ forbids declaration of ‘QLineEdit’ with no type
debug/../console.h:18: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:19: erreur: ISO C++ forbids declaration of ‘QPushButton’ with no type
debug/../console.h:19: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:21: erreur: ‘QString’ does not name a type
debug/../console.h:22: erreur: ISO C++ forbids declaration of ‘QComboBox’ with no type
debug/../console.h:22: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:23: erreur: ‘QStringList’ does not name a type
debug/../console.h:24: erreur: ‘QTranslator’ does not name a type
debug/../console.h:25: erreur: ISO C++ forbids declaration of ‘QApplication’ with no type
debug/../console.h:25: erreur: expected ‘;’ before ‘*’ token
debug/moc_console.cpp:20: erreur: expected constructor, destructor, or type conversion before ‘static’
debug/moc_console.cpp:43: erreur: ‘QMetaObject’ does not name a type
debug/moc_console.cpp:48: erreur: expected initializer before ‘*’ token
debug/moc_console.cpp:53: erreur: no ‘void* Console::qt_metacast(const char*)’ member function declared in class ‘Console’
debug/moc_console.cpp:61: erreur: ‘int Console::qt_metacall’ is not a static member of ‘class Console’
debug/moc_console.cpp:61: erreur: ‘QMetaObject’ has not been declared
debug/moc_console.cpp:61: erreur: expected primary-expression before ‘int’
debug/moc_console.cpp:61: erreur: expected primary-expression before ‘void’
debug/moc_console.cpp:61: erreur: initializer liste d'expressions traitée comme une expression composée
debug/moc_console.cpp:62: erreur: expected ‘,’ or ‘;’ before ‘{’ token
make[1]: *** [debug/moc_console.o] Erreur 1
23/08/2009, 13h19
Alp

Et si tu inclues QObject dans console.h ?
23/08/2009, 13h26
ProgVal

Si je l'inclue, j'ai ça:

Citation:

In file included from debug/moc_console.cpp:10:
debug/../console.h:9: erreur: expected `)' before ‘*’ token
debug/../console.h:16: erreur: field ‘fenetre’ has incomplete type
debug/../console.h:17: erreur: ISO C++ forbids declaration of ‘QGridLayout’ with no type
debug/../console.h:17: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:18: erreur: ISO C++ forbids declaration of ‘QPlainTextEdit’ with no type
debug/../console.h:18: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:19: erreur: ISO C++ forbids declaration of ‘QLineEdit’ with no type
debug/../console.h:19: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:20: erreur: ISO C++ forbids declaration of ‘QPushButton’ with no type
debug/../console.h:20: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:23: erreur: ISO C++ forbids declaration of ‘QComboBox’ with no type
debug/../console.h:23: erreur: expected ‘;’ before ‘*’ token
debug/../console.h:24: erreur: field ‘idLangues’ has incomplete type
debug/../console.h:25: erreur: ‘QTranslator’ does not name a type
debug/../console.h:26: erreur: ISO C++ forbids declaration of ‘QApplication’ with no type
debug/../console.h:26: erreur: expected ‘;’ before ‘*’ token

EDIT: la ligne 9 de console.h est:

Code:

Console(QApplication*, QStringList, QStringList);
23/08/2009, 13h46
Alp

Ah... Il faudrait que tu inclues tous les fichiers relatifs aux composants graphiques que tu utilises, comme l'entête <QComboBox> par exemple.
23/08/2009, 13h47
ProgVal

#include <QtGui> ne suffit pas?

Parce que avant de faire Clean up et Rebuild all, ça marchait...
23/08/2009, 14h04
Alp

Citation:

Envoyé par ProgVal

#include <QtGui> ne suffit pas?

Parce que avant de faire Clean up et Rebuild all, ça marchait...

Non, QtGui ne contient pas toutes les classes. T'imagines, t'as juste besoin d'un bouton, et tu te retrouves avec le code de toutes les classes de GUI de Qt ? Exécutable super gros pour rien.
23/08/2009, 14h10
ProgVal

Euh, tu vas rire, mais, quand je rajoute le #include <QtGui> dans mon .h, ça remarche...

Et puis, le compilateur fait le tri entre ce que l'on doit mettre dans l'exécutable ou non, non?

EDIT: et je n'ai plus les erreurs de runtime non plus...
Par contre, le programme ne réagit toujours pas lorsque je change l'élément sélectionné dans la QComboBox
23/08/2009, 14h12
Alp

Citation:

Envoyé par ProgVal

Euh, tu vas rire, mais, quand je rajoute le #include <QtGui> dans mon .h, ça remarche...

Et puis, le compilateur fait le tri entre ce que l'on doit mettre dans l'exécutable ou non, non?

Ah non. Il ne va pas regarder pour chaque classe si y'a besoin de mettre effectivement dans ton exécutable. Les compilos ne sont pas encore poussés à ce point.

Bon du coup, ça marche ? Tes slots sont bien connectés ? Ils sont appelés ?

PS : par contre, <QtGui> prédéclare toutes les classes, donc si tu n'utilises que des pointeurs, ça peut effectivement suffir.
23/08/2009, 14h13
ProgVal

Oups, doublé

Citation:

Envoyé par ProgVal

EDIT: et je n'ai plus les erreurs de runtime non plus...
Par contre, le programme ne réagit toujours pas lorsque je change l'élément sélectionné dans la QComboBox
23/08/2009, 14h27
Alp

Tu es sûr que le slot est bien exécuté ?
Pour le savoir, il te suffit de lancer une messagebox depuis l'intérieur de ce slot, ou bien de faire std::cout << "Slot exécuté !" << std::endl ou quelque chose de ce genre quoi.
23/08/2009, 14h29
ProgVal

Il est exécuté.

Donc, le bug n'a rien à voire avec les slots. => Problème résolu.

Merci beaucoup pour ton aide ;)
08/09/2009, 23h15
geek-59600

J'vais peut être dire une grosse bêtise mais les slots ne doivent ils pas être de type void ?
08/09/2009, 23h22
Alp

Je crois que leur valeur de retour est ignorée, s'ils ne retournent pas void (i.e : ça compile mais ça ne change rien par rapport à s'ils retournaient void).