Qt QTimer以这种方式阻止它是否安全？

Question

在它的"超时"信号/插槽功能中停止Qt的定时器是否安全？似乎无法在Qt文档中找到有关QTimer的任何信息.

我创建了一个定时向服务器发送"保持活动"消息的计时器.如果在发送消息时出现某种错误,我希望停止此计时器.

private:
   QTimer* mpKeepAliveTimer;

Timer初始化如下:

mpKeepAliveTimer = new QTimer(/* this */);

QObject::connect(mpKeepAliveTimer, SIGNAL(timeout()), this, SLOT(OnKeepAlive()));

mpKeepAliveTimer->start(KEEP_ALIVE_PERIOD);

像这样停下来:

if (mpKeepAliveTimer != NULL) // <-- Edited
{
    if (mpKeepAliveTimer->isActive() == true)
        mpKeepAliveTimer->stop();

    delete mpKeepAliveTimer;
    mpKeepAliveTimer = NULL;
}

超时功能如下所示:

void Classname::OnKeepAlive()
{
   if (isErrorFound == true)
      mpKeepAliveTimer->stop();   // <---- IS THIS SAFE?
}

谢谢.

Answer 1

只要您没有明确使用排队连接,这是安全的.
这是因为在处理完emit timeout()所有连接的插槽之前,该函数不会返回.

但是,如果您使用的是排队连接,理论上可能会发生事件队列中仍有未处理的超时事件,因此为了使其超安全,您可以使用以下命令:

void Classname::OnKeepAlive()
{
    if (!mpKeepAliveTimer || !mpKeepAliveTimer->isActive()) return;

    if (isErrorFound)
    {
        mpKeepAliveTimer->stop();
    }
}

请注意,stop函数中的条件应该是!= NULL而不是== NULL.您也可以按如下方式编写该函数:

if (mpKeepAliveTimer)
{
    delete mpKeepAliveTimer;
    mpKeepAliveTimer = NULL;
}

正如评论中已经建议的那样,QTimer将在其析构函数中停止运行.