Opened 3 years ago

Closed 3 years ago

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

HW #10

Reported by: Georgiy Angeni Owned by: Святослав Власов
Component: WW cpp_io Version: 2.0
Keywords: Cc:

Description

Понимаю, что код суперкостыльный и страшный, но я всё-таки решил заранее отправить, что есть, потому что пока нету идей как сделать лучше.

Change History (3)

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

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

Почти все тесты попадали -- как минимум у тебя проблема в формате вывода, ты там путаешь заглавные и строчные буквы. Может быть есть и еще проблемы, но их нельзя выловить пока ты формат не пофиксишь.
А еще тест на загрузку 1000 записей упал с сегфолтом

По стилю

  1. Лучше использовать std::endl вместо '\n' в выводе
  2. Ты дублируешь ввод/вывод полей Employee в Developer и SalesManager, у тебя там копи-паста. Можно вместо этого явно вызвать функцию у базового класса, например Employee::file_output(stream);
  3. Фабричный метод static Employee* Employee::CreateByType(type) позволит тебе избежать копи-пасты с созданием сотрудника в add и load

comment:2 Changed 3 years ago by Georgiy Angeni

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

Пофиксил все, о чем было сказано. Разве что так и не понял, что может быть не так с выводом, ибо у меня все работает, как над кодом не издевайся. Единственное - это то, что я заменил функцию size на length, когда смотрю, сколько бит строки надо писать в файл. Если дело было в этом, то супер, а если нет, то я без понятия, что делать :(

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

Resolution: задача сдана
Status: assignedclosed

Всё ок, 10/10

Note: See TracTickets for help on using tickets.