WW #5

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

Корректность -- 3/7

1. Не обрабатываются пустые файлы
2. Файлы нужно открывать на запись, а не на добавление. Из-за этого данные дублируются и падают тесты.

Стиль -- 1/3

1. Функцию apply нужно определить в clist.[h,c]
2. container_of лучше определить в clist.h, т.к. этот макрос относится к работе с абстрактными списками, а не со списком точек
3. Глобальные переменные -- это зло, не стоит ими пользоваться, особенно там, где без них прекрасно можно обойтись. Всё что нужно, всегда можно передать через аргументы функциям, параметр void* data ровно для этого и нужен.
4. print_point_bin_file сложно реализован. Можно проще. Посмотри на этот кусок кода и, я уверен, тебе придет решение:

   int x = ...;
   uint8_t* data = &x;
   data[0], data[1], ...
   

5. Аналогично loadbin
6. loadtext и loadbin лучше в отдельные функции вынести, main хочется разгрузить

[ushakov.vladislav]

print_point_bin_file упростил, но не понял, как можно упростить loadbin. Пустые файлы вроде бы корректно обрабатываются.

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

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

Стиль 1.5/3

1. Объявления функций должны быть в заголовочных файлах, get_point в том числе
2. Не надо использовать адресную арифметику там, где в этом нет необходимости. У тебя есть оператор ->, нет никакой необходимости вот так писать:

   uint8_t *korx = (uint8_t *)pnode;
   uint8_t *kory = (uint8_t *)pnode + sizeof(int);
   

3. Макрос container_of зависит от stddef.h, он должен быть включен до объявления макроса.

[ushakov.vladislav]

Исправил. Еще все-таки упростил loadbin.

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

10/10