重载运算符时 C++ 代码中的错误

Question

我试图在我的代码中重载“+”和“=”运算符，但我不断收到运行时错误，并且程序在使用 VS2012 运行时崩溃，但在 borland C 3.1 中运行完美。

这是我的代码：

class employee{
    int eid;
    long esalary;
    char * ename;
    static char company_name[20];
    static int emp_count;

    public:

    static char * getcompanyname(){
        return company_name;
    }
    static int getempcount(){
        return emp_count;
    }
    void set(int empid);
    void set(long empsalary);
    void set(char empname[]);
    int getid();
    long getsalary();
    char * getname();
    employee(int empid=0,long empsalary=0,char empname[]="NA"){
        eid=empid;
        esalary=empsalary;
        ename=new char[strlen(empname)+1];
        strcpy(ename,empname);
        emp_count++;
    }

    employee(employee &ref){
        eid=ref.eid;
        esalary=ref.esalary;
        ename=new char(strlen(ref.ename)+1);
        strcpy(ename,ref.ename);
    }

    ~employee(){
        delete(ename);
    }

    employee operator+(employee &ref){
        employee temp(*this);
        temp.esalary=esalary+ref.esalary;
        return(temp);
    }
    employee& operator= (employee &ref){
        eid=ref.eid;
        esalary=ref.esalary;
        return * this;
    }

}e1,e2,emp;

然后在主目录中：

emp=e1+e2;

Answer 1

说实话，您的代码无效。它甚至不应该编译，因为它违反了引用绑定规则：+ 运算符返回一个临时对象，它不能通过非 const 引用传递给 = 运算符。如果您设法编译了此代码，则仅意味着您的编译器接受它作为语言的“扩展”。

要修复该特定错误，您必须在声明中添加一堆 const 限定符

employee operator +(const employee &ref) const {
  employee temp(*this);
  temp.esalary = esalary + ref.esalary;
  return temp;
}

employee& operator =(const employee &ref){
  eid = ref.eid;
  esalary = ref.esalary;
  return *this;
}

这将使您的代码从 C++ 的角度来看有效，但它可能无法修复崩溃，因为崩溃的原因必须在其他地方。

---

这是导致崩溃的错误：在复制构造函数中您执行了此操作

ename=new char(strlen(ref.ename)+1);

当你用new分配一个数组时，你必须使用[]括号，而不是()

ename = new char[strlen(ref.ename) + 1];

您在第一个构造函数中正确执行了此操作，但由于某种原因，您在复制构造函数中使用了() 而不是[]。 () 在这种情况下意味着完全不同的东西：它分配一个 char 并将其初始化为 strlen(ref.ename) + 1 值。

顺便说一句，您没有在复制赋值运算符中复制ename 的原因是什么？

此外，使用new[] 分配的内存必须使用delete[] 释放。不是delete，而是delete[]。你的析构函数应该是这样的

~employee() {
  delete[] ename;
}

---

最后，您最好使用std::string 存储ename，而不是依赖原始内存管理。 （除非您被明确要求这样做）。