QTimer 在 QThread 中运行

Question

我需要一个带有计时器的类，它每 100 毫秒执行一次任务，这个类需要在线程中运行，所以我想将 qtimer 与 qthread 结合起来。

我创建了以下代码：

class Worker : public QObject
{
    Q_OBJECT
    public:
    void setEnabled(bool enable);
    
    public slots:
    void initialize();
    
    private:
    void doWork();
    
    QTimer *m_timer;
}

void Worker::initialize()
{
    m_timer = new QTimer(this);
    connect(m_timer, &QTimer::timeout, this, &Worker::doWork, Qt::DirectConnection);
    m_timer->start(100);
}

void Worker::setEnabled(bool enable)
{
    if(enable)
        m_timer->start(100);
    else
        m_timer->stop();
}

int main(int argc, char *argv[])
{
    QCoreApplication app(argc,argv);

    QThread *thread = new QThread;
    Worker *worker = new Worker;
    
    QObject::connect(thread, &QThread::started, worker, &Worker::initialize);
    
    worker->moveToThread(thread);
    thread->start();
    
    app.exec();

    delete worker;
    delete thread;
}

使用以下命令我可以启用/禁用时间

worker->setEnabled(false); worker->setEnabled(true);

我已经测试过，它工作正常，但我想知道这是否正确？

感谢您的帮助

Answer 1

不，这并不完全正确。

Worker::setEnabled(bool enable) 也应该是一个插槽，因为它直接调用了QTimer::start() 插槽。直接从主线程调用Worker::setEnabled 会导致未定义的行为。您必须使用信号槽连接从主线程安全地调用 setEnabled。

您还应该在构造函数中初始化Worker::m_timer，而不是将其推迟到initialize()，因此如果Worker::setEnabled 的调用早于预期，您就不会遇到悬空指针。 moveToThread 将移动 Worker 的所有孩子，所以这是非常理智的行为。

Answer 2

我唯一需要提的是，m_timer 不能在构造函数中初始化。请参阅此处来自 qt 的信息：

顺便说一句，这里要注意的一件非常重要的事情是，你永远不应该在 QObject 类的构造函数中分配堆对象（使用 new），因为这个分配是在主线程上执行的，而不是在新的 QThread 实例，这意味着新创建的对象然后由主线程拥有，而不是 QThread 实例。这将使您的代码无法工作。而是在主函数槽中分配此类资源，例如在这种情况下的 initialize()，因为当调用该对象时，对象将位于新线程实例上，因此它将拥有该资源。