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: ожидается проверкаожидаются исправления

Перепроверьте ваш вывод. Сумма второго и третьего числа должны давать размер вашего сжатого файла в байтах, а вы выводите что-то не то.
Для файла

aaaaaaaaaaaaaaaa

вы выводите

16
2
13

при том, что размер сжатого файла получается 23 байта.

И так во всех тестах.
Не забудьте добавить поддержку длинных флагов

Стиль:

  • хочется как-то побольше разбить на функции, а то у вас сейчас прям полотна кода получаются
  • волшебная константа 128
  • у меня при попытке собрать тесты возникает ошибка
    make: *** No rule to make target 'obj/autotest.o', needed by 'test'.  Stop.
    

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

comment:2 Changed 3 years ago by kiselev.igor

Type: ожидаются исправленияожидается проверка
Version: 1.02.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 kiselev.igor

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

comment:5 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

15 + 10 +5

Note: See TracTickets for help on using tickets.