WW #5

[korolev.dmitriy]

Вроде как все работает, единственное реализовано не через apply и с флагами на ошибки будет падать, завтра поправлю, чтобы все было хорошо, пока что хочу знать нет ли других замечаний

[korolev.dmitriy]

Все поправил, теперь должно быть хорошо(некоторые строчки могут быть абсолютно бессмысленными и бесполезными, чтобы переменные, которые я передаю в функции были хоть как-то использованы)

[Дмитрий Свиридкин]

Не создавайте повторные тикеты! Вы так усложняете проверку. Модифицируйте исходный. Учитесь пользоваться поиском: View tickets -> My tickets.
При повторении будем штрафовать.

1. Течет память

2. Зачем изобретать странные переменные, когда нужно проверить число переданных аргументов?

3. файлы периодически не закрываются

4. Почему функция, которая не открывает файл, а читает из уже открытого, называется open_* ?

5. Преобразование из байтов в число некорректно. char бывает знаковым. У вас бывает UB.

6. Используйте типы с фиксированной длиной: uint8_t. int32_t и т.д.

7. Откуда взялась странная константа c, ограничивающая число итераций циклов???

8. Сделайте отдельные функции, преобразующие числа в байты и обратно, чтобы не копипастить.

9. Зачем делать get_point по нескольку раз в одном и том же месте? Не жадничайте на переменные.

10. Объявляйте переменные как можно ближе к их использованию. Переменная i объявлена за 24 строки до точки ее единственного использования. И почему она называется i?

11. Зачем вы продолжаете парсить форматную строку?

12. x = x -- самый худший nop, который можно придумать. Если у вас не используется аргумент функции, можно просто не указывать его имя в определении.

---

4/10

[korolev.dmitriy]

Исправил все кроме 6,там в задании числа по модулю не больше 5 миллионов, так что все должно быть хорошо, но если это влияет на оценку, то я исправлю.

UPD:(7) константа c у меня взялась после случая, когда я случайно сделал txt файл на 7 гб)

[Дмитрий Свиридкин]

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

[Дмитрий Свиридкин]

int x = 257;
char y = x; переполнение знакового числа -- UB.

2. преобразование чисел из байтов некорректное из-за знаковости char: число [128, 128, 0] будет прочитано неправильно

3. Дополнение до 2х так не работает

4. Зачем в main нужна переменная a?

5. Имелось в виду вынести функцию преобразования одного числа, чтобы избавиться от копипасты, когда преобразуете второе...

---

6/12