Глубокая копия класса с перегрузкой оператора присваивания

Question

Глубокая копия класса с перегрузкой оператора присваивания

0

Я пытаюсь создать глубокую копию класса, перегружая оператор equals, однако он, похоже, не работает. Любая помощь ценится!

Это класс, который я пытаюсь скопировать:

class CMap {

public: 

    int m_nWidth;
    int m_nHeight;
    char* m_pData;
    void setDimensions(int nWidth, int nHeight);
    void operator = (CMap* rhs);

};

это мой перегруженный оператор:

CMap& operator = (const CMap&rhs)
    {
        if (this == &rhs)
        return *this;
    memcpy(this -> m_pData, rhs.m_pData, sizeof(char)*rhs.m_nWidth*rhs.m_nHeight);
        return *this;

    }

и это вызов функции в основном. myMap - это массив CMaps.

CMap tempMap;
        tempMap.setDimensions(myMap[0].m_nHeight, myMap[0].m_nWidth);
        tempMap.m_pData = myMap[0].m_pData;

user1816546 21 дек. 2013, в 12:57

Источник

0

«Это не похоже на работу» неинформативно.
user529758 21 дек. 2013, в 11:32
0

@ Йоахим Исакссон в setDimensions я выделяю память. К сожалению, я должен держать это так
user1816546 21 дек. 2013, в 11:36
0

@ H2CO3 не работает, я имею в виду, что он не выполняет глубокое копирование.
user1816546 21 дек. 2013, в 11:37
0

Итак, проверьте некоторые возможные общие проблемы. Может ли быть так, что this->m_pData - это тот же указатель, что и rhs->m_pData , например?
user529758 21 дек. 2013, в 11:39
0

@ H2CO3 я просто включил его в качестве проверки в операторе перегрузки
user1816546 21 дек. 2013, в 11:53

Показать ещё 3 комментария

Теги:

c++

operator-overloading

deep-copy

1 ответ

Ещё вопросы

«Это не похоже на работу» неинформативно.
@ Йоахим Исакссон в setDimensions я выделяю память. К сожалению, я должен держать это так
@ H2CO3 не работает, я имею в виду, что он не выполняет глубокое копирование.
Итак, проверьте некоторые возможные общие проблемы. Может ли быть так, что this->m_pData - это тот же указатель, что и rhs->m_pData , например?
@ H2CO3 я просто включил его в качестве проверки в операторе перегрузки

Manu343726 · Answer 1 · 2013-12-21T09-37-00.000Z

Во-первых: это не оператор равенства, что оператор присваивания.
Второй: оператор присваивания должен иметь параметр const &, а не указатель, и возвращает ссылку на объект, а не что-либо.
В-третьих: кроме того, что вы делаете что-то, что мешает семантике копирования, автоматически возникает перегрузка, которая копирует копию.

Уважайте точку три, вы сделали одну вещь, которая не выполняет "истинное" копирование: использование динамического массива с помощью указателя. Компилятор будет копировать только указатель, поэтому вам нужно написать собственный operator= чтобы скопировать массив вручную (что вы пытаетесь в вопросе).

Самое простое решение: использовать вместо std::vector вместо ручного управления памятью. Он правильно выполнил копирование и настройку, поэтому вам не нужно писать собственный operator= в своем классе.

На # 3: он пытается сделать именно это (недостаточно копировать указатель, также необходимо скопировать содержимое памяти, на которое указывает).
На # 3: у компилятора действительно нет выбора в этом вопросе. Оператор присваивания копии объявляется неявно, если он не объявлен пользователем и определяется при использовании, поэтому компилятор просто делает то, что говорит язык.
@ H2CO3 Я знаю, я редактировал вопрос :)
@ Manu343726 ручное управление памятью является пунктом моего назначения. Я так понимаю, я не перегружаю оператор присваивания правильно?
@ user1816546 да. У него нет этой подписи. Подпись foo& operator=( const foo&) (Предположим, ваш класс называется foo ). Он должен вернуть ссылку на объект после копирования, то есть return *this; ,
@KerrekSB Я знаю, поэтому я предлагаю OP использовать стандартный контейнер, что легко.
@ Manu343726 о, спасибо :) я постараюсь работать над этим тогда
@ Manu343726 Я попытался перегрузить оператора, но теперь программа падает - есть идеи, почему?
давайте продолжим это обсуждение в чате