Opened 3 years ago

Closed 3 years ago

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

HW #1 (BMP) vyguzov.artem

Reported by: Артем Выгузов Owned by: Дмитрий Лапшин (lapshin)
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 3 years ago by Артем Выгузов

Version: → 1.0

comment:2 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Артем Выгузов
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. Один из заголовков про размер изображения в конце получается неправильный. Но на каждом тесте.

12/20.

Бонус при этом ок.

Стиль:

  1. Ты не используешь типы гарантированного размера.
  2. main у тебя как-то дофига знает. По идее bmp.c/bmp.h должен полностью покрывать операции с картинками, а у тебя не так.
  3. Магических констант вида 4-5 многовато.

6/10.

comment:3 Changed 3 years ago by Артем Выгузов

Owner: changed from Артем Выгузов to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

Вроде, всё исправил.

comment:4 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Артем Выгузов
Type: ожидается проверка → ожидаются исправления

20 10

Постоянно передаётся пачка аргументов (заголовки и содержимое), может структура?
Память выделяет bmp.c, освобожадает main.c.
8.

comment:5 Changed 3 years ago by Артем Выгузов

Owner: changed from Артем Выгузов to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 2.0 → 3.0

comment:6 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed

Я бы и массив пикселей туда положил, чего мелочиться, вся картинка в одной упаковке. 9.

Note: See TracTickets for help on using tickets.