Opened 3 years ago

Closed 3 years ago

#191 closed ожидаются исправления (задача сдана)

WW #5 Bobrov

Reported by: bobrov.sergej Owned by: Святослав Власов
Component: WW_c_io Version: 1.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 3 years ago by Святослав Власов

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

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

  1. Исполняемый файл должен класться в директорию с проектом, папки bin быть не должно.

Стиль 1.8/3

  1. В Makefile неверно указаны зависимости. Объектники должны зависеть от заголовочных файлов тоже, если они в них включаются
  2. Длинные строки (>80 символов) нужно разбить на несколько
  3. Макрос container_of лучше поместить в clist.h, т.к. этот макрос относится напрямую к функциям работы со списками, а не специфичен для списка точек.
  4. Кажется, что создавать структуру для списка на куче в функциях load***-- не самая хорошая идея. Будет лучше, если эти функции будут принимать указатель на список в качестве аргумента, тогда список можно будет выделить на стеке в main и не нужно будет его освобождать.
  5. Функция split_bin тебе не понадобится, если ты хорошенько помедитируешь над вот таким куском кода:
    int x = ...;
    uint8_t* = &x;
    
  6. Если уж используешь uint8_t, то когда пишешь их в файл, стоит писать sizeof(uint8_t), а не char, так будет более консистентно.

Бонус +2/2

comment:2 Changed 3 years ago by Святослав Власов

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