WW_cpp_io Panenkov lab_10

[Semyon Panenkov]

Давайте уже потихонечку на CMake переходить. Это куда более мощный и удобный инструмент. А ещё у него нормальная поддержка в CLion.

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

Давайте уже потихонечку на CMake переходить. Это куда более мощный и удобный инструмент. А ещё у него нормальная поддержка в CLion.

Я не возражаю и не буду запрещать. Если вам удобнее писать CMake файлы - пишите. Рассказывать на парах я буду когда-нибудь, когда у нас будет нечего другого важного обсуждать

А вот автосгенерированные makefile - это очень опасная тема. Их редко можно передать с компа на комп и надеяться, что само заработает

$ make
make: /usr/local/Cellar/cmake/3.17.2/bin/cmake: Command not found
Makefile:226: recipe for target 'cmake_check_build_system' failed
make: *** [cmake_check_build_system] Error 127

Пока что подставил другой Makefile, чтобы проверить работоспособность
Тесты проходят, но ругаются на вывод лишних пробелов

Стиль:

• employee.h и employee.cpp не определились, где же будет реализация функций (я намекаю на метод salary())
• в функциях read и write у потомков явно есть копипаста. От неё хотелось бы избавиться, вынеся общую часть в родителя
• хочется функцию createEmployee отдать в какой-нибудь класс. А то негоже ей торчать снаружи, как будто ею можно пользоваться без этих классов

7/10

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

Тесты пройдены

10/10