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