HW_02 xargeliya.sergej

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

Я вижу что для тестов файлы пересобираются. Не критично, но.

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

Тесты: тесты не проверяют какая статистика получается, а ещё хотелось бы тестов деревьев и всякого... Интеграционный набор крутой, не придраться. 4.

Стиль:

1. Почему число байт знаковое?)
2. CHAR_BIT есть плюсовая конструкция для такого же.
3. std::unordered_map: круто, но можно же просто массив)
4. У тебя на самом деле нет разделяемого владения.
5. char min_symbol_in_subtree;
6. Зачем rebuild когда есть конструктор?
7. Деструктор битового вывода не флашат. Кажется было бы не лишним.

А так даже нравится, 6.

[Сергей Харгелия]

Кажется, исправил всё, кроме std::unordered_map (потому что мне показалось, что на производительность это не влияет примерно никак, а код так более читаемый) и разделяемого владения -- вот с ним я просто не понял, как тогда сделать нормально, потому что вместо этого я вижу только варианты написать всё на сырых указателях и смешать std::unique_ptr с сырыми, обе опции идеологически звучат не очень круто

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

map: массив сильно ускорит, это правда не так заметно, но да.

А вектор вполне мог бы хранить unique_ptr.

Короче 15/5/8.