Opened 3 years ago

Closed 3 years ago

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

WW_cpp_io Tulchinskiy lab_10

Reported by: tulchinskij.aleksandr Owned by: Антон Филатов
Component: WW cpp_io Version: 3.0
Keywords: Cc:

Description


Attachments (1)

office-crew.edb​ (345 bytes) - added by Антон Филатов 3 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 3 years ago by Антон Филатов

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

Тесты проходят. Стиль:

  • хочется TYPE отдать родителю. А его значение установить потомкам в конструкторе
  • плохо, что у вас fabric_method лежит в родителе. Получается, что родитель должен что-то знать про своих потомков. Не надо так
  • read_employee не нужен статик
  • а можно ли избежать статика в read_bin_employee?

8/10

comment:2 Changed 3 years ago by tulchinskij.aleksandr

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

Changed 3 years ago by Антон Филатов

Attachment: office-crew.edb​ added

comment:3 Changed 3 years ago by Антон Филатов

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

Что-то у вас теперь некоторые тесты прямо с сегфолтом падают.
Тест на load на заранее созданном файле. Файл прикрепил в attachments в этом тикете

Стиль:

  • не хотите ли read_employee в EmployeeArray? переложить? Вы всё равно внутри read дёргаете фабрику, которая принадлежит EmployeeArray?. Кажется, что если чтение будет лежать в массиве, логика не пострадает

Всё ещё 8/10

comment:4 Changed 3 years ago by tulchinskij.aleksandr

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

Nope it's not Ashton Kutcher, it's Kevin Malone. Equally handsome, equally smart.

comment:5 Changed 3 years ago by Антон Филатов

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

10/10

Note: See TracTickets for help on using tickets.