Opened 4 years ago

Closed 4 years ago

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

HW #1

Reported by: vorobeva.polina Owned by: Vasily Alferov
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 4 years ago by Vasily Alferov

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

Офигенно, мне нравится.

Корректность

10/20.

  • Базовый тест проходит, и это круто.
  • Паддинг пока не реализован — надо реализовать. С учётом того, что всё остальное работает, это довольно просто — надо только честно реализовать.

Без паддинга половина баллов.

Стиль

9/10, одно замечание.

  • В rotate: у тебя buf используется только для того, чтобы хранить там указатели, а заголовки не используется. Кажется, логично его просто заменить на указатель.

comment:2 Changed 4 years ago by vorobeva.polina

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

Спасибо большое, что помог разобраться с паддингом и тонкостями хранения bmp-файла:)

comment:3 Changed 4 years ago by Vasily Alferov

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

Рад помочь =)

Круто, зачтено.

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

Вообще, выходной файл необязательно должен существовать. А ещё файлы закрывать за собой надо.

Какая-то беда в заголовках bfSize и biSizeImage =(
Вроде как они должны быть с учётом паддинга и тут я тебя обманул, сказав, что менять надо только load и save.

А ещё, кажется, область вырезается не та (надо было от левого верхнего угла отсчитывать).

По исправлению двух неточностей работает.

Это всё мелочи. Поскольку исправлять больше нельзя, тут снято совсем мало баллов.

Note: See TracTickets for help on using tickets.