Домашнее задание 2

[Aleksandr Eslikov]

Добрый день.
Посмотрите, пожалуйста, домашнее задание. Спасибо!

[Vladimir Rutsky]

Замечания:

1. Страж включения должен оборачивать весь файл:

#include <unordered_map>

#ifndef HUFFMAN_HUFFMAN2_HPP
#define HUFFMAN_HUFFMAN2_HPP

2. Вы используете std::istream_iterator, но не включаете в huffman.cpp <iterator>.

3. std::__1::ios_base::end: __1 --- это деталь реализации (и не компилируется, например, в Clang). Используйте std::ios_base::end.

4. Необходимо реализовать обработку аргументов командной строки без использования сторонних библиотек, вроде getopt.

5. Заголовочный файл не самодостаточен. Например, в нём используется std::string, но не включается <string>.

6. Используйте enum для SUCCESS и ERROR_CANNOT_OPEN_FILE.

7. Некоторые функции всегда возвращают SUCCESS. В этом нет смысла, возвращайте void. Также вы можете использовать исключения для обработки ошибочных ситуаций.

8. В коде есть неиспользуемые переменные, например, std::unordered_map<std::uint8_t, int> char_frequencies; в readOriginalFile() или i здесь:

 int i = 0;
 for (auto character:charFrequencies) {
     i++;
     queue.push({{character.first}, character.second});
 }

9. Используйте тип size_t для количеств, диапазона значений типа int может не хватить.

10. Используйте typedef для сложных типов.

Исправьте, пожалуйста, в ближайшие 60 часов.

[Aleksandr Eslikov]

1. Перенес #include <unordered_map> внутрь стража включения
2. Изменил include’ы в huffman.cpp
3. Это побочный эффект от использования в CLion Refactor->Extract->Method. Исправил
4. Реализовал обработку аргументов без использования getopt
5. Изменил include’ы в huffman.hpp
6. Переделал коды ошибок на enum
7. Поменял сигнатуры методов. Теперь они возвращают void
8. Удалил неиспользуемые переменные
9. Изменил int на size_t
10. Добавил typedef’ы

[Vladimir Rutsky]

Замечания:

1. Аргументы-объекты, которые не должны быть модифицированы внутри функции стоит передавать константными: Huffman::Huffman(const std::string &inputFileName, const std::string &outputFileName).

Решение зачтено.