Opened 5 years ago

Closed 4 years ago

#488 closed ожидаются исправления (задача сдана)

HW#1 samoylov.viktor

Reported by: samoylov.viktor Owned by: samoylov.viktor
Component: HW #1 (BMP) Version: 2.0
Keywords: Cc:

Description

Извините, пожалуйста, за 22 минутное опоздание!

Change History (4)

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

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

Больше не опаздывай!

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

  1. Заголовки imagesize/filesize неверны.

10.

Стиль:

  1. Функции просят static.
  2. Весьма магические константы.
  3. main не должен знать детали картинки. Пусть вся работа с структурами, чтением и записью будет в bmp.c.
  4. Названия переменных весьма слабо передают суть.

6.

comment:2 Changed 4 years ago by samoylov.viktor

Owner: changed from samoylov.viktor to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

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

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

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

Стиль:

  1. Ввод-вывод структур без выключенного выравнивания!!!
  2. Магические константы.
  3. Местами странные имена переменных (rot_data).
  4. Постоянно есть три вещи — два заголовка и данные. Может структурка?
  5. Ну и это, можно хранить двумерную таблицу пикселей из структур из 3 чаров, это всяко удобнее в коде будет.

6.

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

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.