Pointer to function to handle event.Is a good idea?

i'm using pointer to function inside my class to handle event.
For example:

Code:

#include <iostream>

using namespace std;

// MY_CLASS

class test 
{     
      private:
              string value;        
      public:
              void SetValue(string source) 
                     {  if (OnValueChange!=NULL)(*OnValueChange)(value,source);
                        value = source;}    
              string GetValue() { return value;}
              void (*OnValueChange)(string OldValue,string NewValue);    
      };

// END_MY_CLASS

//MAIN PROGRAM
void handleOnValueChange(string OldValue,string NewValue)
{
     cout<<"OLD VALUE : " << OldValue << "\n";
     cout<<"NEW VALUE : " << NewValue << "\n";
     cout<<"-----------------------------\n";
 }    

int main(int argc, char *argv[])
{
    test *atest ;
    atest = new test;
    atest->OnValueChange = handleOnValueChange;
    atest->SetValue("TEST");
    atest->SetValue("ANOTHER TEST");
    delete atest;
    system("PAUSE");
    return 0;
}

//END MAIN PROGRAM

This is a good idea or is better using virtual function,inheritance ??

thanx,

Gianni

your program does not ask operating member data of test in the handleOnValueChange. so it is enough

using a functor instead of a pointer to function is a better way, it is more flexible.

using virtual function is another way to fix this problem, but is it designed overly?

there are many ways to fix the problem, be sure you thought it enough before using it

I'm curious as to what his code is doing. What is [OnValueChange] ?

You could use Boost.Signals to implement a flexible system of signals and slots.