WW#10

[Daniil Lyubaev]

Пока без обработчика ошибок :(

[Daniil Lyubaev]

Добавил проверку ввода

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

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

Стиль

1. Виртуальные функции хочется убрать в protected
2. Вместо ручной проверки диапазонов символов лучше воспользоваться библиотечной функцией std::is_alnum ​https://en.cppreference.com/w/cpp/string/byte/isalnum
3. Выносить детали чтения сотрудников в main -- плохо. Ты перегружал оператора вывода ровно для того, чтобы не было необходимости так делать. Во-первых, сделай фабричный метод, который будет по типу создавать нужного сотрудника, и читай в него данные с помощью оператора >>.

В остальном всё классно. 9/10

[Daniil Lyubaev]

1. UPD.: Не мог правильно сделать, потому что не то делал. Поправил, сейчас должно быть нормально
2. Добавил
3. Добавил

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

У тебя явная копипаста в read_from_file у девелопера и сейлс менеджера, её так и хочется вынести в отдельную функцию. Самое лучшее для этого место -- эта же функция у базового класса, которую из обоих наследников можно явным образом вызвать Employee::read_from_file(inf);

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

[Daniil Lyubaev]

Поправил.

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

10/10