Opened 4 years ago

Closed 4 years ago

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

WW #11 turov.kirill

Reported by: Кирилл Туров Owned by: Дмитрий Лапшин (lapshin)
Component: WW cpp_io Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Summary: WW #11 → WW #11 turov.kirill
Type: ожидается проверка → ожидаются исправления

Работает!

По стилю:

  1. Заголовочные файлы: мы стараемся не использовать using. Тем более потом std:: у тебя мелькает.
  2. Неконсистентные пробелы (например Employee.hpp).
  3. Неконсистентные имена: Employee &worker это странно, наверное Employee &employee?
  4. read|write_le_int32 — не гарантирует little endian.
  5. Обычно io цепочкой пишут, но и так можно.
  6. Дублирование кода создания работника.

Можно и нужно исправлять. Пока 8.

Last edited 4 years ago by Дмитрий Лапшин (lapshin) (previous) (diff)

comment:2 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Кирилл Туров

Прости

comment:3 Changed 4 years ago by Кирилл Туров

Owner: changed from Кирилл Туров to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

comment:4 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed

Увидел дублирование кода (фабрика, наследование вывода) и магические константы. 9.

Note: See TracTickets for help on using tickets.