Opened 4 years ago

Closed 4 years ago

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

HW #1 krasov.vladislav

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

Description


Change History (8)

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

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

Лишние файлы в репозитории.

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

  1. Заголовки filesize/imagesize неверны.
  2. А местами valgrind и совсем плохо.

9.

Стиль:

  1. Постоянно передаются две структуры и данные… Может структуркой?
  2. Магические константы.

8.

comment:2 Changed 4 years ago by krasov.vladislav

Owner: changed from krasov.vladislav to Дмитрий Лапшин (lapshin)

comment:3 Changed 4 years ago by krasov.vladislav

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

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

А всё так же пока. На банальном тесте ок, дальше заголовок и содержимое не совпадают в каких-то местах.

comment:5 Changed 4 years ago by krasov.vladislav

А когда valgrind ругается?... Я и разные картинки пробовал с разными размерами, не обязательно дящимися на 4. Везде что-то вроде:
in use at exit: 0 bytes in 0 blocks
total heap usage: 1,752 allocs, 1,752 frees, 2,901,431 bytes allocated
All heap blocks were freed -- no leaks are possible
а как вообще найти случай, когда valgrind ругается?

comment:6 Changed 4 years ago by krasov.vladislav

Version: 2.0 → 3.0

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

Не проверяешь аргументы до конца. Что-то не так с выданными изображениями проверяй заголовки и выравнивание.

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

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

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

  1. На отрицательность не проверяешь аргументы, если вообще проверяешь.
  2. Выравниваешь при записи не нулями. Кажется, тебе одного байта нулей не хватает., чтобы писать больше одного.
  3. А ещё заголовки.

9.

Стиль:

  1. Имена. SomeWeirdInfo намекает на бесполезность. Кстати, зачем внутри h, w, x, y? Без них, но с массивом пикселей была бы структура с картинкой целиком.
  2. Магические константы и дублирующиеся выражения.

8.

Note: See TracTickets for help on using tickets.