Opened 3 years ago
Closed 3 years ago
#231 closed ожидаются исправления (задача сдана)
HW #1
| Reported by: | antonenko.mixail | Owned by: | Святослав Власов |
|---|---|---|---|
| Component: | HW #1 (BMP) | Version: | 2.0 |
| Keywords: | Cc: | antonenko.mixail |
Description
Change History (9)
comment:1 Changed 3 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
comment:2 Changed 3 years ago by
| Cc: | antonenko.mixail added |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 1.0 → 2.0 |
comment:3 Changed 3 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
comment:4 Changed 3 years ago by
| Type: | ожидаются исправления → ожидается проверка |
|---|
comment:5 Changed 3 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
comment:7 Changed 3 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
Из тестов по прежнему работает только базовый тест поворота Лены целиком
На других тестах с леной неверный размер файла и изображения пишется в хедер результирующего файла, а на тестах с изображениями, где есть паддинг, программа падает на ассерте (что неудивительно, потому что он у тебя там зачем-то стоит)
По стилю претензий нет, всё хорошо, только работает неправильно :)
По бонусу -- тоже что-то не то делает, изображение извлекается не целиком, что странно. Еще в бонусе вижу, что ты почему-то открываешь файлы с ключем и сообщением в бинарном формате, вроде как к багу это не должно привести, но всё таки так делать не надо
comment:8 Changed 3 years ago by
Хм, почему-то мне казалось, что вы бы сообщили о таком за первую проверку.
comment:9 Changed 3 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Корректность -- 3/20, Стиль 5/10.
Работает пока только базовый вариант, поэтому больше поставить не могу.
Сам по себе код мне очень нравится, если доведешь его до рабочего состояния выдержав в том же стиле -- будет совсем прекрасно.