views:

144

answers:

3

Hello, I try to call a function which passed as function pointer with no argument, but I can't make it work.

void *disconnectFunc;

void D::setDisconnectFunc(void (*func)){
    disconnectFunc = func;
}

void D::disconnected(){
    *disconnectFunc;
    connected = false;
}
+2  A: 

You need to declare disconnectFunc as a function pointer, not a void pointer. You also need to call it as a function (with parentheses), and no "*" is needed.

WhirlWind
Thank you.Final code:void (*disconnectFunc)();void D::setDisconnectFunc(void (*func)()){ disconnectFunc = func;}void D::disconnected(){ (*disconnectFunc)(); connected = false;}
Roland Soós
I'll bet it works way better too ;)
WhirlWind
+8  A: 

The correct way to do this is:

typedef void (*callback_function)(void); // type for conciseness

callback_function disconnectFunc; // variable to store function pointer type

void D::setDisconnectFunc(callback_function pFunc)
{
    disconnectFunc = pFunc; // store
}

void D::disconnected()
{
    disconnectFunc(); // call
    connected = false;
}
GMan
+1 for using my preferred syntax of de-referencing a function pointer (prefer disconnectFunc() to (*disconnectFunc)(), even though they're both fine)
Dan
+1  A: 

Replace void *disconnectFunc; with void (*disconnectFunc)(); to declare function pointer type variable. Or even better use a typedef:

typedef void (*func_t)(); // pointer to function with no args and void return
...
func_t fptr; // variable of pointer to function
...
void D::setDisconnectFunc( func_t func )
{
    fptr = func;
}

void D::disconnected()
{
    fptr();
    connected = false;
}
Nikolai N Fetissov