HW #3 kirillova.evgeniya

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

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

1. Сжать пустой файл оч сложно.
2. Числа иногда выводишь странные: сжатый файл якобы больше исходного. И файлам не соотвествует.
3. А ещё как-то слабенько сжимает. Но может это реакция на неправильные числа.
4. Есть и хорошие новости: не сломалось!

7/9.

Тесты: 8.

Стиль:

1. Пространства имён?
2. Магические константы?
3. > multiset кто повторяться собирается? И вообще контейнер указателей это бяка.
4. > delete this;
5. Почему main зависит от doctest?!

4/8.

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

1. Всё публично?
2. Константа 256, если от чего-то посчитана, должна считаться. А то вряд ли это реально исходный артефакт.
3. Сырые указатели!
4. Коды: std::string или std::vector<bool>?
5. Разбирать параметры объединив их в строчку худший план, тебе уже их дали раздельными. Тем более с какими-то циклами по пробелам. А если я захочу сжать файл Пример к лабе.txt?
6. Два дерева в архиваторе: одно для людей, другое для монстров?

2/5.

[kirillova.evgeniya]

Исправила не все...
Из того, что исправлено: изменен разбор входных параметров, коды теперь записываются в vector<bool>, зависимость main от doctest убрала(она вообще случайно получилась, когда я только разбиралась, как с doctest работать), у деревьев теперь человеческие названия, уже не все публичное, показано, откуда берется 256.

Сырые указатели заменить не справилась...
Насчет multiset - сортировка же по частотам, они могут повторяться, просто set вроде не будет работать

А еще не нашла проблему с размерами файлов, вроде для всех тех, на которых я тестила, выводятся правильные размеры...

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

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

1. Редко падает на сжатии.
2. Чиселки неправильные печатает.
3. На некоторых файлах слишком долго или коэффициент сжатия получился слабый.

7/9.

Тесты: ок.

Стиль:

1. Пространства имён.
2. Мультисет: так может сделать компаратор надёжным?)

6/8.

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

1. Сырые указатели.
2. Столько публичного всё ещё.
3. Опция через чарик это такое. И перепаршивать каждый раз странно. Логичнее было бы один раз разобрать и жить с результатом.

3/5.