Opened 3 years ago

Closed 3 years ago

#527 closed ожидается проверка (задача сдана)

WW #10 (DK)

Reported by: kutyavin.denis Owned by: Святослав Власов
Component: WW cpp_io Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверка → ожидаются исправления

Тесты прошли

По стилю:

  1. Не хватает проверок и инвалидации потока на ошибках чтения
  2. Лучше юзать std::endl вместо '\n'
  3. Чтение/запись общих полей лучше вынести в базовый класс и вызывать из наследников, чем копи-пастить их
  4. Фабричный метод static Employee* Employee::CreateEmployee(type) избавит тебя от копи-пасты в EmployeeArray и main

8/10

comment:2 Changed 3 years ago by kutyavin.denis

Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

comment:3 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: assigned → closed
  1. Функции ***_general не нужны -- ты мог просто вынести общий код в Employee::write/read_text/bin и вызывать их из наследников напрямую Employee::write_text(out);
  2. Зачем побайтово читать строку? Есть же getline, который третьим аргументом принимает символ-разделитель.

В остальном хорошо, 9.5/10

(если вдруг за час пофиксишь -- можешь переоткрыть)

comment:4 Changed 3 years ago by kutyavin.denis

Resolution: задача сдана
Status: closed → reopened
Version: 2.0 → 3.0

comment:5 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: reopened → closed
  1. В read/write_bin ты догадался вынести общий код в Employee, а для read/write_text почему-то нет.

9.9

Note: See TracTickets for help on using tickets.