Opened 3 years ago
Closed 3 years ago
#710 closed ожидается проверка (задача сдана)
HW #3
Reported by: | Pavlova Aleksandra | Owned by: | Святослав Власов |
---|---|---|---|
Component: | HW #3 (Huffman) | Version: | 2.0 |
Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Пожалуйста, четко следуйте формату вывода, который требуется в условии задания! Любые лишние символы ломают наши тесты и мне приходится руками их из вывода вычищать
Тесты на сжатие-распаковку прошли, но почти везде неверно репортится размер исходного файла.
Твоих тестов маловато.
По стилю -- код тяжеловато читать, хочется чтобы вы уже отходили от ручного управления памятью и использовали умные указатели
13/4/9
Note: See
TracTickets for help on using
tickets.
Формат аргументов не соответствует тому, что требуется в задании, поэтому корректность проверить не могу
По стилю:
std::vector <std::pair <long long, char> > freq
-- лучшеmap<long long, char>
std::set <std::pair <long long, std::pair <char, TreeNode*> > >
-- можно подумать как сделать структуру попроще. Зачем, к примеру, хранить char в паре, если он уже естьTreeNode
? Дальнейший код функции очень сложно читать и понимать как он работает.Юнит-тестов тоже не увидел.