WW_cpp_io Martynov lab_10

[martynov.maksim]

gg wp

[Антон Филатов]

Так, тесты у вас проходят. А вот архитектура немного необычна.
Вы решили пользоваться статик функциями. И, кажется, что везде можно как-то этого избежать. Во-первых функции записи/чтения int - это могут быть просто глобальные функции, поскольку они никакого отношения к Employee не имеют.
Во-вторых вы изо всех сил старались избежать копипасты, но копипасту в print_specific_employee в наследниках не углядели. Её можно вынести в функцию базового класса.

• Вот уже три человека, которые делают одну и ту же непонятную ошибку и пишут тело функции salary в хедере. Кто с кого списывал?
• константы следует положить в классы - они же их части. Идеально было бы, чтобы employee ничего не знал о своих потомках
• В догонку к предыдущему коментарию, функции create_by_type не место в employee, поскольку вы таким образом заставляете родителя знать о своих потомках. А это нарушение правил ООП
• ну и напоследок волшебная константа in.read(&c, 1) - есть же sizeof

7/10

[Антон Филатов]

10/10