Opened 3 years ago

Closed 3 years ago

#258 closed ожидается проверка (задача сдана)

HW #1 (BMP) Kozyrev Andrei hw_01

Reported by: Andrei Kozyrev Owned by: Антон Филатов
Component: HW #1 (BMP) Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Антон Филатов

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

Поначалу простые тесты проходят. Но что-то идёт не так, если в файле строчка не делится на 4
Например есть файл с размерами 685 на 610 и при попытке повернуть его полностью - экзит статус вашей программы не 0
Так что пока это 8/20 за корректность
Стиль:

  • все ли функции из bmp.с достойны быть в хедере? или некоторые могут быть статическими внутри bmp.c?
  • может, завести структуру под RGB? чтобы не делать цикл из трёх итераций?
  • проверьте, везде ли стоят const в аргументах функций?

Стиль: 7/10

comment:2 Changed 3 years ago by Andrei Kozyrev

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

Все пофиксил
Все залил

comment:3 Changed 3 years ago by Антон Филатов

Resolution: → задача сдана
Status: assigned → closed

Тесты пройдены, 20/20 за корректность

Стиль: 9/10 - мне по-прежнему не нравится массив из 3 элементов вместо структуры RBG, поскольку в текущем виде кажется, что надо наизусть помнить, что какой элемент массива означает

Note: See TracTickets for help on using tickets.