WW #10

[savinov.vladislav]

...и конкурсы интересные

[Святослав Власов]

Тесты прошли

1. Лучше функции Employee::read/print[_binary] делать не абстрактными, а вынести в них общий код и вызывать их из наследников (это можно делать из класса наследника обращаясь напрямую к функции базового класса, например Employee::print(out);). Так будет чуть меньше копипасты.
2. Зачем в конструкторах наследников повторно инициализировать поля, которые уже проинициализированы в конструкторе базового класса?
3. Из EmployeeArray::get_employees() лучше возвращать ссылку. Возвращая объект по значению, у тебя создается копия вектора при каждом вызове get_employees(). Вообще тебе эта функция не нужна, потому что операторы ввода-вывода у тебя объявлены как friend, а значит они имеют доступ к приватным полям класса.
4. Можно перегрузить оператор >> для обычного потока и файлового, тем самым избежав вынесения кода чтения сотрудника из консоли в main. А еще если ты сделаешь фабричный метод static Employee* Employee::CreateByType(int type), то ты избавишься от копи-пасты в этих операторах.

В остальном очень хорошо. 8/10

[Святослав Власов]

"Base Salary" зачем-то дважды у девелопера выводишь. Выглядит как опечатка, баллы не снижал

В остальном ок, 10/10