Discussion :

[tymop]

Bonjour,
Je cherche à créer un menu dans lequel je met une liste dont chaque clic sur un élément appellera une fonction qui a un QString en paramètre.

Code :

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

1
2
3
4
5
6
7
8
foreach( QextPortInfo item, liste_ports )
     {
        QAction *action = new QAction(item.physName, this);
        action->setVisible(false);
        ui->menuPort->addAction(item.physName);
     }
     connect(ui->menuPort, SIGNAL(triggered(QAction *)),
             this, SLOT(init_port(QAction.text() *)), Qt::UniqueConnection);

Ce qui est inscrit ci-dessus ne marche pas. Et je pense que le problème vient d'ici : SLOT(init_port(QAction.text() *))
Seulement je ne trouve pas comment écrire le fait que init_port() soit appelé en lui donnant le bon paramètre.

Sauriez vous ce que je devrais mettre ?
Merci d'avance de votre aide.

[Invité]

Bonsoir,

C'est faisable avec la classe QSignalMapper :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
QSignalMapper *signalMapper = new QSignalMapper(this);
foreach( QextPortInfo item, liste_ports )
{
    QAction *action = new QAction(item.physName, this);
    action->setVisible(false);
    ui->menuPort->addAction(item.physNameaction);
    connect(action, SIGNAL(triggered()), signalMapper, SLOT(map()));
    signalMapper->setMapping(action, item.physName);
}
connect(signalMapper, SIGNAL(mapped(const QString &)),
        this,         SLOT(init_port(const QString &)) );

[EDIT] Correction item.physName => action

[tymop]

Merci pour l'aide.
En revanche, j'ai toujours une erreur à l'execution:

Code :

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

Object::connect: No such slot MainWindow::init_port(const QString &) in ../test_led_arduino/mainwindow.cpp:51

Je l'ai pourtant déclarée ainsi :

Code :

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

void MainWindow::init_port(const QString &dev_port)

[Gojir4]

Je pense qu'il ne faut pas indiquer le const et le & lors de la connexion signal/slot:

Code :

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

1
2
connect(signalMapper, SIGNAL(mapped(QString)),
this, SLOT(init_port(QString)) );

[Invité]

C'est je pense surtout dû au fait que dans le premier code présenté, le slot init_port prend quelque chose de bizarre SLOT(init_port(QAction.text() *)) qui n'est sûrement pas un QString.

@tymop : As-tu pensé à créer un tel slot dans ta classe MainWindow avec la signature suivante

Code :

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

void init_port(QString const & physName);

?

[tymop]

Bonsoir,

As-tu pensé à créer un tel slot dans ta classe MainWindow avec la signature suivante

Non, je l'ai déclaré comme marqué plus haut void MainWindow::init_port(const QString &dev_port)

J'ai aussi essayé avec

Code :

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

connect(signalMapper, SIGNAL(mapped(QString)),this, SLOT(init_port(QString)) );

mais ça ne marche pas :

Code :

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

Object::connect: No such slot MainWindow::init_port(QString) in ../test_led_arduino/mainwindow.cpp:51

Note : j'ai essayé la déclaration "init_port(QString const & dev_port)" mais c'est pareil

[Invité]

Non, je l'ai déclaré comme marqué plus haut void MainWindow::init_port(const QString &dev_port)

Cela me semble bon, pourrais-tu nous en montrer plus ?

Un nettoyage/recompilation est peut-être nécessaire..

[tymop]

Un nettoyage/recompilation est peut-être nécessaire..

je recompile à chaque modification. qu'est le Nettoyage ?

pourrais-tu nous en montrer plus ?

voici la fonction init_port

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
 
void MainWindow::init_port(QString const & dev_port)
{
    //QString dev_port = "/dev/ttyUSB0";
    qDebug() << "Port :" << dev_port ;
    COM_Open = false;
 
    port = new QextSerialPort(dev_port); //on ouvre le port
    // ls /dev/ | grep USB   << Permet de donner la liste des périphériques USB
 
     port->open(QIODevice::ReadWrite | QIODevice::Unbuffered);
    if(!port->isOpen())
    {
       QMessageBox::warning(this, "Impossible d'ouvrir le port", dev_port);
    }
 
    // On parametre la liaison :
    port->setBaudRate(BAUD9600);
    port->setFlowControl(FLOW_OFF);
    port->setParity(PAR_NONE);
    port->setDataBits(DATA_8);
    port->setStopBits(STOP_1);
 
    connect(port, SIGNAL(readyRead()), this, SLOT(onDataAvailable()));
    COM_Open = true;
 
}

et le code ou le menu est créé et ou je cherche à faire la connection

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
MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    Val1=0;
    Val2=0;
    Val3=0;
    temps=0;
    timer = new QTimer(this);
 
    ui->setupUi(this);
 
    // On connecte le mouvement du bouton a l'envoi de la commande
    //connect(ui->Slider1, SIGNAL(sliderMoved(int)), this, SLOT(transmitCmd(int)));
    connect(ui->Slider1, SIGNAL(sliderMoved(int)), this, SLOT(MAJ_Val1(int)));
    connect(ui->Slider2, SIGNAL(sliderMoved(int)), this, SLOT(MAJ_Val2(int)));
    connect(ui->Slider3, SIGNAL(sliderMoved(int)), this, SLOT(MAJ_Val3(int)));
    connect(ui->checkBoxAuto, SIGNAL(stateChanged(int)), this, SLOT(Conf_timer()));
    connect(ui->vitesse_timer, SIGNAL(sliderMoved(int)), this, SLOT(Conf_timer()));
    connect(ui->dial, SIGNAL(sliderMoved(int)), this, SLOT(set_Temps(int)));
    connect(timer, SIGNAL(timeout()), this, SLOT(effet()));
    timer->start(ui->vitesse_timer->value());
 
    // Menu liste des portcom
    QList<QextPortInfo> liste_ports = QextSerialEnumerator::getPorts();
    QSignalMapper *signalMapper = new QSignalMapper(this);
    foreach( QextPortInfo item, liste_ports )
    {
        QAction *action = new QAction(item.physName, this);
        action->setVisible(false);
        ui->menuPort->addAction(item.physName);
        connect(action, SIGNAL(triggered()), signalMapper, SLOT(map()));
        signalMapper->setMapping(action, item.physName);
    }
    connect(signalMapper, SIGNAL(mapped(QString)),this, SLOT(init_port(QString)) );
 
 
 //   init_port("/dev/ttyUSB0"); //initiation of the port
 //   connect(port, SIGNAL(readyRead()), this, SLOT(onDataAvailable()));
}

S'il y a d'autres bouts de codes qui intéressent, dites le .

PS : soyez indulgent, c'est mon premier code qt

[Invité]

Le nettoyage est la suppression de tous les fichiers générés lors de l'étape de la compilation. Si tu travailles sous Qt Creator, c'est dans le même menu que lorsque tu compiles : "Compiler" > "Nettoyer le projet".

Le code que tu montres semble bon... si dans ton fichier mainwindow.h tu as bien déclaré ton slot comme suit :

Code :

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

1
2
3
4
5
6
7
8
class MainWindow {
 
    // ...
 
public slots:
    void init_port(QString const & dev_port);
 
};

Alors cela devrait être bon.

[tymop]

si dans ton fichier mainwindow.h tu as bien déclaré ton slot comme suit :

Code :

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

1
2
3
4
5
6
7
8
class MainWindow {
 
    // ...
 
public slots:
    void init_port(QString const & dev_port);
 
};

Alors cela devrait être bon.

Bon, tu as mis le doit sur quelque chose : j'avais mis dans "public" et non dans "public slot".
Je n'ai plus le message d'erreur, mais ce n'est pas fonctionnel : la fonction n'est pas appelée lorsque je clic sur les éléments qui le composent

[tymop]

Je ne trouve pas du tout. Pouvez vous me donner un coup de main pour comprendre:

Est ce qu'on peut dire que cette ligne de code appelle la fonction init_port en lui donnant un paramètre de type QString, lorsqu'on clic sur un des choix dans le menu menuPort, et que ce QString est le texte de la ligne sur laquelle on a cliqué ?

Code :

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

connect(signalMapper, SIGNAL(mapped(QString)),this, SLOT(init_port(QString)) );

Si c'est ça, ça ne marche pas : j'ai mis un point d'arrêt au début de la fonction init_port, et jamais je ne l'atteint.

Si j’interprète mal, pouvez vous me dire ce que ce code fait ?

Merci d'avance de votre aide.

[Invité]

Oui c'est exactement ça, par contre, chose à laquelle je n'avais pas prêté attention, tu n'ajoutes pas le QAction que tu viens de créer et lier, mais juste le texte, ce qui en crée derrière un totalement différent.

Code :

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

ui->menuPort->addAction(item.physNameaction);

[tymop]

C'était bien cela.
Merci pour ton aide.