Opened 3 years ago

Closed 3 years ago

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

HW #1 (BMP) Tyuryaev Ilya hw_01

Reported by: tyuryaev.ilya Owned by: Антон Филатов
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (6)

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

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

Интересная бага.
На лене работают практически все тесты.
Но на тесте с параметрами 25 50 130 130
раз примерно в 400 байт (наверное в паддинге, но это не точно) вместо нуля кладётся не ноль
С изначально неделящимся на 4 файлом такая же петрушка.

Стиль:

  • очень много волшебных констант в bmp.c
  • save_bmp с этой чередой fread и fwrite выглядит максимально неожиданно. Может весь fread оставить в load_bmp, а в save пусть будет только fwrite?
  • проверьте, что у вас везде const стоят, где надо в аргументах функций
  • возможно в load можно быстрее читать, чем по одному пикселю?
  • может, часть кода из main должна переехать в bmp? или хотя бы в функции отдельные? А то сейчас main какой-то чересчур длинный

Корректность: 8/20
Стиль: 5/10

comment:2 Changed 3 years ago by tyuryaev.ilya

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

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

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

Сейчас на лене все тесты проходят. Но с неквадратным изображением, не делящимся всё падает (явно ломается паддинг)

Бонус, кстати, работает правильно.
Но я за него пока не могу поставить баллы, пока вы не пофиксите корректность
Корректность: 11/20
Стиль: 10/10
Бонус: 0

comment:4 Changed 3 years ago by tyuryaev.ilya

Version: 2.03.0

comment:5 Changed 3 years ago by tyuryaev.ilya

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

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

Resolution: задача сдана
Status: assignedclosed

20 + 10 + 10

Note: See TracTickets for help on using tickets.