HA2 Petryakov Ivan

[Иван]

Здравствуйте! Немножко не уложился в сроки, возникли некоторые трудности. Но теперь все работает (вроде как), smoke test пройден! Проверьте, пожалуйста.

[Филипп]

Привет! Несколько вещей на доработку:

1. При сравнении аргументов можно смело приводить argv[i] к std::string, чтобы использовать operator==(const char *)
2. Похоже, compress подхватывает нулевой символ даже там, где этого не нужно: 0A.in, ab.in -- из-за этого кодировка неоптимальная
3. Лучше называть переменные более явно: full_size + add_size * 5 + 8, add_size * 5 + 8 выглядят очень по-магически :) ans/answer легко спутать, и по наименованию непонятно, чем они отличаются.
4. Константная ссылка у параметров: в visual_tree

[Иван]

Добрый вечер! Спасибо за замечания. У меня возникли сложности со вторым пунктом еще на предыдущем этапе. До этого я просто декрементировал значение у '\0', но мне показалось это страшным костылем и я решил обойтись без этого =) А существуют ли другие способы исправить эту ошибку?

[Филипп]

​http://www.cplusplus.com/reference/ios/ios/eof/ -- про .eof()
​http://www.cplusplus.com/reference/ios/ios_base/iostate/ -- про eofbit

В текущей реализации кода проверять на eof нужно после чтения символа. Проще это делать через get/read.

И обязательно нужно убрать декремент (сейчас он работает): map::operator[] создаст элемент, если его нет в коллекции.

[Иван]

Здравствуйте! Исправил Ваши замечания. Все работает как нужно, без лишних декрементов!

[Филипп]

Привет!

Проверь, пожалуйста, работу программы с пустым файлом; с файлом 0D.in -- не совпадает статистика.
И логика флага -v стала по-умолчанию включена.

Поправь, пожалуйста, по возможности в течение 60 часов.

[Иван]

Добрый вечер! Спасибо за замечания. Я немножко запутался в выводе. Теперь поправил, чтобы было как в задании.

[Филипп]

Не все поправилось (пример aaaa.in, его работа с флагом -v), но довести до конца можно :)