HW #3

[Дмитрий Свиридкин]

Failed to compress input file input/00_to_ff.core: exit code 139

[Дмитрий Свиридкин]

if (cmds.getAction() == "-c") {

Может, лучше enum?

CommandException::CommandException(const char *msg) : std::exception(), error_msg(msg) {}
Конструктор по умолчанию базового класса можно не вызывать

    std::unordered_map<char, std::string> codes;
    std::unordered_map<std::string, char> decode;
    HuffmanTree tree(freq, codes, decode);

выходные аргументы конструктора -- это, конечно, сильно. Не надо так.
Постройте дерево по первому аргументу. А таблицы кодов заберите из него явно, методами.

void HuffmanArchiver::bitesWrite() { Этот метод не просто пишет биты. Он выполняет кодировние.
Аналогично метод biteRead.

Логику записи/чтения по биту стоит отделить от логики преобразования кодов.

Чтение по биту, формирование строки с кодом и каждый раз запрос к таблице кодов -- очень неэффективно. Используйте дерево:
Читайте по биту, продвигайтесь по дереву. Дошли до листи -- отдали байт.

---

Зачем для построения дерева multiset? просто set должно быть достаточно

char *byte_number = (char*)number_pointer; reinterpret_cast

---

парсер аргументов тоже надо протестировать.

---

8 + 7 + 3.5 + 7

[Дмитрий Свиридкин]

Вложеность кода в main можно уменьшить с помощью function-try-block

template<typename T> using CurNode = T*;

Этот замечательный шаблон стоило сделать не в глобальном неймспейсе, а внутри класса.
И название типа неудачное. Все-таки это указатель. А переменная cur_node (cur_pos). owned_ptr<T>, если сильно хочется.

*

    char getCharAndDrop();

    void move(char direction);

Почему бы не объединить эти два метода в один:
std::option<byte> move(direction)

Логику перемещения и хранения текущей позиции лучше отделить в какой-нибудь класс TreeWalker?.

---

9 + 8 + 4.5 + 8