Problème avec les slots

Version imprimable

09/02/2011, 16h50
Far_Away

Problème avec les slots
Bonjour,

J'ai un petit problème pour récupérer certaines valeurs de QLineEdit

Imaginons que j'ai une classe fenetre simple avec dedans un bouton B et un QLineEdit Q

j'ai mis un connect sur mon bouton B de la facon suivante
Code:

1 2 connect(B, SIGNAL(clicked()), this, SLOT(fonction(Q.text())));
le membre fonction devrait juste afficher le QString récupéré dans un QTextBrowser.

Et voilà, quand je tape quelquechose dans mon QLineEdit, puis que je clic sur mon bouton, il ne se passe rien :(

Comprends pas :(

N.B. Je sais bien que je pourrai faire une fonction sans arguments et à l'intérieur récupérer le string du QLineEdit. Mais j'aimerais voir pourquoi cela ne marche pas avec les arguments comme cela.

Merci de me dire d'ou vient le souci :ccool:
09/02/2011, 17h27
superjaja

La signature d'un signal et d'un slot doit etre identique donc ce que tu fais ne peux pas fonctionner.

Pour plus de precision merci de lire la FAQ ici.

Bonjour,

Voici un exemple de programme ,compilable, qui pourrait t'aider :

programme.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
 
#include "taclasslib.h"
#include <QApplication>
#include <QPushButton>
//#include <QGridLayout>
//#include <QWidget>
 
int main(int argc,char ** argv)
{	QApplication app(argc,argv) ;
	QLineEdit * lignemots = new QLineEdit ;
	QPushButton * bouton = new QPushButton("Validation") ;
	Taclasslib objet ;
	QLabel * label = new QLabel ;
	objet.initialisation() ;
	lignemots->show() ;
	bouton->show() ;
	label->setFixedSize(400,250) ;
	label->show() ;
	objet.ajusteWidgets(lignemots,label) ;
	QObject::connect(bouton,SIGNAL(clicked()),&objet,SLOT(unefonction()) ) ;
app.exec() ;
return 0 ;
}

taclasslib.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
#ifndef _TACLASSLIB_H_
#define _TACLASSLIB_H_
 
#include <QLineEdit>
#include <QLabel>
#include <QString>
#include <QObject>
class Taclasslib : public QObject
{	Q_OBJECT	
    public :
	void initialisation()
	{ m_mots = new QLineEdit ; entier = 0 ; l_label = new QLabel ; }
 
	void ajusteWidgets(QLineEdit * mots,QLabel * label)
	{   m_mots = mots ; l_label = label ; }
 
      public slots :
	void unefonction()
	{    motqstr = m_mots->text() + "  (" + QString::number(entier) + ") " ; 
	        l_label->setText(motqstr) ;
		l_label->update() ;
	entier++ ;	    
	}
 
  private :    
  int entier ;
  QLabel * l_label ;
  QLineEdit * m_mots ;  
  QString motqstr ;  
};
 
#endif

...

10/02/2011, 15h50
Far_Away

Ok donc il faut le même nombre d'arguments pour signal et slot. Cela peut etre handicapant, en tout cas, jtrouve que c'est un leak...
Sur qu'il y a moyen de contourner.

Merci pour le code
11/02/2011, 09h25
ymoreau

Tu peux mettre un argument dans ton slot avec une valeur par défaut, du coup le signal sans argument appellera le slot avec la valeur d'argument par défaut. Mais ça ne permet pas d'utiliser une valeur non statique.

Après dans ton cas je ne trouve pas que récupérer la valeur dans le corps de la fonction soit handicapant, et je n'ai jamais eu besoin de faire ce que tu comptais faire.
11/02/2011, 10h35
gbdivers
Pour rappel, il n'est pas nécessaire d'avoir exactement le même nombre d'argument. Le slot peut ignorer des arguments qu'il n'a pas besoin. Et c'est exactement le cas avec clicked puisque la signature de clicked est :

Code:

void clicked (bool);

Donc :
Code:

1 2 connect(this, SIGNAL(signal_sans_bool()), this, SLOT(slot_bool(bool))); // erreur connect(this, SIGNAL(signal_bool(bool)), this, SLOT(slot_sans_bool())); // OK
Citation:

Ok donc il faut le même nombre d'arguments pour signal et slot. Cela peut etre handicapant, en tout cas, jtrouve que c'est un leak...

Ca dépend du point de vue (comme beaucoup de chose)...

Dans :

Code:

connect(B, SIGNAL(clicked()), this, SLOT(fonction(Q.text())));

Tu voudrais en fait que lorsque tu cliques sur un bouton, ça envoie également le texte contenu dans un autre widget. En terme de conception objet, ça n'est pas du tout naturel et intuitif.

Un objet qui doit aller chercher les informations dans un autre objet totalement indépendant, c'est une mauvaise approche.
Un objet qui envoie un signal à un parent qui va chercher des informations dans un autre objet enfant, ça c'est une bonne approche.

Bref, comme on te l'a dit, il faut un slot sans paramètre qui ira chercher le texte
11/02/2011, 21h09
Far_Away

Ok je vois. Tu me fais voir comment penser objet. Je t'en remercie, ca va m'aider a mieux coder :)
C'est vrai que j'ai pour habitude de faire la première chose qui me passe par la tête et n'étant pas du tout un gros programmeur, je le fais par nécessité, je n'ai jamais pris le temps de penser correctement mon code :cry: