function pointer in class

[morz]

#include <cstdlib>
#include <iostream>

using namespace std;
what wrong with my code? how to correct this code to make it works?
Thanks.


class test {
public:
void hello() {
cout << "hello" << endl;
}

void ok(void (*j)()) {
(*j)();
}

void callme() {
ok(hello);
}
};

int main(int argc, char *argv[]) {
test k;
k.callme();
system("PAUSE");
return 0;
}

 

[Thomas Tutone]

#include <cstdlib>
#include <iostream>

using namespace std;
what wrong with my code? how to correct this code to make it works?
Thanks.


class test {
public:
void hello() {

Change the above line to:

static void hello() {

Best regards,

Tom

 

[morz]

Thanks,it works!

How about if I dont want using static function,because I want my hello
function can access class member.Thanks.

 

[morz]

I try doing like this,but still error

class test {
public:
void hello() {
cout << "hello" << endl;
}

void ok(void (test::*j)()) {
(*j)();
}

void callme() {
ok(&hello);
}
};

 

[amparikh]

I try doing like this,but still error

class test {
public:
void hello() {
cout << "hello" << endl;
}

void ok(void (test::*j)()) {
(*j)();
}

void callme() {
ok(&hello);
}
};

Pointer to member functions are different from regular function
pointers.

change the code to...

void ok(void (test::*j)()) {
(this->*j)();
}

void callme() {
ok(&test::hello);
}

 

[morz]

Wow ! it works! Thanks a lot.

 

[bharath.donnipad]

Wow ! it works! Thanks a lot.

Firstly I dont understand why the initial code in first mail doesn't
work. Can someone please explain?

 

[David Harmon]

Wow ! it works! Thanks a lot.

Firstly I dont understand why the initial code in first mail doesn't
work. Can someone please explain?[/QUOTE]

Read section "[33] Pointers to member functions" in Marshall Cline's
C++ FAQ. You can get the FAQ at:
http://www.parashift.com/c++-faq-lite/