WW10

[morozov.nikita]

F

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

Тесты прошли.

1. Когда функция проверяющая состояние потока называется check_str -- это вводит в заблуждение, потому что str обычно сокращают string. Имя check_stream или check_state подошло бы лучше.
2. К тому же эта функция переписывается одной строчкой assert(!str.fail() && "...");. Потому что состояние basic_ios::bad является подмножеством состояния basic_ios::fail.
3. Юзай std::endl вместо '\n'
4. Пустые тела в функциях Employee::read/write/... не нужно делать. Их либо делают абстрактными = 0, либо переносят в них общий код и вызывают их из классов-наследников. Второй вариант в твоем случае лучше подойдет, избавит от копи-пасты.
5. std::string s = ""; -- избыточно. Дефолтный конструктор строки и без того инициализирует её пустой.
6. Фабричный метод позволит избежать копи-пасты в операторах >> для EmployeeArray

9/10

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

Если виртуальную функцию не предполагается использовать у базового класса (как в случае с salary, лучше сделать её абстрактной.

В остальном вроде всё ок, 10/10