HW #3 (Huffman) Tulchinskiy hw_02

[tulchinskij.aleksandr]

Есть небольшое копирование кода. Я его не стал убирать, так как функции (с копированием) используются для тестирования, а если выражать рабочие функции через них, то программа станет работать медленнее и потреблять больше памяти.
Также возник небольшой вопрос: лимит по времени работает для простого запуска программы, или на запуск с валгриндом?

[tulchinskij.aleksandr]

Добавил нужные функции в private

[Антон Филатов]

почему-то ваши исполняемые файлы в obj/ положились, но им место в корневой директории, как всегда.

Нет, конечно, лимит времени на валгринд не распространяется

На файле, состоящем из 5МБ рандомных символов в рандомном порядке, выпадаете по таймауту на довольно некислом компьютере с процессором i5-8500 3.00GHz и 16ГБ ОЗУ. Подумайте, нельзя ли оптимизировать ваш код.

Не обрабатываете длинные флаги

Стиль:

• не хотите ли использовать умные указатели, вместо сырых?
• замените ассерты в main на exception-ы
• действительно ли столько функций должны быть публичными у HuffTree?? или какие-то без других не используются, и можно их спрятать и вызвать в одном публичном методе
• вместо \n используйте std::endl

Корректность: 11/15
Стиль: 7/10
Тесты: 5/5

[tulchinskij.aleksandr]

Расположение исполняемых файлов поправил (сбило с толку предложение: цель по умолчанию собирает исполняемый файл hw_02 и объектные файлы в директорию obj).
Реализацию поправил, теперь есть обработка длинных флагов. Также ускорил работу, на 5МБ файлах, программа стала работать в 2 раза быстрее.
Все замечания по стилю тоже поправил, пока кроме указателей.

[Антон Филатов]

Корректность: 15/15
Стиль: 10/10
Тесты: 5/5