Opened 3 years ago
Closed 3 years ago
#707 closed ожидается проверка (задача сдана)
HW #3 Huffman Kiselev Igor hw_03
Reported by: | kiselev.igor | Owned by: | Антон Филатов |
---|---|---|---|
Component: | HW #3 (Huffman) | Version: | 3.0 |
Keywords: | Cc: |
Description
Вылезают странные ошибки make при попытке выполнить цель make test. Но не смотря на это автотесты есть, пускай и не хотят собираться(
Change History (5)
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
Type: | ожидается проверка → ожидаются исправления |
---|
Тесты почти все проходят. Кроме вот этих:
Не забудьте добавить поддержку длинных флагов
Класс HuffmanArchiver? почему-то тестируете в классе HuffmanTests?, а не HuffmanArchiverTests?. Какие-то тесты на него вообще закомментировали.
Структура файла huffman_test.cpp не очень дружелюбна к дополнению. Представьте, что вы будете дописывать логику - вы в какое место будете записывать новые тесты, например, для HuffmanTree?? Плюс статики вперемешку с тестирующими функциями. Добавьте файлов (на последней практике договорились, что можно на файлы разбивать), пусть каждый класс тестируется в своём файле, пусть все статические функции либо живут в незаметном месте этого файла (в начале или в конце), или вообще их тоже в отдельные файлы положите.
Структурируйте исходя из мысли, что вдруг вам потом дописывать ещё логику и тесты к ней
Корректность: 13/15
Стиль: 10/10
Тесты: 3,5/5
comment:4 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
Перепроверьте ваш вывод. Сумма второго и третьего числа должны давать размер вашего сжатого файла в байтах, а вы выводите что-то не то.
Для файла
вы выводите
при том, что размер сжатого файла получается 23 байта.
И так во всех тестах.
Не забудьте добавить поддержку длинных флагов
Стиль:
Корректность: пока 3/15
Стиль: 8/10
Тесты ну вы пока сами понимаете