Question

I haven’t been able to find a good answer to this question.

I’m working on a C++ program and I’m trying to implement a function named copy which takes a reference to another object as an argument. Then, it returns a deep copy of this object.

Some background on my project: the Scene class contains a dynamic array (called “Images”) of pointers to either NULL or an instance of the Image class, which is not shown here – but works as it should (it inherits all of its methods from a third party library, EasyBMP)

The reason I’m doing this is to avoid duplicating code in two places, but it’s very possible that I’m taking the wrong approach.

I call this function in my assignment operator:

Scene const & Scene::operator=(Scene const & source)
{
    if (this != &source) {
        clear();
        copy(source);
    }
    return *this;
}

And my copy constructor:

Scene::Scene(Scene const & source)
{
    copy(source);
}

Finally, my copy() method looks like this:

Scene const & Scene::copy(Scene const & source)
{
    Scene res(source.Max);
    for (int i=0; i<res.Max; i++)
    {
        delete res.Images[i];
        if (source.Images[i] != NULL) 
            res.Images[i] = new Image(*(source.Images[i]));
        else
            res.Images[i] = NULL;
    }   

    return res;
}

Currently, it does NOT work. One problem I can see is that I’m trying to return a variable that goes out of scope as soon as the copy function ends. I tried returning a reference before, but the compiler threw errors and this wouldn’t help with the scope issue anyways.

But I’m not even sure that my logic is right, i.e. can you even do something like this in a constructor? Or should I just explicitly write out the code in the copy constructor and assignment operator (without implementing the helper method copy)?

I’m very new to C++ and pointers, so any guidance would be much appreciated.

Answer 1

Scene const & Scene::operator=(Scene const & source);

overloaded assignment operator copies the content of this to the argument received source. For copy there is no need to return any thing or to create a local object. Just make a member wise copy from this to source.

 void Scene::copy(Scene const & source){
     // Member wise copy from this to source
 }

Rule of three should be helpful to better understand more about these.