WW #3

[Sokolov Viacheslav]

1. Здесь ошибка.

         free(l.head);
         free(l.tail);
   

2. &(*x) это x. Лишний код стоит удалить.
3. Проверка контрактов функции: не хватает assert-ов. Контракты: указатели ненулевые, вершина лежит (или не лежит) в списке и тд
4. Может быть удобно: ​https://en.cppreference.com/w/c/string/byte/isdigit
5. Вместо 1,...6, лучше использовать enum.

[Sokolov Viacheslav]

Сейчас программа не работает на примере из условия.
Предлагаю использовать sscanf, чтобы поправить эту проблему.

Именование: nod и node рядом сводят с ума

Еще контракт: при добавлении точки ее нет в списке, а при удалении - есть. Удобно завести отдельную функцию для проверки, есть ли точка в списке. Тогда, в частности, в release сборке от проверки контракта не будет никакого оверхэда.

commands можно сразу инициализировать:
char* commands[6] = {"add", "rm", "len", "rma", "print", "exit"};

Хотелось бы изолировать все детали реализации списка в clist.c, в частности, знание о фиктивной вершине. Для этого стоит сделать функцию, которая возвращает "настоящую" голову.

[Sokolov Viacheslav]

Имелось в виду

intrusive_node *get_head(intrusive_list *list) { return list->head->next; }

чтобы в main.c фигурировали только такие указатели, что от них безопасно звать container_of.

В частности, желательно, чтобы free(l->head) располагался в clist.c

Желательны пустые строки между функциями.

[Brilliantov Kirill]

Насчет getHead. Я не очень понимаю зачем делать как Вы говорите, если в моей реализации list->head указывает на элемент, который был последним добавленным. А если в списке нет элементов, то has_next(...) вернет -1 и container_of вызыван не будет.

[Sokolov Viacheslav]

Вроде бы устно обсудили, что имелось ввиду.