HW #3

[gordeeva.tatyana]

"И я не видел смысла делать плохо,
И я не видел шансов сделать лучше."

[Vasily Alferov]

Неплохо. Только не особо работает.
С косметической точки зрения мне нравится.

Договорённость выполнена, баллы не урезаются, попытку до четверга можно.

Корректность: 3/9
Тесты: 7/8
Архитектура: 5/5
Стиль: 7/8

По делу:

• Тесты под валгриндом показывают ub.
• Числа выводятся хронически не те.
• На бинарных данных есть проблемы. Попробуй тест "все байты от 00 до ff". Или просто один байт "ff".
• Зависимость от fstream — фу. В мире столько разных классных стримов. Почему бы их не сжимать. Заодно и файлы в тестах заводить и удалять не надо.

По мелочи:

• CLIException. Во-первых, принято наследоваться от потомков std::exception. Во-вторых, можно так, как у тебя, но вообще проще так:

  class CLIExcpetion : public std::runtime_error {
  public:
      explicit CLIException(const std::string& report)
        : std::runtime_error(report)
      {}
  };
  

[Vasily Alferov]

Круто.

Корректность 7/9, остальное фулл.

По корректности:

• Числа всё ещё не те :(
• Рандомный большой бинарный файл всё же не работает. Сходу не нашёл, почему. По заголовку бы предположил, что беды с кодом байта '\n'.