Change History (5)

comment:1 Changed 5 years ago by Vasily Alferov

Молодец!

Стиль: 3/3
Корректность: 7/9

  • Формат вывода неправильный: нету скобочек.
  • remove_all_points: ты сначала удаляешь элемент, а потом берёшь у него next. Таким образом, ты обращаешься к освобождённой памяти.
  • Ты выделяешь фиктивный элемент в куче, а в intrusive_list хранишь указатель на него. Это плохо по двум причинам. Во-первых, ты так забыла его освободить при выходе из программы. Во-вторых, тут на одно выделение в куче больше.

comment:2 Changed 5 years ago by Vasily Alferov

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

comment:3 Changed 4 years ago by Екатерина Иценко

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

comment:4 Changed 4 years ago by Vasily Alferov

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

Что-то лучше не стало :(
Кажется, если голову не выделять вообще нигде, то это точно неправильно.
Я имел ввиду

struct intrusive_list {
  struct intrusive_node head;
};

Без указателя, как было на паре.

Ну и да, в rma ты имела ввиду удалять небось не по значению, а конкретную точку. Можешь сама подумать, почему удалять по значению не работает.

comment:5 Changed 4 years ago by Egor Suvorov

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