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

[alexey-malov]

• [ ] После перемещения строка-донор должна оставаться в валидном состоянии. В частности, метод GetStringData() должен возвращать указатель на zero-terminated массив символов (у вас возвращается nullptr). Операции, в которых фигурирует строка-донор также должны корректно выполняться (в частности, не должен передаваться nullptr в функции вроде memcmp или memcpy)

[alexey-malov]

https://github.com/voronindim/OOP/blob/5c2c10e19b3475eaddcdb4ced2009d552300751f/lab_5/CMyString/CMyString.cpp#L72-L78

• [ ] Если оператор new выбросит исключение, указатель m_pString будет указывать на уже удаленную область памяти, а последующий вызов деструктора приведет к повторному вызову delete уже удаленного массива символов

[alexey-malov]

https://github.com/voronindim/OOP/blob/5c2c10e19b3475eaddcdb4ced2009d552300751f/lab_5/CMyString/CMyString.cpp#L116

• [ ] Исправьте опечатки. Вместо слова номер лучше использовать индекс

[alexey-malov]

https://github.com/voronindim/OOP/blob/5c2c10e19b3475eaddcdb4ced2009d552300751f/lab_5/CMyString/CMyString.cpp#L128-L135

• [ ] Утечка памяти - память, выделенная через new нигде не освобождается

[alexey-malov]

https://github.com/voronindim/OOP/blob/5c2c10e19b3475eaddcdb4ced2009d552300751f/lab_5/CMyString/CMyString.cpp#L137-L151

• [ ] Можно ускорить сравнение, воспользовавшись функцией memcmp