Un buon modo per implementare callback utilizzabili in C++

Question

Ho una classe Menu personalizzata scritta in C++. Per separare il codice in funzioni di facile lettura, sto utilizzando i callback.

Poiché non desidero utilizzare Singletons per l'host del menu, fornisco un altro parametro (obiettivo) che verrà assegnato al callback come primo parametro (una sorta di soluzione alternativa per il riferimento "questo" mancante) .

Registrazione-Firma

AddItem(string s, void(*callback)(void*,MenuItem*), void* target = NULL) 

Esempio di una registrazione

menu->AddItem(TRANSLATE, "translate", &MyApp::OnModeSelected); 

Esempio di Handler

/* static */ 
void MyApp::OnModeSelected(void* that, MenuItem* item) { 
    MyApp *self = (MyApp*)that; 
    self->activeMode = item->text; 
} 

C'è qualcosa si potrebbe considerare sporco con questo approccio? Ce ne sono forse di migliori?

Answer 1

L'approccio richiede che le funzioni di callback siano funzioni libere o membri statici di una classe. Non consente ai client di utilizzare le funzioni membro come richiamate. Una soluzione a questo è utilizzare boost::function come tipo della richiamata:

typedef boost::function<void (MenuItem*)> callback_type; 
AddItem(const std::string& s, const callback_type& callback = callback_type()); 

client può quindi utilizzare boost::bind o boost::lambda passare nella callback:

menu->AddItem("Open", boost::bind(&MyClass::Open, this)); 

Un'altra opzione è quella di utilizzare boost::signals che consente più callback da registrare per lo stesso evento.

Answer 2

Mi piace il tuo approccio. Un'alternativa sarebbe quella di dichiarare un'interfaccia, che è in un certo senso il "OO equivalente" di un callback:

class IMenuEntry { 
public: 
    virtual void OnMenuEntrySelected(MenuItem* item) = 0; 
}; 

La firma di registrazione sarebbe diventato

AddItem(string s, IMenuEntry * entry); 

E l'implementazione del metodo

class MyApp : public IMenuEntry { 
public: 
    virtual void OnMenuEntrySelected(MenuItem* item){ 
     activeMode = item->text; 
    } 
} 

L'approccio dell'interfaccia consente di evitare la "soluzione nulla * per il puntatore mancante this.

Answer 3

Non vedo niente di sbagliato tranne che la firma del puntatore della funzione è difficile da leggere. Ma, probabilmente farebbe lo schema observer per raggiungere questo obiettivo.

Answer 4

Si potrebbe dare un'occhiata utilizzando boost::bind.

menu->AddItem(TRANSLATE, 
       "translate", 
       boost::bind(&MyApp::OnModeSelected, this, _1, _2)); 

Answer 5

Mi raccomando guardando boost::function e boost:bind per questo. Imparare renderà la tua funzione vincolante centinaia di volte più facile.

Answer 6

Leggi this carta bianca. Costruisce varie tecniche per un meccanismo di callback analizzando prestazioni, usabilità e altri compromessi in modo abbastanza dettagliato.Ho trovato una lettura difficile però :-(

Answer 7

vostro potrebbe utilizzare un functor per incapsulare il callback. Ciò consentirebbe di utilizzare sia una funzione C-stile o un'interfaccia oggetto di fornire il callback.