WW #5

[Egor Suvorov]

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

1. point_list.c зависит ещё и от clist.h.
2. print должен выводить перевод строки в конце.

Стиль 1.5/3:

1. Не n, а argc.
2. Добавьте assert, проверяющий, что argc хватает.
3. Лучше объедините цепочку if'ов на команду в else if, а в конце сделайте assert(!"Unknown command");
4. FILE *fout = fopen(...), сразу на одной строчке.
5. В savetext лучше цикл for, а не while. Или вообще apply.
6. fscanf() >= 0 лучше заменить на fscanf() == чтото
7. У loadtext/savetext параметры должны называться одинаково для симметрии. Лучше file_name.

[Egor Suvorov]

P.S. Попытка засчитана, окончательный дедлайн - 3 ноября, 22:59.

[Egor Suvorov]

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

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

Стиль 2.5/3:

1. Отделите в main пустыми строчками блоки: начало, конец, загрузка, выполнение команды.
2. Ветку loadtext тоже проверьте при помощи assert: программа должна падать, если дать ей неизвестный параметр.
3. Проверка argc > 1 недостаточно строгая. Можно строже.

[Egor Suvorov]

Некорректный формат в бинарных данных: в условии требуется записывать 4 байта, а не 3. Но если вы сейчас просто поменяете 4 на 3, то получите UB. Плюс надо использовать wb, а не w.

По стилю:

1. Лучше не > 3 и < 6, а >= 4 (минимальное количество аргументов) и <= 5 (максимальное). А ещё лучше — оставить только >= 4, а остальные проверки добавить в конкретные ветки с ==.
2. Объявляйте переменные внутри цикла, если можете.

[Egor Suvorov]

Корректность и бонус есть (7/7+2).

По стилю 2.5/3:

1. Поставьте в count assert на количество аргументов.
2. savebin: объявите pnode внутри цикла.
3. loadbin: незачем занулять переменные в конце итерации. fread перезапишет первые три байта.
4. text --> extend_sign_24bit, а функция binary вам просто не нужна.
5. В extend_sign_24bit вам надо идеологически не арифметику делать, а биты менять: поставить старшие в единицы. Это | с константой 0xFF000000.

[Egor Suvorov]

2.7/3, всё ещё актуальны замечания по стилю 1, 4, 5.

6. Внутри binary не нужен иф, можно просто всегда брать &. А на самом деле не нужен и сам binary, можно в savebin напрямую писать pnode->x/pnode->y.