Замечания по программе extract

alexey-malov commented 4 years ago

[x] Программа не соответствует принятому стилю кодирования. Ознакомьтесь с правилами именования переменных, функций, классов, расстановкой скобок, знаков препинания и отступов.
[x] Вместо optional<Args> лучше принимать отдельные аргументы. Во-первых, Args имеют значение только в контексте функции main. Во-вторых, имена файлов функция не использует. В-третьих, опциональность аргументов командной строки была нужны тоже только внутри main

alexey-malov commented 4 years ago

[x] Выделить функцию, извлекающую фрагмент входного файла в выходной, зная их имена и фрагмент
```
bool ExtractFrament(const string& inputFileName, const string& outputFileName, int start, int length)
```
Это упростит функцию main

alexey-malov commented 4 years ago

[x] Предпочтительнее использовать std::stoi
[x] Надо ловить исключения (см. у меня в репозитории пример numeric_string_conversion)

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[x] Добавьте тесты на ситуацию с извлеченим фрагмента за пределами файла (в положительную область) полностью или частично в соответствии с заданием

alexey-malov commented 4 years ago

[ ] объекты следует передавать по константной ссылке, если функция не модифицирует их состояние. Примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению
[ ] Принимайте istream& и ostream& вместо ifstream& и ofstream&

alexey-malov commented 4 years ago

[x] Этот блок следует выделить в отдельную функцию, то есть функция Extract, принимающая имена файлов и смещение с длиной должна строиться на функции Extract, принимающей стримы и смещение с длиной

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[x] Дополнить информацию об использовании программы, чтобы было ясно, какой синтаксис командной строки

artyRay12 / OOP