Opened 4 years ago

Closed 4 years ago

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

WW #3 yanin.vasiliy

Reported by: Василий Янин Owned by: Василий Янин
Component: WW_intrusive_list Version:
Keywords: Cc:

Description


Change History (2)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Василий Янин
Summary: WW #3 → WW #3 yanin.vasiliy
Type: ожидается проверка → ожидаются исправления

Тут чего-то не очень хорошо ему.

У тебя голова списка хранит указатель на голову, а код написан, как будто:

  1. Голова всегда есть.
  2. Список не зациклен.

Это хорошая реализация, но

  1. Этот указатель надо во что-то выставить (выделив память?)
  2. Потом эту память, видимо, удалить.

А лучше вместо этого хранить не указатель на элемент, а сам элемент.

Течёт память: очистка списка плохая.
Весь код был бы проще, если бы ты всегда оперировал текущим, а не следующим элементом.

Отступы и пробелы местами не хороши, следи за стилем.

Пока на 5/7.

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

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