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将在其析构函数中停止运行。