Opened 5 years ago

Closed 5 years ago

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

WW #3

Reported by: sukhodolskiy.maksim Owned by: Sokolov Viacheslav
Component: WW_intrusive_list Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 5 years ago by sukhodolskiy.maksim

Version: → 1.0

comment:2 Changed 5 years ago by Sokolov Viacheslav

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

#define container_of(ptr, type, member) (type*)((char*)(ptr) - offsetof(type, member))
тоже нужно занести под include-guard

Расписал ​https://wiki.compscicenter.ru/index.php/C%2B%2B_1MIT_осень_1_2019#.D0.A2.D1.80.D0.B5.D0.B1.D0.BE.D0.B2.D0.B0.D0.BD.D0.B8.D1.8F_.D0.BA.D0.BE.D1.80.D1.80.D0.B5.D0.BA.D1.82.D0.BD.D0.BE.D1.81.D1.82.D0.B8.2C_.D0.BF.D1.80.D0.B5.D0.B4.D1.8A.D1.8F.D0.B2.D0.BB.D1.8F.D0.B5.D0.BC.D1.8B.D0.B5_.D0.BA_.D1.80.D0.B0.D0.B1.D0.BE.D1.82.D0.B0.D0.BC и, в частности, не хватает:

  • проверок ненулевых указателей
  • желательно в функциях add_node / remove_node проверять, что вершины отсуствует / присутствует, для чего удобно сделать вспомогательный метод

С моей точки зрения для реализации списка с фиктивной вершиной будет неплохой мыслью сделать в пару к init_list метод deinit_list, который освободит выделенную память (чтобы детали реализации оставались в одном месте)

В Makefile не хватает зависимостей .o от .h

В целом работа сделана хорошо.

comment:3 Changed 5 years ago by sukhodolskiy.maksim

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

comment:4 Changed 5 years ago by Sokolov Viacheslav

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