Opened 3 years ago

Closed 3 years ago

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

HW#1

Reported by: kutyavin.denis Owned by: Святослав Власов
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Святослав Власов

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

Лена целиком поворачивается корректно, но размер файла в хедере пишется неверный.
Вырезается не тот кусок. Например с аргументами 100 50 128 128 у тебя вырезается кусок, тот что слева, а должен тот, что справа:

За корректность 5/20 пока

Стиль -- 4/10

  1. В crop можно копировать строки не побайтово, а сразу целиком
  2. Странно, что у тебя функции crop и rotate уничтожают исходное изображение. Это неочевидное поведение, обычно таким должен заниматься вызывающий код.
  3. Не надо так писать. Это плохой стиль и неприятно читать. Заведи отдельные переменные под размеры и их передавай.
    crop_and_rotate(argv[2], argv[3], atoi(argv[4]), 
    	atoi(argv[5]), atoi(argv[6]), atoi(argv[7]));
    

comment:2 Changed 3 years ago by kutyavin.denis

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

comment:3 Changed 3 years ago by Святослав Власов

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

Размер изображения и файла в хедера по прежнему пишется неверный
15/20 за корректность

Стиль -- 8/10

  1. Функции, локальные для bmp.c стоит сделать статическими
  2. В rotate пиксель можно копировать целиком с помощью memcpy, цикл из трех итераций смотрится неуместно
  3. Количество байт на пиксель стоит абстрагировать в константу

Бонус работает, но зачесть смогу только после полного балла за корректность
Из замечаний -- путаешь порядок следования байтов в пикселе, прочитай на вики в каком порядке они следуют.

comment:4 Changed 3 years ago by kutyavin.denis

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

comment:5 Changed 3 years ago by Святослав Власов

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

Дедлайн

Note: See TracTickets for help on using tickets.