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

[alexey-malov]

https://github.com/miredirex/oop-course/blob/36f00894819cf8d6442987b69d676ed0c957298c/Task1/Replace/replacer.h#L7-L27

• [ ] Зачем имя передаётся каждый раз в метод AppendResult

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

• [ ] А что, если надо будет выполнить замену не в файле, а в строковом потоке?

• [ ] А что, если искомая строка будет очень длинной и тогда придётся каждый раз вычислять длину, пробегая по всем символам

[alexey-malov]

https://github.com/miredirex/oop-course/blob/36f00894819cf8d6442987b69d676ed0c957298c/Task1/Replace/replacer.cpp#L5-L12

• [ ] Использование exit в классе делает сомнительным его пригодность к повторному использованию.

[alexey-malov]

• [ ] У строки есть метод find

[alexey-malov]

• [ ] Для индексации строки следует использовать size_t, а не int