Opened 3 years ago
Closed 3 years ago
#257 closed ожидается проверка (задача сдана)
HW #1 (BMP) Cherepanova Alyona
Reported by: | cherepanova.alyona | Owned by: | Антон Филатов |
---|---|---|---|
Component: | HW #1 (BMP) | Version: | 2.0 |
Keywords: | Cc: |
Description
Change History (3)
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
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Что-то то по-прежнему падают все тесты, по-прежнему структура не упакована во избежание выравнивания, int и unsigned по-прежнему вместо int16_t и int32_t
Размер файла в заголовке также не изменён
Стало немного получше, но за корректность всё равно больше 4 баллов дать, к сожалению, не могу
4/20
Стиль: 10/10
Note: See
TracTickets for help on using
tickets.
Пока у вас все автотесты падают с сегфолтом.
Кажется, что ваша структура устроена не очень здорово. sizeof(int) == 4, sizeof(unsigned) == 4, так что вам лучше использовать типы int16_t и int32_t - с прямым указанием, сколько бит предоставлять на тип
Кроме того необходимо запретить компилятору делать выравниваение. Делается это при помощи pragma pack или _ _attribute_ _ - погуглите
Пока проходят только тесты с некорректно введёнными границами
Стиль:
То, как вы пользуетесь тройным указателем - неправильно. вы не выделяете память, но делаете crop_pict[i][j][k]. А это - сегфолтТак можно пользоваться, но зачем..Пока корректность: 4/20
Стиль: 6/10