HW #3 krasov.vladislav

[krasov.vladislav]

Сам алгоритм на моих тестовых файлах работает, сжимает и разжимает и с diff хорошо, но всё остальное очень плохо

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Под валгриндом не дождались конца сжатия.
2. Без него тоже. Я подождал 10 тестов на один-два байта, дальше ждать не стал.
3. Разбор опций не по заданию.

0/9.

Тесты: 0.

Мне кажется, или это дисквал?

Стиль:

1. // this is a functor это крайне полезный коментарий, класс с единственным методом, который к тому же operator(), очень не понятно кто. Ещё не понятно зачем он в заголовке.
2. Пространства имён.
3. Магические константы!
4. Зачем архиватор на куче?

5/8.

Архитектура:

1. Вообще всё публично?
2. Сырые указатели? Ссылки, умные указатели, значения.
3. Кажется, работа с потоком бит просится вынестись.
4. Почему архиватор дофига выводит? Почему он вообще считает, что вывод есть?
5. Метод compress потрясающий.

2/5.

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Разжималка то падает, то восстанавливает не то. Или оч долго.
2. Сжималка тоже периодически падает.
3. Опции в кривом порядке тоже не ест.

4/9.

Тесты: ох не собралось, ох предупреждения. Построение дерева не протесчено. 5/8.

Стиль:

1. Пространства имён.
2. delete x без скобочек.

7/8.

Архитектура:

1. bool to_compress, to_extract: енам?
2. Сырые указатели.
3. Зачем list_of_nodes когда уже есть root? Особенно как поле класса? Очень странная структура в памяти и система владения.
4. Продолжение предыдущего: а надо столько полей везде?
5. Архиватор не сообщает вызывающему об ошибке, но сразу пишет на вывод. Логично ли это?

2/5.