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

[alexey-malov]

1>------ Build started: Project: replace, Configuration: Debug x64 ------
1>mian.cpp
1>C:\teaching\oop-labs\romanov\oop\lab1\replace\replace\mian.cpp(48,47): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data

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

[alexey-malov]

• [ ] mian.cpp -> main.cpp

[alexey-malov]

• [ ] Функция main не должна открывать файлы. Ее задача получить аргументы командной строки и вызвать функцию, копирующую содержимое файла с заменой подстроки

[alexey-malov]

https://github.com/RomanovAleksandr/oop/blob/04d4a8fdb4cb23189382b083698ae7dec7364738/lab1/replace/replace/mian.cpp#L57-L91

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

[alexey-malov]

https://github.com/RomanovAleksandr/oop/blob/04d4a8fdb4cb23189382b083698ae7dec7364738/lab1/replace/replace/mian.cpp#L32-L55

• [ ] объекты следует передавать по константной ссылке, если функция не модифицирует их состояние. Примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению
• [ ] Выделите функцию замены всех вхождений подстроки в строке
• [ ] Достаточно просто объявить переменную типа string, чтобы получить пустую строку. Использование = "" при инициализации строки вызывает вместо конструктора по умолчанию конструктор, который сперва должен вычислить длину zero-terminated строки.