hw_02 rachkin.vladimir

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

Корректность: см первый пункт стиля. 0.

Тесты: интеграционные лишь, крайние случаи не разбирают, какая именно статистика считается не проверяют. 3.

Стиль:

1. Ой сколько предупреждений! В том числе ооооочень плохих: argv[i] == "-c" сравнивает указатели. Оно не может работать.
2. Судя по тому, что все файлы выглядят исполняемыми (даже текстовые!), а исходник с тестами выглядит ​вот так, ты с винды.
3. Война и мир строчкой на куче, копируемой из функции, это ооой. Если что:

   std::string_view sampleTest = "a"....
   

   const char[] тоже ок был бы. Да даже если вернуть std::string_view было бы лучше)
4. const int COUNT_OF_CHAR = 256; а стандартные константы? Вообще какие-то странные волшебные константы.
5. Сырые указатели.
6. //private:.

6.

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

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

1. Периодически падает. С ором валгринда.
2. А статистику оно печатает?

9.

Тесты:

1. А ЧО ИМЕНА ТАКИЕ ГРОМКИЕ
2. Было бы круто проверить конкретную статистику и построение дерева.

3.

Стиль:

1. -Wreorder: списки инициализации в правильном порядке. Ещё предупреждения.
2. По байтиками лучше бежать как unsigned char, а то потом индекс в массиве случается иногда.
3. Я вижу что ты восстанавливаешь текст целиком в памяти.
4. Сырые указатели а правила 5 не вижу.
5. Волшебные числа. Совсем много.
6. Странные имена методов. Что такое длинный архив? Точно ли всё публично?

6.

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

Корректность: но просили печатать не так. Всё ещё падает, если в тексте есть символ больше 127 по коду. 9.

Тесты:
Ну ок. Хотя от одной беды не спасли. 5.

Стиль:

1. -Wreorder.
2. Сырые указатели без всех спец методов и всё публичное.
3. Волшебные числа.
4. CHAR_MAX, UCHAR_MAX и ппочая знаковость. Если это поправить то работает!

7.