WW #11

[Дмитрий Свиридкин]

Вроде работает

Строку в конструктор надо бы передавать по константной ссылке

Деструкторы наследников, если они тривиальные, можно не определять.

создавать объект, зачем читать, затем копировать его, а потом удалять исходный?
Создайте объект в куче вместо стека и отдайте владение первым указателем сразу массиву (нужно где-то убрать или добавить const)

Код с определением типа и чтением из main надо убрать. Этот код должен быть частью функционала модуля, а не пользовательским кодом.

Магические константы для типов стоит заменить на enum/именованные константы

7.5/10

[luchinin.aleksey]

Исправил недочёты

[Дмитрий Свиридкин]

Конструктор Developer принимает строку по неконстантной ссылке.
SalesManager - по копии

    void read_employee(std::istream &istr, EmployeesArray &employees);

    void read_bin(std::ifstream &ifstr, EmployeesArray &employees);

    void read_bin_employee(std::ifstream &ifstr, EmployeesArray &employees);

Эти методы, видимо, должны быть статическими, либо только с одним параметром.
а то это какой-то кошмар: array.read_employee(std::cin, array);

---

void Employee::convert_to_string_name(char *buf) {
    int cnt = 0;
    while (buf[cnt] != '\0') {
        name_ += buf[cnt++];
    }
}

У строк есть оператор присваивания.

---

8.9/10