Opened 4 years ago

Closed 4 years ago

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

HW #1 chistyakova.polina

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

Description

Привет))
Извини, там код пока не очень красивый... Зато работает) (Ну я его уронить не смогла.. Если не вводить что-то совсем страшное.)
Я же правильно понимаю, что надо было первую версию ставить, а не вторую?..

Change History (5)

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

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

Корректность: Нётъ.

  1. Как бы сказать… Что надо сделать — что сделала ты?
    • Ну предположим я это починил.
  2. Упало на всех остальных тестах. Читай стиль.

5.

Стиль:

  1. Ну предупреждения ж.
  2. Хранить картинку как одномерную версию неудобнааа.
  3. Особенно вместе с выравниваниями.
  4. ОСОБЕННО ШАГАЯ ПО НЕЙ С ++.
  5. Магические константы.
  6. Комментарии с мёртвым кодом не приветствуются. Комментариями с мыслями пишущего тоже, лучше отображать детали для читающего (хотя, конечно, очень мило).

5.

comment:2 Changed 4 years ago by chistyakova.polina

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

Ну ожидается проверка это сильно сказано...
Извини, пожалуйста ТТ
Не успела ничего, только по стилю замечания исправила (и то не все, потому что хранение в двумерном массиве мне так и не далось)
Проверять лучше в плохом настроении, чтобы себе хорошее не портить...(можно вообще не проверять, ничего принципиально нового там нет...)

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

Действительно страдает.

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

Содержимое картинки лажает, почему-то.

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

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

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

  1. Ну на тест на всю Лену оно создало изображение, и оно попиксельно не то. И заголовки кажется недосчитались.
    • Почему? Смотри стиль.
  2. А дальше вообще не справилось.

5.

Стиль:

  1. #define WORD __uint16_t:
    • Сча по рукам за дефайн.
    • Что за потусторонние мистические типы с двумя подчёркиваниями? Чем не угодил ​uint16_t?
  2. Я говорил, что хранить двумерную картинку с выравниваниями в одномерном буфере сложно? Хранила бы без выравнивания в двумерном. У меня нет сейчас времени реверсинжинировать что происходит и где ошибка, код слишком сложен.
  3. static у внутренних функций?
  4. Магические константы. malloc 46 my ass.

7.

Note: See TracTickets for help on using tickets.