WW_cpp_io Skalt (lab10)

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

$ make
mkdir -p obj
g++ -O2 -Wall -Werror -std=c++11 -Iinclude -c -MMD -o obj/main.o src/main.cpp
g++ -O2 -Wall -Werror -std=c++11 -Iinclude -c -MMD -o obj/employees.o src/employees.cpp
src/employees.cpp: In member function ‘void EmployeesArray::list()’:
src/employees.cpp:52:23: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     for (int i = 0; i < _employees.size(); ++i) {
                     ~~^~~~~~~~~~~~~~~~~~~
cc1plus: all warnings being treated as errors
Makefile:18: recipe for target 'obj/employees.o' failed
make: *** [obj/employees.o] Error 1

Да, по структуре у вас сейчас всё правильно: виртуальный print/scan, который вы правильно дёргаете от базового класса в наследниках, а также в операторах << и >>,
фабричный метод тоже правильный. (его, правда, можно в EmployeeArray? засунуть - это всё же часть класса по-хорошему, а не совсем отдельная функция)

И сделайте, наверное, pragma once - это короче, чем такие вот дефайны

Пока без оценки, но вы идёте в правильную сторону

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

Да, сейчас это выглядит очень внушительно и очень правильно
10/10

[skalt.albert]

Спасибо, что проверили!)