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

[alexey-malov]

• [x] Поддержку c++ 17 выставить во всех конфигурациях и платформах в настройках проекта

[alexey-malov]

• [x] Пересохранить bat-файл в 1251

[alexey-malov]

• [x] Отформатировать исходники при помощи clang-format

[alexey-malov]

• [x] Программа не соответствует принятому стилю кодирования. Ознакомьтесь с правилами именования переменных, функций, классов, расстановкой скобок, знаков препинания и отступов.

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L33

• [x] объекты (например, строки) следует передавать по константной ссылке, если функция не модифицирует их состояние. Примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению

• [x] newStr - неудачное имя переменной. Подобрать получше

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L37

• [x] Можно в C++ не писать = ""

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L39-L41

• [x] Эту проверку не нужно выполнять на каждой итерации цикла

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L43-L45

• [x] Проверку на равенство npos нужно выполнить до использования переменной

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L50-L53

• [x] Можно заменить на break

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L62-L65

• [x] Добавление символа перевода строки не является задачей функции Replace

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L67-L69

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

• [x] лучше принимать istream& и вместо ifstream&, тогда можно будет принимать не только ifstream, но и, например, istringstream

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L75-L84

• [x] Не используйте exit, кроме как в крайних ситуациях

[alexey-malov]

https://github.com/AnyaGl/oop/blob/52d843c537025db80c42a78f35b0b6abd01abc52/labs/lab1/replace/replace.cpp#L96-L115

• [x] Лучше выделить в отдельную функцию, принимающую имена файлов и строки для замены

[alexey-malov]

• [x] В CopyFileWithReplace вместо out-параметра использовать возвращаемое значение. Вообще инициализацию out-параметра вызывающей функции не нужно доверять.