Opened 5 years ago

Closed 4 years ago

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

HW #1 sunko.elena

Reported by: sunko.elena Owned by: Дмитрий Лапшин (lapshin)
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (5)

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

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

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

  1. Аргументы недопроверяешь.
  2. Ни разу не выдало правильную картинку. Но заголовки, вроде, правильно пишешь.

4.

Стиль:

  1. Отступы!
  2. Пробелы!
  3. Дикие пустые строки стаями!
  4. Зачем внутренние функции выставляются в заголовочный файл? Почему они не static?
  5. Глобальные переменные.

6.

comment:2 Changed 4 years ago by sunko.elena

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

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

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

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

Стиль:

  1. В структурах отступы. и не только там: в перемешку табы и отступы.
  2. Много лишних пустых строк.
  3. Очень много функций выставляется. Всё из-за хранения изображения как char *. Рекомендую более удобноприменимый формат.
  4. Константы просятся в static.
  5. Не везде проверяешь malloc.

5.

comment:4 Changed 4 years ago by sunko.elena

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

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

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

Стиль:

  1. Какие-то функции так и не обзавелись static.
  2. А хранить картинку одномерно и с выравниванием всё так же больно!
  3. Пустые строчки блин.

За пределами этого ок вполне, 7.

Note: See TracTickets for help on using tickets.