2012-04-12 10 views
7

Quando provo a ottenere (non impostato!) Il tempo di scadenza corrente utilizzando il boost expires_from_now() di sembra effettivamente annullare il timer, ma effettivamente viene eseguito come previsto, ma alla fine non chiama il gestore.Perché la funzione asio boost expires_from_now() annulla un deadline_timer?

In altre parole, quando si accede a un deadline_timer con expires_from_now() chiamerà immediatamente il gestore e non chiamerà il gestore quando scade.

Risparmi il seguente codice ed corrispondente uscita:

#include <boost/asio.hpp> 
#include <boost/thread.hpp> 
#include <boost/date_time/posix_time/posix_time.hpp> 
#include <boost/date_time/posix_time/posix_time_io.hpp> 
#include <iostream> 

using namespace boost::posix_time; 
using namespace std; 

void handler1(const boost::system::error_code &ec) 
{ 
    if (ec == boost::asio::error::operation_aborted) 
    { 
     std::cout << microsec_clock::local_time() << " Handler1: Timer 1 was cancelled or retriggered." << std::endl; 
    } 
    else 
    { 
     std::cout << microsec_clock::local_time() << " Handler1: expired." << std::endl; 
    } 
} 

boost::asio::io_service io_service1; 

class Mytimer { 
public: 
    Mytimer(); 
    void startTimer(); 
    void runTimerThread(); 
    bool isRunning(); 
private: 
    bool m_isRunning; 
    boost::asio::deadline_timer* m_pTimer; 
    boost::thread* m_pThread; 
}; 

Mytimer::Mytimer() 
    : m_pTimer(NULL), 
    m_pThread(NULL) 
{ 
    m_pTimer = new boost::asio::deadline_timer(io_service1); 
    m_pTimer->async_wait(handler1); 
} 

void Mytimer::runTimerThread() 
{ 
    io_service1.run(); 
} 

void Mytimer::startTimer() 
{ 
    m_pThread = new boost::thread(&Mytimer::runTimerThread, this); 
    m_pTimer->expires_from_now(boost::posix_time::seconds(10)); 
} 

bool Mytimer::isRunning() 
{ 

    time_duration td = m_pTimer->expires_from_now(); 
    if (td.total_seconds() > 0) 
    { 
     return true; 
    } 
    return false; 
} 

int main() 
{ 
    time_facet *facet = new time_facet("%Y%m%d %H:%M:%S%f"); 
    std::cout.imbue(std::locale(std::cout.getloc(), facet)); 

    cout << microsec_clock::local_time() << " before timer construction" << endl; 
    Mytimer timer; 
    cout << microsec_clock::local_time() << " before startTimer()" << endl; 
    timer.startTimer(); 
    cout << microsec_clock::local_time() << " IsRunning: " << timer.isRunning() << endl; 
    for (int i = 0; i <= 20; i++) 
    { 
     sleep(1); 
     cout << microsec_clock::local_time() << " IsRunning: " << timer.isRunning() << endl; 
    } 
} 

20120412 22: 41: 45.689.235 prima costruzione timer
20.120.412 22: 41: 45.689.514 prima startTimer()
20.120.412 22:41: 45689619 IsRunning: 1
20120412 22: 41: 45689693 Handler1: il timer 1 è stato annullato o riattivato.
20.120.412 22: 41: 46.689.792 IsRunning: 1
20.120.412 22: 41: 47.689.994 IsRunning: 1
20.120.412 22: 41: 48.690.179 IsRunning: 1
20.120.412 22: 41: 49.690.375 IsRunning: 1
20.120.412 22:41 : 50690530 IsRunning: 1
20.120.412 22: 41: 51.690.712 IsRunning: 1
20.120.412 22: 41: 52.690.884 IsRunning: 1
20.120.412 22: 41: 53.691.069 IsRunning: 1
20.120.412 22: 41: 54.691.236 IsRunning: 0
20120412 22: 41: 55691428 IsRunning: 0
20.120.412 22: 41: 56.691.614 IsRunning: 0
20.120.412 22: 41: 57.691.810 IsRunning: 0
20.120.412 22: 41: 58.692.001 IsRunning: 0
20.120.412 22: 41: 59.692.193 IsRunning: 0
20.120.412 22:42 : 00692364 IsRunning: 0
20.120.412 22: 42: 01.692.542 IsRunning: 0
20.120.412 22: 42: 02.692.706 IsRunning: 0
20.120.412 22: 42: 03.692.886 IsRunning: 0
20.120.412 22: 42: 04.693.071 IsRunning: 0
20120412 22: 42: 05693267 IsRunning: 0
20.120.412 22: 42: 06.693.465 IsRunning: 0

+0

Penso che ci siano due problemi qui: prima nel costruttore il timer viene chiamato senza alcuna scadenza, quindi in realtà scadrà immediatamente. Secondo quando la funzione membro riavvia il timer, non vi è alcun gestore. Quindi expires_from_now funziona ancora (come puoi vedere le 1 modifiche a 0 dopo 10 secondi), ma il gestore non verrà mai chiamato, perché non ne è associato nessuno. –

+0

Terrò la domanda aperta per ora, forse qualcuno può confermare le mie conclusioni. –

+1

+1 per il riproduttore –

risposta

10

Quando provo ad ottenere (! Non impostato) La durata di usare il boost expires_from_now() è sembra di annullare in realtà il timer, eppure realtà funziona come previsto, ma alla fine non chiama il gestore.

questa ipotesi non è corretta. Quando si crea il deadline_timer

m_pTimer = new boost::asio::deadline_timer(io_service1); 
m_pTimer->async_wait(handler1); 

hai detto che scada immediatamente.Poi, quando si avvia il io_service

m_pThread = new boost::thread(&Mytimer::runTimerThread, this); 

successivamente si richiama

m_pTimer->expires_from_now(boost::posix_time::seconds(10)); 

che, come la documentazione describes clearly, questo annullerà qualsiasi gestore di circolazione, enfasi aggiunta è mia

Questa funzione imposta il tempo di scadenza Qualsiasi attesa asincrona in attesa delle operazioni verrà annullata. Il gestore per ogni operazione annullata verrà richiamato con il codice boost::asio::error::operation_aborted errore .

Per risolvere questo, cambiare il metodo di Mytimer::startTimer() come segue

--- timer4.cc.orig 2012-04-13 11:18:47.000000000 -0500 
+++ timer4.cc 2012-04-13 11:16:54.000000000 -0500 
@@ -1,4 +1,3 @@ 
- 
#include <boost/asio.hpp> 
#include <boost/thread.hpp> 
#include <boost/date_time/posix_time/posix_time.hpp> 
@@ -39,7 +38,6 @@ 
    m_pThread(NULL) 
{ 
    m_pTimer = new boost::asio::deadline_timer(io_service1); 
- m_pTimer->async_wait(handler1); 
} 

void Mytimer::runTimerThread() 
@@ -49,8 +47,9 @@ 

void Mytimer::startTimer() 
{ 
- m_pThread = new boost::thread(&Mytimer::runTimerThread, this); 
    m_pTimer->expires_from_now(boost::posix_time::seconds(10)); 
+ m_pTimer->async_wait(handler1); 
+ m_pThread = new boost::thread(&Mytimer::runTimerThread, this); 
} 

bool Mytimer::isRunning() 

che si traduce nel comportamento previsto

mbp:stackoverflow samm$ ./a.out 
20120413 11:17:22922529 before timer construction 
20120413 11:17:22923155 before startTimer() 
20120413 11:17:22923530 IsRunning: 1 
20120413 11:17:23924702 IsRunning: 1 
20120413 11:17:24925971 IsRunning: 1 
20120413 11:17:25927320 IsRunning: 1 
20120413 11:17:26928715 IsRunning: 1 
20120413 11:17:27929969 IsRunning: 1 
20120413 11:17:28930601 IsRunning: 1 
20120413 11:17:29931843 IsRunning: 1 
20120413 11:17:30933098 IsRunning: 1 
20120413 11:17:31934366 IsRunning: 0 
20120413 11:17:32923594 Handler1: expired. 
20120413 11:17:32934692 IsRunning: 0 
20120413 11:17:33935922 IsRunning: 0 
20120413 11:17:34936638 IsRunning: 0 
20120413 11:17:35937905 IsRunning: 0 
20120413 11:17:36939133 IsRunning: 0 
20120413 11:17:37940407 IsRunning: 0 
20120413 11:17:38941043 IsRunning: 0 
20120413 11:17:39942319 IsRunning: 0 
20120413 11:17:40942662 IsRunning: 0 
20120413 11:17:41943989 IsRunning: 0 
20120413 11:17:42945284 IsRunning: 0 
20120413 11:17:43946555 IsRunning: 0 
mbp:stackoverflow samm$ 
+0

Grazie per la spiegazione dettagliata. –

+0

L'output apre un'altra domanda. Come progettare una funzione membro IsRunning() che non restituisce 0 mentre il timer è ancora in esecuzione. Una cosa che mi viene in mente è usare un flag booleano privato all'interno del gestore che è impostato su false nella funzione del gestore membro. La modifica di total_seconds a total_nanoseconds potrebbe essere un'altra opzione. –

+0

@ Michael, non è ovvio per me cosa speri di ottenere con questo approccio. Un 'deadline_timer' scadrà dopo il tempo indicato, o sarà cancellato. In tal caso, la soluzione è controllare l'errore fornito al gestore di completamento. –

1

Il seguente codice ha ora l'inizializzazione deadline_timer corretta, e ha un indicatore bool privato, che consente di determinare, se il timer è attualmente in esecuzione. Ha anche un gestore locale membro.

class Mytimer { 
public: 
    Mytimer(); 
    void startTimer(int timeDelaySecs); 
    void cancelTimer(); 
    void runTimerThread(); 
    bool isRunning(); 
    void timerHandler(const boost::system::error_code& ec); 
private: 
    bool m_isRunning; 
    boost::asio::deadline_timer* m_pTimer; 
    boost::thread* m_pThread; 
}; 

Mytimer::Mytimer() 
    : m_pTimer(NULL), 
     m_pThread(NULL) 
{ 
    m_pTimer = new boost::asio::deadline_timer(io_service1); 
} 

void Mytimer::runTimerThread() 
{ 
    io_service1.run(); 
} 

void Mytimer::startTimer(int timeDelaySecs) 
{ 
    m_pTimer->expires_from_now(boost::posix_time::seconds(timeDelaySecs)); 
    m_pTimer->async_wait(boost::bind(&Mytimer::timerHandler, this, _1)); 
    m_pThread = new boost::thread(&Mytimer::runTimerThread, this); 
    m_isRunning = true; 
} 

bool Mytimer::isRunning() 
{ 
    return m_isRunning; 
} 

void Mytimer::timerHandler(const boost::system::error_code& ec) 
{ 
    if (ec == boost::asio::error::operation_aborted) 
    { 
     std::cout << microsec_clock::local_time() << " Handler1: Timer 1 was cancelled or retriggered." << std::endl; 
    } 
    else 
    { 
     std::cout << microsec_clock::local_time() << " Handler1: expired." << std::endl; 
    } 
    m_isRunning = false; 
}