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

[alexey-malov]

https://github.com/arina-kaa/OOP/blob/9bffba8f1b44aa12985230ac52b17cbc4ed7f6ec/lab_01_04/lab_01_04.cpp#L69

• [x] Принимать 4 аргумента вместо опциональной структуры

• [x] Возвращать лучше не int, а bool, который внутри main должен сконвертироваться в int должным образом

https://github.com/arina-kaa/OOP/blob/9bffba8f1b44aa12985230ac52b17cbc4ed7f6ec/lab_01_04/lab_01_04.cpp#L52-L67

• [x] Принимать istream& и ostream& вместо ifstream& и ofstream&

• [x] Принимать строки для замены

• [x] Проверку на пустоту искомой строки лучше вынести внутрь ReplaceString:

• Не будет риска зависания при передаче пустой строки

[arina-kaa]

Замечания исправлены.

[alexey-malov]

https://github.com/arina-kaa/OOP/blob/e3e7619da0c0c49d54e70f258ca530d0c8eba708/lab_01_01_04/lab_01_04.cpp#L68-L69

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

[alexey-malov]

https://github.com/arina-kaa/OOP/blob/e3e7619da0c0c49d54e70f258ca530d0c8eba708/lab_01_01_04/lab_01_04.cpp#L57-L59

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

[arina-kaa]

Замечания исправлены.