Opened 5 years ago
Closed 4 years ago
#482 closed ожидается проверка (задача сдана)
HW #1
Reported by: | yakovlev.aleksandr | Owned by: | Дмитрий Свиридкин |
---|---|---|---|
Component: | HW #1 (BMP) | Version: | 3.0 |
Keywords: | Cc: |
Description
Промежуточная попытка, должно работать с картинками любой размерности, но пока нет проверки аргументов.
Change History (6)
comment:1 Changed 5 years ago by
Owner: | changed from Дмитрий Свиридкин to yakovlev.aleksandr |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
Owner: | changed from yakovlev.aleksandr to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 1.0 → 2.0 |
comment:4 Changed 4 years ago by
Owner: | changed from Дмитрий Свиридкин to yakovlev.aleksandr |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Второй и 34 байт поломаны. У вас данные и заголовок хранятся разрозненно. А это один объект. Каждая операция должна сохранят консистентность данных и заголовка. Поэтому у вас что-то неправильно сохраняется. Сделайте нормальную структуру, чтобы не стучаться каждый раз в файл напрямую.
Код читающий заголовки в принице не должен напрямую вызываться в main. Это часть функции load.
Магическим константам (54, 34 и т.д) лучше дать имена.
9 + 1
comment:5 Changed 4 years ago by
Owner: | changed from yakovlev.aleksandr to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 2.0 → 3.0 |
К сожалению, из всего успел только поправить ломаные биты :(
comment:6 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
До исправления работало лучше.
Теперь даже с квадратной Леной не работает.
Проверьте 34й байт
Почему логика, читающая заголовки, находится в main?
Это часть функционала модуля bmp. И она должна быть скрыта за простым интерфейсом.