Opened 4 years ago

Closed 4 years ago

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

HW #3 fomenko.ivan

Reported by: fomenko.ivan Owned by: Дмитрий Лапшин (lapshin)
Component: HW #1 (BMP) Version:
Keywords: Cc:

Description


Change History (4)

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

Owner: changed from Дмитрий Лапшин (lapshin) to fomenko.ivan
Summary: HW #1 → HW #1 fomenko.ivan
Type: ожидается проверка → ожидаются исправления

Корректность: 20. Успешно врываемся!

Стиль:

  1. Советую использовать типы точного размера напрямую, без typedef.
  2. Лишние пустые строки.
  3. Магические константы 3 местами.
  4. Хранить картинку как поток байт неудобно, храни как двумерный массив структур из 3 байт.

6.

comment:2 Changed 4 years ago by fomenko.ivan

Owner: changed from fomenko.ivan to Дмитрий Лапшин (lapshin)
Summary: HW #1 fomenko.ivan → HW #3 fomenko.ivan
Type: ожидаются исправления → ожидается проверка

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

Ему кажись хорошо. Бонус что-то не то с картинкой сделал, но работает.

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

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

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

Бонус:

  1. Чот он лишнее делает. Один пиксель не такой!
    • Нефиг кодировать и перевод строчки!

9.

Стиль:

  1. Попрошу не использовать typedef DWORD.
  2. Пустые строчки.
  3. Магические константки.
  4. Структуры можно присваивать и через = .
  5. Хранить картинку как буфер не очень удобно, у тебя куча кода считает координаты.
  6. Из бонуса:
    1. Я бы из функции конвертации просто char возвращал.

6.

Note: See TracTickets for help on using tickets.