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: ожидается проверкаожидаются исправления

Пока у вас все автотесты падают с сегфолтом.
Кажется, что ваша структура устроена не очень здорово. sizeof(int) == 4, sizeof(unsigned) == 4, так что вам лучше использовать типы int16_t и int32_t - с прямым указанием, сколько бит предоставлять на тип
Кроме того необходимо запретить компилятору делать выравниваение. Делается это при помощи pragma pack или _ _attribute_ _ - погуглите

Пока проходят только тесты с некорректно введёнными границами
Стиль:

  • Мне кажется, что будет лучше завести структуру из трёх чаров, чем вести за собой тройной указатель и постоянно делать цикл из трёх итераций
  • То, как вы пользуетесь тройным указателем - неправильно. вы не выделяете память, но делаете crop_pict[i][j][k]. А это - сегфолт Так можно пользоваться, но зачем..
  • не звбудьте потом, что надо будет в выходном файле изменить size в двух местах в заголовке

Пока корректность: 4/20
Стиль: 6/10

Last edited 3 years ago by Антон Филатов (previous) (diff)

comment:2 Changed 3 years ago by cherepanova.alyona

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

comment:3 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

Что-то то по-прежнему падают все тесты, по-прежнему структура не упакована во избежание выравнивания, int и unsigned по-прежнему вместо int16_t и int32_t
Размер файла в заголовке также не изменён
Стало немного получше, но за корректность всё равно больше 4 баллов дать, к сожалению, не могу
4/20
Стиль: 10/10

Note: See TracTickets for help on using tickets.