Opened 3 years ago

Closed 3 years ago

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

WW #10

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

Description


Change History (3)

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

Resolution: → задача сдана
Status: assigned → closed
  1. Тест на чтение длинного имени упал :(
  2. Буфер под имя константной длины -- это плохо. Особенно когда вам стрингом разрешили пользоваться.
  3. Копи-паста в вводе-выводе. Хочется вынести общую часть ввода-вывода в базовый класс.

7/10

Если за пару часов пофиксишь -- можешь переоткрыть тикет.

comment:2 Changed 3 years ago by Kuznetsov Kirill

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

Постарался исправить, копипасту частично поправил (чтение типов осталось), сделал имя стринговым.

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

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

Ошибки лучше в std::cerr выводить, а не std::cout.
И вообще, операторы ввода-вывода не должны падать, они должны переводить поток в failed-состояние. А падать и ругаться ошибками должен код, который эти операторы вызывает.

Для выделения общей части не нужно было новые функции создавать, достаточно было вынести общую часть в те же самые read и print у базового класса, а потом из read и print классов наследников вызывать методы базового класса Employee::read(in);

В остальном всё ок, код стал сильно лучше и тесты прошли.
9.5/10

Note: See TracTickets for help on using tickets.