Opened 4 years ago

Closed 4 years ago

#504 closed ожидается проверка (задача сдана)

HW #1

Reported by: Filippov Denis Owned by: Дмитрий Свиридкин
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 4 years ago by Дмитрий Свиридкин

Version: 1.02.0

comment:2 Changed 4 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to Filippov Denis
Type: ожидается проверкаожидаются исправления

Второй байт картинки неправильный. 34й скорее всего в случае неквадратной картинки тоже.

Почему функции crop (и rotate), вызываемые из main, принимают какое-то дикое количество аргументов, вместо, например объекта-картинки, нового размера и куда сохранить (указатель на картунку)?
У вас есть функция load, она же не принмает тучу аргументов (буферы под размер, заголовок, и т.д.).

С функцией save та же беда.

Внутри блока pragma pack ни в коем случае нельзя подключать заголовки, особенно системные! Если она применится на стандартные структуры, вы скомпилируете программу не совместимую с стандартной библиотекой. У вас оно как-то работает только потому что сошлись звезды.

Так же блок pragma pack у вас не закрыт (pragma pack (pop)). И все, кто подключает ваш заголовок, получают отключение выравнивания и на собственные структуры.

Визуально как бы работает, но файлы битые. Бонус бонус тоже как бы визуально работает, но файл с картинкой не верный. У него точно не верный заголовок. А содержимое не совпадает с эталонным -- судя по различиям, перепутан порядок либо строк, либо стоблцов.

Функции, реализующие вставку и вычленение сообщения не разбиты на логически независимые части: чтение ключа дублируется. Функции преобразования символов не отделены.

В файле stego.c используются функции стандартной библиотеки, но не подлючен ни они заголовок, таким образом получается неявная зависимость от других заголовков, их подключающих.

10 + 4 + 4

comment:3 Changed 4 years ago by Filippov Denis

Owner: changed from Filippov Denis to Дмитрий Свиридкин
Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:4 Changed 4 years ago by Дмитрий Свиридкин

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

При обрезке картинки перепутана нумерация строк.

Код, заполняющий заголовок, дублируется

18.5 + 8 + 8

Note: See TracTickets for help on using tickets.