Question

I am trying to do a deep copy of class B, but A doesn’t get set.

Why does b3->print return a garbage number instead of 1?

From what I understand, b2 and b3 are both pointing to the same A object. but I created a new object on the heap with B’s copy constructor. so why are they both still poiting to the same object?

I hope this makes sense.

#include <cstdlib>
#include <iostream>

using namespace std;

class A{
      int num;
public:
       A(int n):num(n){ cout<< "A "<< num << " constructor" <<endl;}  
       ~A(){ cout<< "A "<< num <<" destructor. " <<endl; }   

       int print(){
        cout<< num<< endl;
       }
};

class B{
      A *a;
      int num;
public:
       B(int n):num(n){
           a = new A(n);
           cout<< "B "<< num <<" constructor" <<endl;    
       }  
       ~B(){
            delete a; 
            cout<< "B "<< num <<" destructor"<<endl; 
       }    
       // Copy contructor
       B(const B & b): a(new A(b.num)){ 
       } 

       <strike>int<\strike> void print(){
        cout<< num << endl;
       }

       int get_num(){
           return num;
       }
};

int main(int argc, char *argv[])
{ 
    B *b2 = new B(1);
    B *b3(b2);
    b2->print();
    delete b2;
    b3->print();
    system("PAUSE");
    return EXIT_SUCCESS;
}

Answer 1

B *b3(b2); doesn’t do what you think.

It’s equivalent to B* b3 = b2. The pointers will point to the same location. When you do delete b2;, you’re also freeing the memory pointed to by b3.

To do a deep copy, do:

 B* b3 = new B(*b2);

There’s also undefined behaviour here:

int print(){
   cout<< num << endl;
}

because you never return. Change the return type to void.

To get the value expected:

B(const B & b): a(new A(b.num)), num(b.num){ 
}