Opened 3 years ago

Closed 3 years ago

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

hw 1 ostapenko.stepan

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

Description

Нужно ли проверять корректность первого аргумента (crop-rotate, insert, extract)?

Change History (12)

comment:1 Changed 3 years ago by ostapenko.stepan

Пожалуйста, не проверяйте, т. к. я не могу заслать дз.
svn: E155011: Commit failed (details follow):
svn: E155011: Directory '/home/stephen/cprac/ostapenko.stepan/hw_01' is out of date
svn: E170004: Directory '/ostapenko.stepan/hw_01' is out of date

comment:2 Changed 3 years ago by ostapenko.stepan

Всё, починил

comment:3 Changed 3 years ago by ostapenko.stepan

Добавил реализацию бонуса.
Версию менять не буду, т. к. проверки не было.

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

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

А зачем ты оставляешь мусорные папки в SVN?)

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

Бонус: кексос! Ну и не лишнее было бы строчку перевести. 8/10.

Стиль:

  1. А давай без typedef на вариации числах. И использовать настоящие типы гарантированного размера!
  2. Не читаешь структуры за раз и ладно, но строчку пикселей можно прочитать и за одну операцию, а не 3 раза пикселей в строчке (со стороны заметно по скорости).
  3. Проверять malloc?

6/10.

comment:5 Changed 3 years ago by ostapenko.stepan

Owner: changed from ostapenko.stepan to Дмитрий Лапшин (lapshin)
Version: 1.0 → 2.0

comment:6 Changed 3 years ago by ostapenko.stepan

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

comment:7 Changed 3 years ago by ostapenko.stepan

Попытался исправить некоторые вещи
Мб позже ещё научусь проверять malloc

comment:8 Changed 3 years ago by ostapenko.stepan

Научился проверять malloc

comment:9 Changed 3 years ago by ostapenko.stepan

Заменил malloc + memset 0 на calloc

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

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

20/10

А теперь читаешь в буфер, который выделяешь лишь для копирования в массив. Давай сразу куда надо)
Кстати зачем calloc?

8/10.

comment:11 Changed 3 years ago by ostapenko.stepan

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

Кажется, теперь сразу куда надо

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

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

Ну да! 20 10 10

Note: See TracTickets for help on using tickets.