带有类的运算符重载继承[关闭]

Question

我是编码新手，我正试图了解这个程序有什么问题：

class Company:public Employee{
private:
    std::vector<Employee*> _table;
public:
    Company& operator+=(const Employee* emp) {
         _table.push_back(emp->clone());
          return *this;
     }
     virtual Company* clone() const {
         return new Company(*this);
     }
     virtual Company& setDescription(std::string des){
          _des=des;
     return *this;
}

在 main 中有这个：

Company* company = new Company();
a = new DeveloperEmployee(description, project);
int id = a->getID();
cout << *a << endl; //Developer ID = 2, project = hw5

company += a;

我有这个错误：

   error: invalid operands of types 'Company*' and 'DeveloperEmployee*' to binary 'operator+'|

Answer 1

您正在将运算符“+=”应用于指向 Company 的 指针，而不是 Company 实例或对实例的引用。 取消引用指针，例如(*company) += a; 将毫无怨言地编译。

请不要在指针上重载运算符，首先在 const 引用上重载，然后从中获取。

Answer 2

FWIW，使用

Company& operator+=(const Employee* emp) {
     _table.push_back(emp->clone());
      return *this;
 }

将Employee 添加到Company 是滥用+= 运算符的语义。

更好的选择是：

Company& addEmployee(const Employee& emp) {
     _table.push_back(emp.clone());
      return *this;
 }

然后，你可以使用：

Company* company = new Company();
a = new DeveloperEmployee(description, project);
int id = a->getID();
cout << *a << endl; //Developer ID = 2, project = hw5

company->addEmployee(*a);

这将使代码更易于阅读和遵循。

进一步改进

不要使用指向Employee 的指针列表，而是使用智能指针列表。

std::vector<std::share_ptr<Employee>> _table;

和

Company& addEmployee(std::shared_ptr<Employee> emp) {
     _table.push_back(emp);
      return *this;
 }

并将其用作：

Company* company = new Company();
std::shared_ptr<Employee> a(new DeveloperEmployee(description, project));
int id = a->getID();
cout << *a << endl;

company->addEmployee(a);