HW #3 turov.kirill

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

Корректность: Ни один тест не прошло, сразу падало. Что это было?

Тесты: формально не собрались. Тестируешь только разбор параметров, по-сути то. 1/8.

Стиль:

1. observer_ptr. Сильно, но вот это место скорее надо или обёртку делать, или оставить уже сырой, потому что тут сырой указатель имеет смысл!
2. Оверкил с разбором параметров, просто строчки на равенство было бы ок.
3. Код мёртвый в комментах.

5/8.

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

1. Константа в 8 кажется выражается через стандартное. Константы 256 больше похоже на магические числа.
2. Итераторы какие-то хитрые, кажется перестарался.

4/5.

Почини.

[Кирилл Туров]

Хочу много комментариев)
Кроме того, что внутренности compress и extract надо было вынести в отдельные функции,
я очень хотел это сделать, но не успел ((((

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

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

1. Чиселки неправильные печатаются.
2. Разжималка периодически падает. Кажется, это всё когда в исходном файле всего один вид байт.

6/9.

Тесты:

1. Недетерминированные тесты зло.
2. Ну и кроме маленького пустого теста получается не тестишь)

6/8.

Стиль:

1. using в заголовке: не неймспейс, но всё равно лучше стараться неймспейс не захламлять.
2. map: unordered_? Почти везде тебе последовательность символов не нужна.
3. // топ костыль
4. Хитрый формат дерева. Работает и норм, конечно, но процедура построения получилась хитрая. Я бы хранил тупо таблицу частот.

6/8.

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

1. Класс CLI: кажется, можно не хранить мапу. А ещё возвращать режим строчкой странно.
2. Обход шаблонными итераторами: что происходит-то?) Это точно архиватор?)

4/5.