Opened 5 years ago

Closed 5 years ago

#206 closed ожидаются исправления (задача сдана)

WW #3

Reported by: fedorov.vladimir Owned by: fedorov.vladimir
Component: WW_intrusive_list Version:
Keywords: Cc:

Description


Change History (4)

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

Owner: changed from Дмитрий Лапшин (lapshin) to fedorov.vladimir
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. init_list не должен выделять память.
  2. add_node/remove_node: можно проще.
  3. Где-то лажа: иногда ты остаёшься у разбитого списка. Перепроверь add_node/remove_node на корректность, кажется ты творишь дичь.
  4. В том числе ты список наизнанку хранишь. Постарайся напрямую всё-таки ;)
  5. Undefined behaviour в add_point.

Стиль:

  1. strcmp(cmd, "Hello").

8/10.

comment:2 Changed 5 years ago by fedorov.vladimir

Owner: changed from fedorov.vladimir to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка

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

Owner: changed from Дмитрий Лапшин (lapshin) to fedorov.vladimir
Type: ожидается проверка → ожидаются исправления
  1. add_node/remove_node: можно проще. Подумай о том, какие предложения из веток на деле одинаковые в каждой. И вообще случаются ли они, или можно объявить UB.
  2. remove_point: что случилось в конце функции?
  3. Не страдай, printf("(%d %d)", a, b))
  4. command зачем глобальный?

10/12.

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

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.