Passing object references to classes

[jdm2104]

Is this sort of thing safe to do (passing by reference)?

View::OnIntialUpdate()
{
      pPAInterface = new CPAInterface;

      pPan = new CPan(*pPAInterface);

      pDec = new CDec(*pPan);
}

The reason I ask is I want the data stored in the CPAInterface class to be accessable and modifyable by the CPan class, and the data held in Cpan to then be accessable by CDec.

Thanks for any advice.

[DragForce]

If you know what your are doing, if you sure that your design is right, if you considered the lifetime of all created objects, then it is Ok

[jdm2104]

Thanks

[exterminator]

Is this sort of thing safe to do (passing by reference)?

View::OnIntialUpdate()
{
      pPAInterface = new CPAInterface;

      pPan = new CPan(*pPAInterface);

      pDec = new CDec(*pPan);
}

The reason I ask is I want the data stored in the CPAInterface class to be accessable and modifyable by the CPan class, and the data held in Cpan to then be accessable by CDec.

Thanks for any advice.

Why not have members of the required types in the respective classes? CPan would have a member CPAInterface and CDec would have a member CPan. Now when you construct CDec you simply do:

CPan obj_CPan(CPAInterface());
CDec obj_CDec(obj_CPan);

Otherwise, if this doesn't suffice what you want - you could have the member functions operating on the objects take the respective arguments by reference. That would be a better way to go about achieving this and would not necessitate the need for dynamic allocations. Hope this helps. Regards.

[SuperKoko]

As exterminator pointed, you probaby don't need dynamic allocation.
And, as a proof, I can show that your code has a memory leak:
You don't free the object pointed-to by pPAInterface, and don't save this pointer anywhere.