WW #5 bubnov.danil

[Дмитрий Лапшин (lapshin)]

Стиль:

1. Зачем тебе struct point в main?
2. Магические константы.
3. Если читаешь прямым fread, почему не fwrite напрямую?

8/10. Удачи!

[Бубнов Данил Константинович]

Убрал struct point(тоже не понял, зачем его делал). Про константы не понял, видимо это if (argc < 4), но а какую ещё делать). Поменял запись в бинарный файл, теперь это делается просто fwrite. Также сделал на доп балл: при чтении из бинарного проверяю старший бит в третьем байте, если он 1, значит число отрицательное, значит все биты в старшем 4 байте нужно сделать 1, делаю это с помощью побитового или.

[Дмитрий Лапшин (lapshin)]

Кроме магических констант (вынеси их) вопросов нет, 9/10)

[Бубнов Данил Константинович]

Я так и не понимаю до конца, какие я здесь использую константы. Что-то я использую только один раз, зачем это выносить. Не буду же я 3 выносить в отдельную константу. Единственное, я вынес команды из argv в отдельные переменные. И что насчёт дополнительного задания?

[Дмитрий Лапшин (lapshin)]

Конечно не будешь выносить 3 (хотя я бы вынес). А вот то, что есть и 3, и 23, и 0xFF000000 и все они связаны и повторяются это нехорошо. Это точно надо вынести, чтобы были рядом.

К сожалению, бонус не работает, если есть больше одной точки.

9.