Opened 3 years ago
Closed 3 years ago
#267 closed ожидается проверка (задача сдана)
HW#1
Reported by: | kutyavin.denis | Owned by: | Святослав Власов |
---|---|---|---|
Component: | HW #1 (BMP) | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Размер изображения и файла в хедера по прежнему пишется неверный
15/20 за корректность
Стиль -- 8/10
- Функции, локальные для
bmp.c
стоит сделать статическими - В
rotate
пиксель можно копировать целиком с помощьюmemcpy
, цикл из трех итераций смотрится неуместно - Количество байт на пиксель стоит абстрагировать в константу
Бонус работает, но зачесть смогу только после полного балла за корректность
Из замечаний -- путаешь порядок следования байтов в пикселе, прочитай на вики в каком порядке они следуют.
comment:4 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
Note: See
TracTickets for help on using
tickets.
Лена целиком поворачивается корректно, но размер файла в хедере пишется неверный.
Вырезается не тот кусок. Например с аргументами 100 50 128 128 у тебя вырезается кусок, тот что слева, а должен тот, что справа:
За корректность 5/20 пока
Стиль -- 4/10
crop
можно копировать строки не побайтово, а сразу целикомcrop
иrotate
уничтожают исходное изображение. Это неочевидное поведение, обычно таким должен заниматься вызывающий код.