WW #11

[Sokolov Viacheslav]

lab_11: src/employees.cpp:88: virtual void Developer::fstream_read(std::ifstream&): Assertion `_has_bonus' failed.

Классы стоит помечать как final, если от них не предполагается наследоваться, а конструкторы делать explicit (нужно ли пояснить, почему?)

20 in.read(&c, 1);

можно же просто in.read(c)? (с write та же история)

57 out.write(data.x, strlen(data.x));

можно же сразу strlen+1?

Рекомендую посмотреть на ​​https://en.cppreference.com/w/cpp/io/basic_istream/getline

void add(const Employee *e)
такой интерфейс порождает неудобство, требуется копирование. Проще было бы исправить сам интерфейс (принимать объект во владение). В неучебных целях здесь стоило бы использовать std::unique_ptr.

Между 7 void add(EmployeesArray?& array) и 202 std::ifstream& operator >> (std::ifstream& in, EmployeesArray?& array) {
код дублируется, может быть, можно с этим что-то сделать?

Сейчас в этих функциях явно прописаны значения 1,2, из-за чего страдает расширяемость - при добавлении новых Employee будет тяжело следить за тем, что везде все правильно поддержано. Стоит использовать enum (class), чтобы держать все возможные значения в одном месте.

[Sokolov Viacheslav]

195 Developer* cur = new Developer();
196 assert(in.good());
197 in >> *cur;
198 assert(!in.fail());
199 array.add(cur);
200 } else if (type == Employees::SALES_MANAGER) {
201 SalesManager?* cur = new SalesManager?();
202 assert(in.good());
203 in >> *cur;
204 assert(!in.fail());
205 array.add(cur);
здесь можно было бы избежать дублирования кода: единственное отличие - в типе при аллокации