WW #5

[Kazakov Maksim]

Вопрос по отрицательным числам. Можно ли выделять дополнительный байт в бинарном файле или нет?

[Egor Suvorov]

Попытка проходит все автотесты, кроме бонусного сохранения отрицательных чисел в бинарный файл.

Попытка сдачи есть, дедлайн теперь — до 3 ноября, 22:59. Подробно проверю позже.

Дополнительный байт выделять нельзя: три байта, дополнительный код.

[Kazakov Maksim]

Добавил возможность работы с отрицательными числами, исправил Makefile

[Egor Suvorov]

Корректность 6.5/7: не работает count.

По стилю 1.5/3:

1. Ставьте пробелы между условием if, самим if и {. Пройдитесь автоформаттером.
2. Добавьте assert в ветку else, который проверяет аргумент. Не стоит трактовать всё, что не loadtext, как loadbin. Аналогично, не стоит молча игнорировать неизвестные аргументы.
3. Не пишите пробелы внутри fscanf, они всё равно не нужны.
4. Объявите переменные x, y, px, py внутри цикла.
5. Не стоит обращаться напрямую по указателям: вы сейчас очень нагло пользуетесь знанием о том, в каком порядке идут байты в int. Лучше соберите число при помощи арифметических операций.
6. wtite --> write
7. datax --> data_x.

Бонус работает, за это +2.

Итого пока 10/10, можете добрать до 12/10, если исправите корректность и стиль.

[Kazakov Maksim]

Исправил все, кроме 5 пункта

[Egor Suvorov]

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

По стилю 2.5/3:

1. write_list_text --> write_point_text_file (я вообще не понял, при чём тут list), write_list_bin --> write_point_bin_file, print_list --> print_point.
2. sizeof(char) по определению в стандарте равен единице, лучше так тут и написать.
3. Не ставьте пробел в & x.
4. Очень стоит дописать честное арифметическое чтение int из бинарного файла. Там почти в точности как в write_list_bin.