#473 closed ожидается проверка (задача сдана)
HW #1
Reported by: | luchinin.aleksey | Owned by: | Дмитрий Свиридкин |
---|---|---|---|
Component: | HW #1 (BMP) | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 4 years ago by
Owner: | changed from Дмитрий Свиридкин to luchinin.aleksey |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
Owner: | changed from luchinin.aleksey to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 1.0 → 2.0 |
Исправил. Пока без доп задания
comment:3 Changed 4 years ago by
Owner: | changed from Дмитрий Свиридкин to luchinin.aleksey |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Второй и 34 байт поломаны -- размер неправильно выставлен после поворота неквадратной картинки.
Безобразие с работой с байтиками заголовка все также в main.
У магических констант (54, 34 и т.д.) должны быть имена.
Логика выравнивания размеров продублирована в main.
Логика перевыставления размеров оторвана от логики обрезки/поворота -- отсюда проблемы
10 + 1
comment:4 Changed 4 years ago by
Owner: | changed from luchinin.aleksey to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 2.0 → 3.0 |
Исправил все недочёты
comment:5 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
free мужду операциями лучше перенести внутрь. Операции над данными следует реализовывать так, чтоб они сами поддерживали консистентсность данных, а не полагались на то, что это будет делать пользовательский код.
Имена аргументов check_borders ничего не говорят.
константу для размера зашголовка можно объявить в хэдере, чтобы убрать магическое число из структуры.
Почему функция, копирующая параметры картинки в новую пустую, называется work_with_BMP?
20 + 5
Проверьте 34й байт
Почему логика, читающая заголовки файлов, находится в main?! Эту работу должны делать функции чтения BMP. И это все должно быть глубоко скрыто за простым интерфейсом.