WW #11

[Дмитрий Свиридкин]

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

Тривиальные конструкторы лучше отдать компилятору.
Чтобы запретить конструкторы копирования лучше использовать = delete, а не реализовывать их в приватной секции со странным поведением -- вы ими можете случайно воспользоваться при реализации других методов.

read_c_str должен читать в существующий буфер, а не создавать новый. Иначе в параметре, ограничивающем длину, никакого смысла.

свойство mutable в манипуляторах не нужно.

    unsigned char res;
    res = (this->value) ? 1 : 0;

Лучше не разрывать объявление и инициализацию.

Назначение константы size_of_byte = 4 -- не ясно. Скорее всего, она вообще не нужна.

 out << (std::string) str;

C-style каст вместо вызова конструктора -- очень странная запись. Обычно все-таки std::string(str)
C-style кастами в C++ лучше не пользоватсья. Тут есть static_, dynamic_, reinterpret_ casts.

У вас введены отдельные константы (лучше сделать один enum) для разных типов работников, а в коде читающем вместо них используются магические числа 1, 2.
Код, читающий и определяющий тип работника, дублируется минимум дважды. Его можно завернуть в фабричную функцию Employee* ReadEmployee(std::istream&)

---

6.5/10