Opened 4 years ago

Closed 4 years ago

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

WW #11 smirnov.ilya

Reported by: smirnov.ilya Owned by: Дмитрий Лапшин (lapshin)
Component: WW cpp_io Version:
Keywords: Cc:

Description


Change History (5)

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

Owner: changed from Дмитрий Лапшин (lapshin) to smirnov.ilya
Summary: WW #11 → WW #11 smirnov.ilya
Type: ожидается проверка → ожидаются исправления
Version: 1.0

Работает! А теперь про интересное:

  1. Тела методов/функций в заголовочном файле. inline? Вынести? Выбирай мудро :D
  2. Если friend не обязателен, то не нужен.
  3. Ты используешь std::string только чтобы хранить строчковые литералы… Что бесполезно. Наоборот, смысл ей заменить все char *. Чтобы не было delete[], new[], strncpy (кстати n).
  4. read_c_str — зачем два параметра-то?
  5. _type явно не поле Employee. Да, эта штука нужна в чтении-записи, потому что как догадаться что перед нами в файле, но когда объект жив оно уже не нужно ;) Пахнет оно и тем, что только присваивается в записи.

Так что полировать можно. 8.

comment:2 Changed 4 years ago by smirnov.ilya

Но ведь в условиях требуется реализация read_c_str с двумя параметрами : read_c_str(z, sizeof z)

(хотя мне sizeof и правда не нужен)

Last edited 4 years ago by smirnov.ilya (previous) (diff)

comment:3 Changed 4 years ago by smirnov.ilya

Owner: changed from smirnov.ilya to Дмитрий Лапшин (lapshin)

comment:4 Changed 4 years ago by smirnov.ilya

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

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

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

Что я теперь вижу:

  1. Дублируется код создания работника.
  2. Дублируется код вывода общих полей.

Но в остальном прекрасно, 10, обсудим на паре.

Note: See TracTickets for help on using tickets.