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

时间:2012-08-16 07:08:48

标签: c++ qt timer

在它的“超时”信号/插槽功能中停止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?
}

感谢。

1 个答案:

答案 0 :(得分:6)

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

如果您使用的是排队连接,理论上可能会发生事件队列中仍然存在未处理的超时事件,因此为了使其超级安全,您可以使用以下内容:

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

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

请注意,停止功能中的条件应为!= NULL而不是== NULL。您也可以按如下方式编写该函数:

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

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

相关问题