Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

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

HW #1 vityazev.arseniy

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

Description


Change History (5)

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

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

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

  1. Падает на всех тестах. Где-то успевает выдать местами неправильный файл.
  2. Проверки аргументов не видно.

Вообще тянет на 0, но зачатки решения есть. Делай дальше.

Стиль:

  1. Магические константы.
  2. Чот многовато магии вида ±1.
  3. Заголовок в виде 54 байт это очень удобно и надёжно.
  4. Пиксель можно читать за один read. А если много пикселей, так вообще хорошо. А ещё memcpy есть.

7.

comment:2 Changed 4 years ago by Витязев Арсений

Version: 2.0 → 3.0

Почему-то не считывает файлы из sample, потому lena_512.bmp не в ней.

comment:3 Changed 4 years ago by Витязев Арсений

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

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

Owner: changed from Витязев Арсений to Дмитрий Лапшин (lapshin)

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

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

(сорян тут был текст не для тебя)

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

  1. Выдаёт неверные заголовки.
  2. За пределами просто теста выдаёт неверные файлы.
  3. Падает, валгринд орёт.

5.

Стиль:

  1. Да не надо работать с заголовками как 54 байта. Ужас!!!
  2. Отступы плохие, там ещё табы с пробелами в перемешку.
  3. Из-за того, что картинка хранится снизу вверх, не надо страдать отражением координат везде в коде!
  4. Где-то там ещё код дублируется.

7.

Last edited 4 years ago by Дмитрий Лапшин (lapshin) (previous) (diff)
Note: See TracTickets for help on using tickets.