C ++ вспомогательная функция для конструктора копирования

Question

Мне не удалось найти хороший ответ на этот вопрос.

Я работаю над программой на C ++ и пытаюсь реализовать функцию с именем copy , котораяпринимает в качестве аргумента ссылку на другой объект.Затем он возвращает глубокую копию этого объекта.

Некоторые сведения о моем проекте: класс Scene содержит динамический массив (называемый «изображения») указателей либо на NULL, либо на экземпляркласса Image , который здесь не показан, но работает как следует (он наследует все свои методы от сторонней библиотеки, EasyBMP)

Причина, по которой я это делаючтобы избежать дублирования кода в двух местах, но вполне возможно, что я выбрал неправильный подход.

Я вызываю эту функцию в своем операторе присваивания:

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

И мой конструктор копирования:

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

Наконец, мой метод copy () выглядит следующим образом:

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;
}

В настоящее время он НЕ работает.Одна проблема, которую я вижу, состоит в том, что я пытаюсь вернуть переменную, которая выходит из области видимости, как только заканчивается функция копирования.Я пытался вернуть ссылку раньше, но компилятор выдавал ошибки, и это все равно не помогло бы с проблемой объема.

Но я даже не уверен, что моя логика верна, то есть вы можете даже сделать что-то вродеэто в конструкторе?Или я должен просто явно написать код в конструкторе копирования и операторе присваивания (без реализации вспомогательного метода copy )?

Я очень новичок в C ++ и указателях, поэтому любые рекомендациибудет высоко ценится.

Answer 1

Существует более простой и более идиоматический способ делать то, что вы хотите: идиома копирования и обмена .

// N.B. Not tested, but shows the basic structure of the copy-and-swap idiom.
class Scene
{
public:
    Scene(int)
    {
        // Initialize a pointer array of Images
    }

    ~Scene()
    {
        // Get rid of our pointer array of Images
    }

    // Copy constructor
    // N.B. Not exception safe!
    Scene(const Scene& rhs) : imgPtrArray(new Image*[rhs.max])
    {
        // Perform deep copy of rhs
        for (int i=0; i < rhs.max; ++i)
        {
            if (rhs.imgPtrArray[i] != 0)    
                imgPtrArray[i] = new Image(*(rhs.imgPtrArray[i]));
            else   
                imgPtrArray[i] = 0;   
        }      
    }

    // Copy assignment constructor
    // When this is called, a temporary copy of Scene called rhs will be made.
    // The above copy constructor will then be called. We then swap the
    // members so that this Scene will have the copy and the temporary
    // will destroy what we had.
    Scene& operator=(Scene rhs)
    {
        swap(rhs);
        return *this;
    }

    void swap(Scene& rhs)
    {
        // You can also use std::swap() on imgPtrArray
        // and max.
        Images** temp = imgPtrArray;
        imgPtrArray = rhs.imgPtrArray;
        rhs.imgPtrArray = temp;
        int maxTemp = max;
        max = rhs.max;
        rhs.max = maxTemp;
    }

private:
    Images** imgPtrArray;
    int max;
};

---

При этом я настоятельно рекомендуючто вы подберете хорошую вводную книгу по С ++ , в которой будут рассмотрены основы правильной реализации конструктора копирования и операторов назначения копирования.

Answer 2

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

перегруженный оператор присваивания копирует содержимое this в полученный аргумент source .Для copy нет необходимости возвращать что-либо или создавать локальный объект.Просто сделайте копию для члена от этого до источника .

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

Правило трех должно быть полезным для лучшего понимания этих вопросов.