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: ожидается проверкаожидаются исправления

Валгринд возмущается
Попробуй создать файлик, в котором будут все байты от 0x00 до 0xff и запустить на нем компрессию под валгриндом.
На этом же файле у тебя размер сжатых данных почти в 20 раз больше исходных.
Еще на паре тестов с достаточно большими случайными данными всё рухнуло с таймлимитом.

По стилю особых замечаний нет, кроме того, что незачем для каждого класса делать свой неймспейс. Класс -- это сам по себе уже неймспейс. Неймспейсы имеет смысл делать для набора логически связанных классов и для выделения отдельных подсистем. В твоем случае одного неймспейса huffman было бы достаточно.

Тестов нет.

7/9/0

comment:2 Changed 3 years ago by kutyavin.denis

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

comment:3 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed

По корректности все те же проблемы остались: валгринд продолжает возмущаться, а на 5-мегабайтовом файле с рандомными байтами твоя декомпрессия свалилась с тайм-лимитом.

Тестов маловато.

8/3/10

Note: See TracTickets for help on using tickets.