Qt QTimer ¿es seguro detenerlo de esta manera?

Question

¿Es seguro detener el cronómetro de Qt en su función de ranura/señal de "tiempo de espera"? Parece que no se puede encontrar información en la documentación de Qt sobre QTimer.

He creado un temporizador que periódicamente envía un mensaje de "mantener vivo" al servidor. Quiero detener este temporizador si hay algún tipo de error al enviar mi mensaje.

private: 
    QTimer* mpKeepAliveTimer; 

temporizador se inicializa como esto:

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

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

mpKeepAliveTimer->start(KEEP_ALIVE_PERIOD); 

Detenido así:

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

    delete mpKeepAliveTimer; 
    mpKeepAliveTimer = NULL; 
} 

función de tiempo de espera es el siguiente:

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

Gracias.

Answer 1

Siempre que no esté utilizando explícitamente las conexiones en cola, esto es seguro.
Esto se debe a que la función emit timeout() no volverá hasta que se hayan procesado todas las ranuras a las que está conectado.

Si embargo estaban utilizando conexiones en cola, en teoría podría ocurrir que todavía hay eventos de tiempo de espera no procesados ​​en la cola de eventos, por lo que para que sea hiper-seguro que podría utilizar lo siguiente:

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

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

Nota que la condición en su función de parada debe ser != NULL en lugar de == NULL. También puede escribir que la función de la siguiente manera, sin embargo:

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

Como ya se ha sugerido en los comentarios, QTimer dejará de sí misma en su destructor.