HW #3 chizhova.mariya

[Maria Chizhova]

Вроде работает...

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

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

1. Выводишь числа, не соответствующие файлам.
2. Падает на сжатии. Не могу выделить на чём именно, но кажется когда в тесте не искусственно мало видов байт. Но когда дали 5 миллионов 0xff тоже упал. И когда весь файл 1 байт периодически падает.

4/9.

Тесты: работают. Но они лишь интеграционные, и крайние случаи не покрывают. Хочется ещё всяких более конкретных. 5/8.

Стиль:

1. Пробелы.
2. Пространства имён.
3. CLI: std::string_view, operator ==.
4. Предложения через запятую.
5. Предложение else на той же строчке.
6. if (tree[i].left != -1): isLeaf?

4/8.

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

1. Константа в 256, чую я, не является артефактом оригинальным, наверно она из чего-то считается.
2. Конструкция хождения по дереву прикольная, но это же не итератор ;)
3. Работа с потоком бит просится в отдельный класс(ы).

4/5.

[Maria Chizhova]

Я не понимаю почему у меня падает именно что-то с символом 0xff. А так вроде остальное работает.

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

Не собирается.

Кстати, Archiver, не Achiever)

Корректность: А падает всё так же, причины единой не видно.

Тесты: там же

Стиль:

1. Пространства имён всё ещё не всё захватили.

Остальные вопросы там же.
4.

Архитектура: аналогично.

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

Про стиль: да, что-то проглядел.

Стиль:

1. Пространства имён всё ещё не всё захватили. Более того, такие разные пространства сами по себе не полезны. Вот если бы всё положить в huffman.
2. Пробелы в паре мест остались.
3. buildCode: isLeaf не появился.

6/8.

Архитектура: без вопросов. 5.

Это +3 балла, но формально посылка из-за мелочи не собирается, но я прибавил.