Opened 3 years ago

Closed 3 years ago

#704 closed ожидается проверка (задача сдана)

HUFFMAN

Reported by: Tatiana Pridonyants Owned by: Святослав Власов
Component: HW #3 (Huffman) Version: 2.0
Keywords: Cc:

Description

Тесты не успела пока сделать

Change History (3)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверка → ожидаются исправления

По корректности прошли все тесты кроме тех, что используют аргументы в нестандартном порядке и с полными именами ключиков.

По стилю:

  1. std::unordered_map<uint8_t, std::string> table_; и std::unordered_map<uint8_t, int> map_; зачем? Обычный массивчик лучше подойдет, у тебя диапазон ведь константный -- от 0 до 255.
  2. Проверь инклюды. huffman.h} инклюдит не всё, что использует.
  3. HuffmanException лучше от logic_error отнаследовать, он по факту просто копирует его поведение
  4. std::priority_queue<TreeNode *, std::vector<TreeNode *>, Comparator_tree> чтобы сократить имя типа, советую либо специализировать std::less<TreeNode*> как я показывал на паре, либо использовать using PriorityQueue = ...
  5. Хранить битовую строку в виде строчки единиц и нулей -- такое себе решение, давай std::vector<bool>

13/0/7

comment:2 Changed 3 years ago by Tatiana Pridonyants

Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

Пока не все успела исправить

comment:3 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: assigned → closed

Твоя программа теперь пишет Error! Wrong input file на каждом тесте. Пришлось закомментить функцию err, чтобы это вылечить. При том что программа работает правильно.

Тесты появились, но они не юнит. Юнит-тесты должны тестировать каждый нетривиальный метод по отдельности. У тебя тестируется весьь архиватор целиком.

Замечания по стилю не поправлены.

15/2/7

Note: See TracTickets for help on using tickets.