Перегрузка операторов наследование с классами

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; 
} 

с этим в основной:

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

Вы подаете оператор '+ =' на указатель т o Компания вместо Компания экземпляр или ссылка на экземпляр. Выведите указатель на указатель, например (*company) += a; будет компилироваться без рекламаций.

Пожалуйста, не перегружайте операторов указателями, сначала перегружайте ссылки на константу, и берете их от этого.

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);