Discussion :

[VivienD]

Bonsoir,

J'ai créé un petit projet avec Qt Creator pour apprendre à manipuler les threads avec QThread. Ce projet qui n'a qu'un but ludique consiste à créer une sorte de métronome dans un thread différent du thread principal. Jusque là, rien d'anormal.

Voici le fichier 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
17
18
19
#include <QApplication>
 
#include "mygui.h"
#include "myclockwork.h"
 
int main(int argc, char* argv[])
{
    QApplication app(argc, argv);
 
    MyGui gui;
    MyClockwork clockwork;
    app.connect(&clockwork, SIGNAL(tick()), &gui, SLOT(tick()));
    app.connect(&app, SIGNAL(aboutToQuit()), &clockwork, SLOT(stop()));
 
    gui.show();
    clockwork.start();
 
    return app.exec();
}

Comme on peut le voir, le métronome (classe MyClockwork) possède deux slots publics, start() et stop(), ainsi qu'un signal tick(). Par ailleurs, le slot stop() est appelé quand l'application est sur le point de quitter; le problème réside là.
Voici les fichiers liés à la classe du métronome:

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
#ifndef MYCLOCKWORK_H
#define MYCLOCKWORK_H
 
#include <QObject>
 
#include <QApplication>
#include <QThread>
#include <QTimer>
 
class MyClockwork : public QObject
{
    Q_OBJECT
 
    public:
        MyClockwork(QObject* parent = 0);
        ~MyClockwork();
 
    public slots:
        void start();
        void stop();
 
    signals:
        void tick();
 
    private:
        QThread* m_thread;
        QTimer* m_timer;
 
    private slots:
        void timerFired();
};
 
#endif // MYCLOCKWORK_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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
#include "myclockwork.h"
 
MyClockwork::MyClockwork(QObject* parent) :
    QObject     (parent)        ,
    m_thread    (new QThread)   ,
    m_timer     (new QTimer)
{
    m_timer->setSingleShot(false);
}
 
MyClockwork::~MyClockwork()
{
    delete m_timer;
    delete m_thread;
}
 
void MyClockwork::start()
{
    m_timer->setInterval(1000);
    connect(m_timer, SIGNAL(timeout()), this, SLOT(timerFired()), Qt::DirectConnection);
    m_timer->start();
    m_timer->moveToThread(m_thread);
    m_thread->start();
}
 
void MyClockwork::stop()
{
    if(m_thread->isRunning())
    {
        m_timer->moveToThread(QApplication::instance()->thread());
        m_thread->quit();
    }
    m_timer->stop();
    disconnect(m_timer, SIGNAL(timeout()), this, SLOT(timerFired()));
}
 
void MyClockwork::timerFired()
{
    emit tick();
}

Le problème est que, lorsque je renvoie le minuteur m_timer dans le thread principal via la commande m_timer->moveToThread(QApplication::instance()->thread()), je reçois l'erreur suivante:

Code :

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

1
2
QObject::moveToThread: Current thread (0x3779a8) is not the object's thread (0x7262a0).
Cannot move to target thread (0x3779a8)

En gros, l'application ne parvient pas à mettre l'objet QTimer dans le thread-cible.
Pourquoi? Je ne sais pas; c'est pourquoi je sollicite votre aide.

Merci d'avance.

Adishatz!

PS: Quand l'application est terminée, j'obtiens aussi cette erreur:

Code :

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

QObject::killTimer: timers cannot be stopped from another thread

[yan]

Salut.
C'est normal. Un QOBject (ici ton timer) est associé à un QThread.

Au debut, le timer est associé au thread principale.
Tu appel movetothread depuis le thread principal => le timer est associé à ton thread

Au stop tu veux changer l'association du timer avec le thread principal => error. Cette fonction ne peut être exécuté que par le thread associé au timer. Tout comme le stop.

http://doc.qt.digia.com/stable/qobje...l#moveToThread

Code :

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

Warning: This function is not thread-safe; the current thread must be same as the current thread affinity. In other words, this function can only "push" an object from the current thread to another thread, it cannot "pull" an object from any arbitrary thread to the current thread.

c'est ce que tu fais

[VivienD]

Je vois.

J'avais créé cette fonction stop() non seulement pour arrêter proprement mon thread mais aussi pour ne pas avoir l'erreur ci-dessous.

Code :

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

QObject::killTimer: timers cannot be stopped from another thread

Dois-je créer une classe spéciale pour déplacer le timer de l'intérieur du thread? Dois-je envisager une autre alternative?

[yan]

Dois-je créer une classe spéciale pour déplacer le timer de l'intérieur du thread? Dois-je envisager une autre alternative?

Ca depend de ce que tu veux faire...

Que cherche tu à tester?

[VivienD]

Je veux juste arrêter proprement mon minuteur pour ne plus avoir cette erreur.

[yan]

Je veux juste arrêter proprement mon minuteur pour ne plus avoir cette erreur.

Au plus simple, tu lance un signal qui est connecté au stop du timer.

Regarde la FAQ :
http://qt.developpez.com/faq/?page=m...d-appartenance
http://qt.developpez.com/faq/?page=m...read-connexion