Please comment on the code

[sci]

For B to call back a member of A, the code is design as:

class A;

class B
{
A* p;
public:
void Init(void* m) { p = (A*)m; }
void docallback();
};


class A
{
B b;
public:
A() { b.Init(this); }
void dowork() { }

};


void B::docallback()
{
((A*)p)->dowork();
}

Thanks!

 

[Kevin Goodsell]

For B to call back a member of A, the code is design as:

class A;

class B
{
A* p;
public:
void Init(void* m) { p = (A*)m; }

C-style casts are evil. Why didn't you just make Init take a A*?

void docallback();
};


class A
{
B b;
public:
A() { b.Init(this); }
void dowork() { }

};


void B::docallback()
{
((A*)p)->dowork();

Why are you casting p to the type it already is?

}

-Kevin