QObject::startTimer: Timers can only be used with threads started with QThread

Question

I am trying to start a Timer in a worker thread's event loop, but I get this error: QObject::startTimer: Timers can only be used with threads started with QThread

Whats wrong with this?

#include <QObject>
#include <QThread>
#include <QTimer>

class A : public QObject
{
    Q_OBJECT
public:
    A();

private:
    QThread m_workerThread;
    QTimer m_myTimer;

};

A::A()
{
    this->moveToThread(&m_workerThread);
    m_myTimer.moveToThread(&m_workerThread);
    m_workerThread.start();
    m_myTimer.start(1000);
}

score 3 · Answer 1 · answered Jan 04 '18 at 19:01

Initialize your timer anywhere, but start it right when the thread is started (attach it to QThread::started signal):

class A : public QObject
{
    Q_OBJECT
public:
    A();

private slots:
    void started();
    void timeout();

private:
    QThread m_workerThread;
    QTimer m_myTimer;
};

A::A()
{
    moveToThread(&m_workerThread);

    connect(&m_workerThread, SIGNAL(started()), this, SLOT(started()));
    connect(&m_myTimer, SIGNAL(timeout()), this, SLOT(timeout()));

    m_myTimer.setInterval(1000);
    m_myTimer.moveToThread(&m_workerThread);

    m_workerThread.start();
}

void A::started()
{
    timer.start();
}

void A::timeout()
{
    // timer handler
}

score 1 · Answer 2 · answered Mar 14 '14 at 08:45

1

I Think i figured it out, i tried to start the timer from the GUI thread, after i moved it to the worker thread, this way it seems to work:

class A : public QObject
{
    Q_OBJECT
public:
    A();

private:
    QThread m_workerThread;
    QTimer m_myTimer;

public slots:
    void sl_startTimer();
};

A::A()
{
    this->moveToThread(&m_workerThread);
    m_myTimer.moveToThread(&m_workerThread);
    m_workerThread.start();
    QMetaObject::invokeMethod(this, "sl_startTimer", Qt::QueuedConnection);
}

void A::sl_startTimer()
{
    m_myTimer.start(1000);
}

answered Mar 14 '14 at 08:45

user2950911

873
3
12
19

3

@SebastianLange The timer should be in the same threads where most of its signal recipients are. Otherwise you'll stall the recipient thread when the emitter thread is stalled. – Kuba hasn't forgotten Monica Mar 14 '14 at 11:48
1

true. just got something wrong in my mind, still was at single shot. But still, if the whole object is located in the thread, i would not need to move timer residing in this object to the other thread too... – Sebastian Lange Mar 14 '14 at 12:07
@SebastianLange u mean when the QTimer Member instead was a QTimer * and i added a slot like sl_init() where i allocate 'new QTimer(this)'? i guess you are right, but then you have more code. i prefer less code. – user2950911 Mar 14 '14 at 15:23

score 0 · Answer 3 · answered Mar 15 '14 at 03:18

This approach seems a little dangerous to me. By moving the QObject onto the QThread, you're making the thread responsible for the object's events (signals, slots, messages, etc). When the object is deleted, however, the thread will be deleted before the object itself, which can lead to some unexpected behaviours.

The recommended approach is to instantiate the thread and the object separately.

score 0 · Answer 4 · answered Dec 10 '19 at 15:01

Hope this will be helpful:

class ReadYoloResult : public QObject
{
    Q_OBJECT
public:
    ReadYoloResult(QObject *parent = 0);
    void startTimer();
    QThread workerThread;

private:
    QTimer *timer;

public slots:
    void timerSlot();
};

ReadYoloResult::ReadYoloResult(QObject * parent)
{
    this->moveToThread(&workerThread);
    timer = new QTimer();
    connect(timer,SIGNAL(timeout()),this,SLOT(timerSlot()));

    workerThread.start();

    //timer->start(1000);
}

void ReadYoloResult::startTimer(){
    timer->start(100);
}
void ReadYoloResult::timerSlot(){
    qDebug()<<"In timer slot";
}

QObject::startTimer: Timers can only be used with threads started with QThread

4 Answers4

Linked