HW_02 Huffman yusufov.arsenij

[Дмитрий Лапшин (lapshin)]

Ох не хочет собираться. И мейкфайл кривой, и программы с не теми именами.

Но если таки собрать

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

1. Неверную статистику печатает.
2. На файлах где всего один вид байт иногда падает.

8.

Тесты:
Весьма базово. Хорошо что несколько файлов туда-сюда прогнал, но сразу же просятся: крайние случаи просят детального разбора, какие там деревья, коды и статистики. 4.

Стиль:

1. Волшебные числа.
2. Сырые указатели. Не видно правила 5 при этом и публичные поля ой.
3. Весь код в заголовках это странновато.

6.

[Arseny Yusufov]

1. Статистику поправил
2. Волшебные числа – 256 поправил
3. Правило 5 постарался соблюсти
4. Код разнес на файлы

Сырые указатели – можно было бы заменить на shared_ptr, но не зря же деструкторы учился делать. Публичные поля – убрал в классе архиватора, но в TreeNode? оставил, так как по-моему это такая структура на стероидах – с конструкторами, но при этом без логики.

[Дмитрий Лапшин (lapshin)]

Сейчас есть утечка памяти. 14. Лучше умные указатели чем забывать дестуркторы.

Ещё лучше было бы брать константы из стандартных. Стиль 8.