#66 closed ожидается проверка (задача сдана)
HW2 huffman
Reported by: | Luchko Alexander | Owned by: | Vladimir Rutsky |
---|---|---|---|
Priority: | проверка | Milestone: | |
Component: | HA#2 huffman | Version: | 1.0 |
Keywords: | Cc: |
Description
задание номер 2
Change History (9)
comment:1 Changed 7 years ago by
Milestone: | ha2-milestone1 → ha2-milestone2 |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 7 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
Здравствуйте.
всё исправил
про 10 пункт действительно похож, так как чтение из бинарников падало, считывать в таком "побитовом" виде была действительно не моя идея, времени было мало, попросил показать способ чтения. Также прошу прощения за то, что не смог исправить за 60 часов, в Универе были контрольные и сдача статей.
comment:3 Changed 7 years ago by
Milestone: | ha2-milestone2 → ha2-deadline |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Замечания:
- Заголовочный файл должен быть самодостаточным: в
huffman.hpp
вы используетеstd::ofstream
,std::ifstream
и другие типы. Плюс вы неявно предполагаете, что пользовательhuffman.hpp
делаетusing namespace std;
--- вы используетеstd::uint8_t
и аналогичные типы без указанияstd
.
- Замечания 1 из предыдущей проверке про передачу объектов по константной ссылке исправлено не до конца:
bool dec(std::string const & ifile, std::string const & ofile); bool enc(std::string const & ifile, std::string const & ofile);
- В выводимой статистике вы выводите сжатый размер на 4 байта больше, чем на самом деле.
Исправьте, пожалуйста, в ближайшее время.
comment:5 Changed 7 years ago by
Replying to rutsky:
Александр, вы будете исправлять ваше решение?
Здравствуйте
В течение дня исправлю. Вчера была защита, потом домой ехал на прадники(приехал поздно, только проснулся)
comment:7 Changed 7 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
Note: See
TracTickets for help on using
tickets.
Замечания:
using namespace std;
в заголовочном файле --- в противном случае вы "загрязняете" глобальную область видимости для всех пользователей вашего заголовочного файла. Используйтеusing namespace std;
в*.cpp
файлах (или в своей области видимости).size_t
для индексов и размеров:std::priority_queue
не гарантирует, что при добавлении элементов с одним приоритетом они будут извлечены в каком-то определённом порядке, поэтому теоретически ваше решение может строить различные деревья при кодировании и декодировании, если у каких-то символов одинаковый частота встречаемости, при приведёт к ошибчному декодированию.-output
должен поддерживаться флаг--output
.Исправьте, пожалуйста, в течение 60 часов.