HW #03 smirnov.ilya

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

Корректность: чиселки неправильные пишешь. Иногда не восстанавливаешь оригинал. Падаешь при расжатии. Ещё один раз слишком долго расжимало.
4/9.

Тесты: неверное имя файла. Но вроде ок. 7/8.

Стиль:

1. Пространства имён. Не, заводить их для констант не считается. Константы больше просятся внутрь классов тогда уж.
2. Тем более не всё это в заголовке.
3. Тем более два чара вместо перечисления.
4. Пробелы.
5. const std::string_view & вся суть вьюх что они и есть как бы ссылки, передавай целым.
6. ALPHAVITE. Альфа-виттэ.
7. INF = 1e9 я знаю эти симптомы.
8. Код сборки дерева сложноват. Выражения дублирются.

4/8.

Архитектура:

1. CLI: так когда объект начинает жить? Когда его создали, или когда распарсили?
2. Куча каких-то странных констант не пойми откуда и зачем. 20, 6, 7, они точно нужны?

3/5.

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

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

1. На пустом файле печатаешь неправильные числа.
2. Разжатие медленно.

8/9.

Тесты: там же. 8.

Стиль:

1. noexcept(false): то есть пустое место?
2. Пробелы.
3. Пространства имён.
4. BUFFER_SIZE: чему-чему-чему она равна? Именно вот так? Именно CHAR_BIT?

6/8.

Архитектура:

1. Перечисление, завязанное на чарики, такое себе.
2. Архиватор, считается, всегда выводит статистику именно в cout.
3. static поля в потоках бит: значит, одновременно работать может лишь один?
4. throw std::exception() поймавший исключение запросто поймёт что имелось в виду.

3/5.