HW_02, Huffman (pavlenko.daniil)

[Daniil Pavlenko]

Заменил все shared_ptr на unique_ptr
Подправил флаги (теперь работает --file)

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

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

Тесты: ну имя программы не то. Ох наивно наивно наивно собирать команду через пробелы. А вообще можно было бы подумать и написать программу так, чтобы тестировать было легко и невинно. Прям просятся тесты на построение дерева, скажем. 3.

Стиль:

1. const int charBits = 8; давайте приличные стандартные константы.
2. Парочки пробелов.
3. Мы считаем сколько раз вошли 256 байт, так куда же проще использоваться.
4. operator < можно сильно красивее написать.
5. Я бы не сортировал по числам на выходе из подсчёта статистики.
6. Tree::go выглядит как будто слегка дублируется. Вообще то что состояние обхода хранится в дереве выглядит чуууть странно.
7. Для простой структуры из трёх полей конструктор не нужен.

7.

[Daniil Pavlenko]

Тесты:
Поменял название программы и добавил тестов на построение/структуру дерева. Про сборку команды не совсем понял, как стоит собирать?

Стиль:
1) fixed
2) удалил лишние пустые строки (как я понял, проблема была в них)
3) fixed
4) удалил, так как больше не нужен
5) fixed
6) fixed
7) fixed

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

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

Тесты: 5. Про сборку команд: в твоём случае вместо rm лучше пользоваться каким-нибудь std::filesystem. Потому что один нечайный пробел в имени файла и усё. Приветствую diff при этом.

Стиль: Да нормально. Мне не очень нравится что заговочный файл выставляет все внутренние функции. 9.

[Daniil Pavlenko]

Тесты:
Переписал удаление файлов на std::filesystem::remove()

Стиль:
Вынес все функции реализации в cpp, в заголовочном файле оставил только code и decode

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

Можно было бы тогда и все структуры в реализацию подвинуть... Останется 9.