HW #3 (Huffman) vyguzov.artem hw_02

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

Корректность: не подключил <iostream>, на паре крайних тестов упал (хотя казалось бы). Хотя есть гипотеза... Ты случаем не один байт тратишь на число разных байт в файле?))) 10.

Тесты: я не смог заставить их не упасть.

Стиль:

1. additional.h вызывает вопросы всем.
2. Сырые владеющие указатели. Не пришлось бы городить try.
3. Подсчёт чариков: почему не std::unordered_map, а ещё лучше std::vector/массив?
4. get_min: кажется есть стандартная функция на тему.
5. int additionalData = (int)in.tellg(); зачем int?
6. Для подсчёта числа символов можно вместо map использовать unordered_map, а ещё лучше vector/массив.

6.

[Артем Выгузов]

Корректность: <iostream> подключил, бага действительно интересная получилась) (Тест на всякий случай добавил)
Тесты: Мне подсказали, что проблема может быть из-за того, что локально у меня Makefile генерируется в отдельной папке cmake-build-debug, и из-за этого путь до файлов для тестов получается неправильный. Если проблема действительно была в этом, то я её пофиксил.
Стиль: Исправил всё кроме additional.h
Он существовал, чтобы можно было проверить парсинг аргуметов в тестах, но раз к нему возникли вопросы, то я его содержимое поместил в main, а соответствующие тесты убрал.

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

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

Тесты: Я бы ещё статистику проверил. 4.

Стиль:

1. У TreeNode всё прям публичное. Может она не нужна в заголовке?
2. strcmp и прочее: std::string/std::string_view.

8.

[Артем Выгузов]

Добавил структуру для статистики и тесты для неё.
TreeNode оставил в заголовке, потому что она используется и в HuffTree, и в HuffmanArchiver, и в тестах.
strcmp больше нет, произошла замена на std::string_view.

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

Так тоже можно. Полный балл.