C++重载赋值运算符报definition of implicitly declared copy assignment operator错误如何解决
错误产生原因
- C++语法要求所有类成员函数的实现,必须先在类定义中显式声明对应函数原型,否则编译器不会认可该函数属于类成员。
- 你当前的
Cabinet类头文件定义中,没有显式声明拷贝赋值运算符Cabinet& operator=(const Cabinet& right);,编译器只会为类自动生成隐式的默认拷贝赋值运算符,而C++不允许用户手动提供隐式声明函数的实现,因此你在cpp文件中直接写该运算符的实现时,编译器会抛出对应的报错。
修复方案
- 先解决编译报错:在
Cabinet类的public成员区域添加拷贝赋值运算符的声明即可,修改后的类声明示例如下:
class Cabinet{ private: int rows; int id_cabinet; int columns; Chemical*** chemicals; string alphabet [9]; public: Cabinet(int id = 0, int rows = 0, int columns = 0); ~Cabinet(); // 新增拷贝赋值运算符声明 Cabinet& operator=(const Cabinet& right); int getRow(); int getColumn(); int plusCount(); };
- 额外运行逻辑修复提示:你的类中包含动态分配的三级指针
Chemical*** chemicals,当前编写的赋值运算符实现只做了内部元素赋值,没有处理自身原有动态内存释放、基础成员复制、深拷贝等逻辑,运行时大概率会出现内存泄漏、野指针、浅拷贝崩溃问题,可参考如下框架完善实现:
Cabinet& Cabinet::operator=( const Cabinet& right ) { if(&right != this){ // 释放当前对象已分配的动态内存,避免泄漏 for (int i = 0; i < rows; i++) { for (int j = 0; j < columns; j++) { delete chemicals[i][j]; } delete[] chemicals[i]; } delete[] chemicals; // 复制基础成员变量 id_cabinet = right.id_cabinet; rows = right.rows; columns = right.columns; for(int i=0; i<9; i++){ alphabet[i] = right.alphabet[i]; } // 重新分配内存后深拷贝元素 chemicals = new Chemical**[rows]; for (int i = 0; i < rows; i++) { chemicals[i] = new Chemical*[columns]; for (int j = 0; j < columns; j++) { chemicals[i][j] = new Chemical(*right.chemicals[i][j]); } } } return *this; }
内容的提问来源于stack exchange,提问作者PoePew
相关产品推荐
相关产品推荐

