Change History (7)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Бубнов Данил Константинович
Type: ожидается проверкаожидаются исправления

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

  1. Пожалуйста, отправляйте решение без санитайзера.
  2. На почти всех тестах неправильные заголовки filesize/imagesize.

10.

Стиль:

  1. Магические константы!
  2. Очень наверное удобно разбирать заголовки как 54 байта ерунды. Сделай структурки.
  3. И картинку бы тоже лучше хранить двумерно. Или хотя бы функциями себе помочь.
  4. finish.
  5. Я бы сказал, что зона отвественности между main и crop/rotate не самая правильная. Пусть они сами выделяют сколько нужно памяти.

6.

comment:2 Changed 4 years ago by Бубнов Данил Константинович

Owner: changed from Бубнов Данил Константинович to Дмитрий Лапшин (lapshin)
Version: 1.02.0

Вроде бы избавился от констант.
Сделал структуры, для заголовков, теперь меняю filesize, imagesize.
Храню теперь двумерно пиксели.
Теперь память выделяется в структурах

comment:3 Changed 4 years ago by Бубнов Данил Константинович

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

Вроде бы избавился от констант.
Сделал структуры, для заголовков, теперь меняю filesize, imagesize.
Храню теперь двумерно пиксели.
Теперь память выделяется в структурах

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

Owner: changed from Дмитрий Лапшин (lapshin) to Бубнов Данил Константинович
Type: ожидается проверкаожидаются исправления

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

  1. filesize. Глупая лажа.

15.

Стиль:

  1. ROUNDstatic? Тем более не везде используешь, где логично ;)

9.

comment:5 Changed 4 years ago by Бубнов Данил Константинович

Owner: changed from Бубнов Данил Константинович to Дмитрий Лапшин (lapshin)
Type: ожидаются исправленияожидается проверка
Version: 2.03.0

Исправил лажу: был sizeof от указателей, а не от самих структур
Сделал ROUND static, и заменил последнюю 4 на ROUND

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

Работает

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

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

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

Стиль:

  1. Пиксели можно было не паковать, и так чарики.
  2. Хранить картинку в одномерном буфере неудобно ж.
  3. Местами дублируются выражения.

8.

Note: See TracTickets for help on using tickets.