HW #2 (Huffman) Mittcev

[mitcev.nikita]

К первой проверке работает кодирование/декодирование по формату, нет тестов, обработки ошибок, private, const.

[Антон Филатов]

У вас сейчас падает, если входной файл пустой. Причём проверяющая система написана так, что она не позволяет проверить на других тестах. Но я потыкал руками немного - выглядит, как будто в большинстве своём работает правильно

Стиль:

• мне почему-то захотелось объеденить BitReader? и BitWriter? в один класс
• не хватает также нэймспейсов
• мне не очень нравятся волшебные константы 7 и 8

В остальном выглядит ок. Пока
Корректность: 0/10 (хотя там больше на самом деле, но вы сами говорите, что пока не доделано)
Стиль: 5/10 (по тем же причинам, плюс мои замечания)
Тесты: 0/10

[mitcev.nikita]

Все исправил, но BitReader? и BitWriter? объединять очень не хочу. Если это будет один объект, то достаточно нелогично одновременно из него читать и в него писать, а если делать для них общего предка, то там не совсем функции совпадают, поэтому кода получится в 2 раза больше, использование никак удобнее не станет и переиспользовать это потом тоже возможности не предвидится.

[Антон Филатов]

падает программа, если её просят сжать-разжать файл, состоящий из 1 символа (или из одного символа, который повторяется много раз)
К тому же пока не поддерживаете флаги --file и --output
Когда вы выводите на экран размеры, то

$ ./huffman -c -f myfile.txt -o result.bin
15678 - это размер входных данных
6172 - это размер сжатых данных без доп информации
482 - это размер доп данных

а вы во второе число выводите размер полного выходного файла. Не надо так

Стиль:

• (optional) не удобнее ли работать с shared_ptr, чем с сырыми?

Корректность: 7/15
Стиль: 10/10
Тесты 0/5

[Антон Филатов]

Корректность: 15/15
Стиль: 10/10
Тесты 5/5