我正在尝试通过重载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是CMap的数组。
CMap tempMap;
tempMap.setDimensions(myMap[0].m_nHeight, myMap[0].m_nWidth);
tempMap.m_pData = myMap[0].m_pData;
最佳答案
首先:不是等于运算符,而是赋值运算符。
第二:赋值运算符应具有const &
参数,而不是指针,并返回对该对象的引用,而不是任何内容。
第三:除了您做的事情会干扰复制的语义之外,编译器会自动提供复制的分配超载。
尊重第三点,您做了一件不做“真实”复制的事情:通过指针使用动态数组。编译器将仅复制指针,因此您必须编写自己的operator=
才能手动复制数组(您要在问题中尝试的内容)。
最简单的解决方案是:使用类似std::vector
的容器来代替手动内存管理。它已经正确实现了复制和分配,因此您不必在您的课堂上写自己的operator=
。