HW #3 (Huffman) Martynov

[martynov.maksim]

Тестов пока нет :(
Должно ли декодирование быстрее работать?

[martynov.maksim]

Добавил проверку аргументов

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

Ворнинги при сборке

make
mkdir -p obj
g++ -O2 -g -ggdb -Wall -std=c++11 -Iinclude -c -MMD -o obj/huffman.o src/huffman.cpp
src/huffman.cpp: In member function ‘void HuffmanUnarchiver::read_data(std::__cxx11::string)’:
src/huffman.cpp:155:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (int i = 0; i < NUM_BYTES; i++) {
                     ~~^~~~~~~~~~~
src/huffman.cpp: In member function ‘void HuffmanUnarchiver::decode_and_write(std::__cxx11::string) const’:
src/huffman.cpp:180:48: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
             if (i == encoded.size() - 1 && pos < extra_bits)
                                            ~~~~^~~~~~~~~~~~
g++ -O2 -g -ggdb -Wall -std=c++11 -Iinclude -c -MMD -o obj/main.o src/main.cpp
g++ obj/huffman.o obj/main.o -o hw_02 -g -ggdb

Тесты пройдены. Декодирование работает с нормальной скоростью
Стиль:

• не хотители переписать на умных указателях, вместо сырых
• не хватает нэймспейсов

корректность: 15/15
Стиль: 8/10
Тестов пока нет

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

Честно говоря, я надеялся, что вы не просто всё запихнёте в один нэймспейс. Я надеялся, что те классы, которые действительно можно отдать пользователю, и те классы, которые ему отдельно давать нельзя, вы положите в разные нэймспейсы

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