Opened 3 years ago

Closed 3 years ago

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

HW #1 (BMP) [volkov.ivan]

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

Description


Change History (4)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Ivan Volkov
Type: ожидается проверка → ожидаются исправления

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

  1. Один из заголовков про размер изображения неправильный.

12/20.

Бонус чот никак, ругается на длину файлов.

Стиль:

  1. Давайте без DWORD.
  2. #define MAX_MESSAGE_LEN 239 а что а зачем а почему а почему тут.
  3. При всём уважении, функции работы с изображении лучше бы не считали, что есть куда печатать printf-ом.
  4. И давай поменьше констант-дефайнов, если можно не дефайном.

Так-то симпотично. 7/10.

comment:2 Changed 3 years ago by Ivan Volkov

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

Вроде пофиксил багу с размером.

По поводу доп задания -- у меня похоже считывается \n каждый раз в конец message, поэтому он пытается и его закодировать. Этот \n (если он есть) надо игнорировать? А добавлять при выводе?

comment:3 Changed 3 years ago by Ivan Volkov

Summary: HW #1 (BMP) [volkov.ivan] quick fixes → HW #1 (BMP) [volkov.ivan]
Version: 1.0 → 2.0

вроде теперь все должно работать нормально

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

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

20 10 10

Note: See TracTickets for help on using tickets.