HW_03 vorobeva.polina

[vorobeva.polina]

Привет)

Для коммита ДО дедлайна справедливо: работает ТОЛЬКО на маленьких объемах данных, таких, чтобы коды в результате алгоритма Хаффмана были не длиннее 8 символов. Это плохо, но я так и не разобралась, как это исправить:(
Мало тестов. Очень мало, для дерева они вообще не написаны, т.к. я было не совсем понятно, что там нужно тестить...
Не работает для файлов, где всего один символ - жесткое UB.

Для коммита ПОСЛЕ дедлайна (на 20 минут, к сожалению) справедливо: Все тоже самое, только тестов там на один больше и этот код работает для файлов, состоящих ровно из 1 символа (в общем-то, это тот баг который фиксился после отправки предыдущего коммита)

В общем и целом, извини, что мой коммит такой. Возможно, он был бы лучше, если бы я дружила с тайм-мэнеджментом)

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

Корректность: валгринд орёт на каждом тесте. На пустом файле смерть. А как-то дальше ваще не работает.

Тесты: вроде тестят. Но хотелось бы тестов на построение дерева или какой-то такой. 6/8.

Стиль:

1. Зачем ты хранишь компаратор?
2. Compere. как-как?
3. Неймспейсы есть, но они все не несут того имени, что изолируют эти имена от других. my_exceptions. Вот положить бы всё это в huffman_archiver.

6/8.

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

1. Сырые указатели. Тем более хранить их в контейнере.
2. BYTES = 4. Все байты мира почему-то 4.

4/5.

[vorobeva.polina]

Привет)
Тикет отправляю значительно позже, чем закоммитила последнюю версию...

Версия ДО дедлайна содержит дополнительные тесты, работает для пустых файлов. Всё также падает, почти везде, я полагаю)
Версия ПОСЛЕ дэдлайна, но отправленная ещё в четверг - вроде лучше. По крайней мере у меня на больших объемах данных работала)

Память течёт, ибо я не разобралась, как туда внедрить умные указатели) Ну или я хотела усложнить себе жизнь и не поняла, что там на самом деле было нужно...
Компаратор остался храниться в дереве, т.к. без него внутри нельзя настроить правильный компаратор set'а во время компиляции. Как это побороть без лишнего хранения - не знаю)

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

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

1. Валгринд. Умные указатели упростили бы жизнь.
2. Расжималка тупо падает. Не упало на 4 тестах, их них только 2 всё ок, на други разжалось не то.

1/9.

Тесты: закомменченное не считается. 2/8.

Стиль:

1. Пробелы!
2. Управляющие конструкции без переноса строчки.
3. Компаратор с <=.

5/8.

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

1. avaliableFlags: идея такая себе. Имена плохие. И я бы сказал что вообще это перечисление не нужно в заголовке.
2. CLI::printStatistics: принимает копию архиватора?
3. Указатели сырые.

3/5.

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

Предыдущая посылка в сумме сильнее, но я поставил 1 за корректность.