WW #5

[Egor Suvorov]

Попытка сдачи засчитана.

[Egor Suvorov]

Корректность:

1. main.o на самом деле также включает clist.h. Как и point_list.o.
2. Утекает память.
3. print выводит лишние пробелы.
4. Не тот формат бинарных файлов, получается неверный ответ.

По корректности 4/7:

• Корректные заголовочные файлы +0.5
• Во всех исходных файлах подключены нужные заголовки и только они +0.5
• Корректное разделение по файлам, корректная реализация списка +1
• Корректно работает загрузка из текстовых файлов (в том числе отрицательные координаты) и команда print реализована при помощи apply (возможно, без параметра void *data) +1
• Корректно реализована команда count при помощи apply и без глобальных переменных +0.5
• Корректно работает запись текстовых файлов +0.5

Стиль 1/3:

1. В apply есть лишние переменные. Можно просто for (;, а ещё лучше объявить вместо счётчика node. Аналогичная проблема в других местах.
2. Поставьте assert на argc.
3. Объявляйте переменные внутри циклов.
4. Пробелы. Пройдитесь автоформаттером.
5. Не arg, а имя, описывающее, что лежит в переменной.
6. ​Мойте тарелки перед едой, смотрю на x и y.

[Egor Suvorov]

По корректности всё ещё 4/7:

1. Makefile и print стали норм.
2. Всё ещё утекает память. Пройдитесь валгриндом или санитайзером.
3. Всё ещё неверный формат бинарных файлов в savebin.

По стилю 1.5/3:

1. get_count(unused) — не unused, а node. Не get_count, а inc_count.
2. Объявляйте переменные внутри циклов.
3. Называйте X/x одинаково.
4. Вместо ручных циклов по спискам и переменной head на половину main используйте apply.
5. (int) в арифметических операциях не нужны, само добавится.