Opened 5 years ago

Closed 5 years ago

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

WW #3

Reported by: abramov.nikita Owned by: Sokolov Viacheslav
Component: WW_intrusive_list Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 5 years ago by Sokolov Viacheslav

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

У .o не хватает зависимостей от .h файлов

Как я расписал в ​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 проверять, что вершины отсуствует / присутствует, для чего удобно сделать вспомогательный метод

нужны переводы строк между функциями

сейчас формально есть проблема при считывании 240 символов: strcmp предполагает, что строка 0-терминированная, а это будет не так.

вместо not_end можно просто делать break из цикла

В задании запрещено делать вот так

list->head = del->next; в main.c, все модифицации списка разрашены только в clist.c

Имена find, del, next_find не очень удачные. Возможно, лучше будут current_node, match, next_node.

comment:2 Changed 5 years ago by abramov.nikita

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

comment:3 Changed 5 years ago by Sokolov Viacheslav

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