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]

sci wrote:

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
--
My email address is valid, but changes periodically.
To contact me please use the address from a recent posting.