Opened 3 years ago

Closed 3 years ago

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

hw01 haha

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

Description


Change History (3)

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

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

Корректность -- 20/20
Тесты все прошли с первого раза, молодец! Можешь делать бонус.

Стиль -- 8/10

  1. Так нет смысла писать printf("%s", "Cannot create new pixels\n");, можно просто printf("Cannot create new pixels\n");, а еще проще assert(bmp->pixel && "Cannot create new pixels");
  2. fread(&bmp->header, sizeof(bmp_header), 1, file) == 0 -- лучше ... != sizeof(bmp_header), если хедер прочитается не до конца, явно что-то пошло не так.
  3. Тут memcpy кажется что получше смотрится:
    for (int k = 0; k < 3; k++) {
      get_pixel(i, j, dst)[k] = get_pixel(x, y, src)[k];
    }
    
  4. Да и вообще в crop ты можешь копировать куски строк не попиксельно, а целиком, все размеры у тебя для этого есть.

comment:2 Changed 3 years ago by petrov.vladimir

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

Поправил предыдущие ошибки и добавил решение дополнительного задания

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

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

Задание 30/30
На бонусе валгринд детектит ошибки. Ты читаешь не из своей памяти. А читаешь потому, что забыл, что после того, как прочитал строчку fread надо поставить завершающий ноль. А еще лучше было читать используя fgets.

Note: See TracTickets for help on using tickets.