У вас неправильно определены конструктор копирования, копирующий оператор присваивания и деструктор.
Например, конструктор копирования должен копировать строку, а не просто присваивать один указатель другому. Копирующий оператор присваивания должен сначала удалить текущий указатель name_
, прежде чем выделять новую память. А деструктор должен высвобождать выделенную память.
Ниже приведена демонстрационная программа, в которой показано, как они могут быть определены. Кроме того функцию copyString
следует объявить как статическую и закрытую.
#include <iostream>
#include <list>
#include <string>
#include <cstring>
class Employee
{
private:
char *name_;
private:
static void copyString( char **dest, const char *source )
{
*dest = new char[ std::strlen( source ) + 1 ];
strcpy( *dest, source );
}
public:
Employee( const char *name ) : name_( nullptr )
{
copyString( &name_, name );
}
Employee( const Employee © ) : name_( nullptr )
{
copyString( &name_, copy.name_ );
}
Employee() { delete [] name_; }
Employee & operator =( const Employee © )
{
if ( this != © )
{
delete [] name_;
name_ = nullptr;
copyString( &name_, copy.name_ );
}
return *this;
}
friend std::ostream &operator <<( std::ostream &out, const Employee &employee )
{
return out << employee.name_;
}
};
class Company
{
public:
Company( const std::string &name ) : name( name ) {};
virtual ~Company() = default;
void hire( const Employee &employee )
{
employees.push_back( employee );
}
friend std::ostream & operator <<( std::ostream &out, const Company &company );
private:
std::string name;
typedef std::list<Employee> EmployeeList;
EmployeeList employees;
};
std::ostream & operator <<( std::ostream &out, const Company &company )
{
out << company.name << "(" << company.employees.size() << "): ";
for ( const auto &employee : company.employees )
{
out << "\"" << employee << "\" ";
}
out << std::endl;
return out;
}
void addStuff( Company &company )
{
company.hire( Employee( "Max Mustermann" ) );
company.hire( Employee( "Erika Musterfrau" ) );
}
int main()
{
Company company( "OwesomeCo" );
addStuff( company );
std::cout << company;
return 0;
}
Вывод программы на консоль
OwesomeCo(2): "Max Mustermann" "Erika Musterfrau"
При желании вы можете добавить в класс Employee
также перемещающий конструктор и перемещающий оператор присваивания. Например,
Employee( Employee &© ) : name_( nullptr )
{
std::swap( name_, copy.name_ );
}
Employee & operator =( Employee &© )
{
if ( this != © )
{
std::swap( name_, copy.name_ );
}
return *this;
}