WW #4

[Georgiy Angeni]

Я осознал, что делает макрос, который был дан в подсказке, но я не догадался, как обработать указатель на node, чтобы получить поле структуры point, а не какую-то рандомную штуку, как получается у меня. Как следствие, всё, где используется container_of, работает некорректно.

[Georgiy Angeni]

Вроде разобрался, все должно работать корректно.

[Святослав Власов]

Корректность -- 4/8.

1. Точки выводятся не в том порядке, посмотри на пример из задания и сравни со своим выводом.
2. Длинная неизвестная команда заставляет твою программу спамить сообщениями Unknown command.

Стиль -- 2/4

1. Макрос container_of лучше поместить в заголовочный файл. И перед инклюдами ничего стоять не должно.
2. Длинные строки (>80 символов) нужно подрезать
3. От tmpDel можно избавиться. У тебя ведь есть указатель на point, а в ней есть нужное тебе поле.
4. Может как-то от копипасты внутри show_all_points избавиться?
5. Зачем в remove_point удалять точки с конца, если можно делать это с начала. Это упростит код.
6. Память на куче для буфера команд выделять не нужно. Можно обойтись массивом на стеке.

[Святослав Власов]

Корректность 7/8

1. Добавление ноды должно происходить за константное время

Стиль -- 3.5/4

1. Макрос container_of зависит от stddef.h его нужно включить до объявления макроса
2. Не стоит юзать strtok, у тебя есть scanf и sscanf, которые намного проще и удобнее, пользуйся ими.

[Святослав Власов]

12/12