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

[alexey-malov]

1>------ Build started: Project: MyString, Configuration: Debug x64 ------
2>------ Build started: Project: MyStringTests, Configuration: Debug x64 ------
2>MyString.cpp
2>MyStringTests.cpp
1>MyString.cpp
1>MSVCRTD.lib(exe_main.obj) : error LNK2019: unresolved external symbol main referenced in function "int __cdecl invoke_main(void)" (?invoke_main@@YAHXZ)
1>C:\teaching\oop-labs\gladysheva\oop\labs\lab5\MyString\x64\Debug\MyString.exe : fatal error LNK1120: 1 unresolved externals
1>Done building project "MyString.vcxproj" -- FAILED.

• [ ] Программа не собирается. Добавьте функцию main в основной проект

[alexey-malov]

https://github.com/AnyaGl/oop/blob/1a4fe40392a892c3119fc503a50d5802b0c6d874/labs/lab5/MyString/MyString/MyString.cpp#L114-L124

• [x] Утечка памяти. Массив char-ов, выделенный при помощи new[], нигде не удаляется при помощи delete[]

[alexey-malov]

https://github.com/AnyaGl/oop/blob/1a4fe40392a892c3119fc503a50d5802b0c6d874/labs/lab5/MyString/MyString/MyString.cpp#L126-L155

• [ ] Лучше поместить в безымянный namespace внутри .cpp файла, так как эти функции снаружи не вызываются

• [ ] Сравнение строк должно выполняться в лексикографическом порядке. То есть слово abdacadabra должно предшествовать слову zebra.

[alexey-malov]

https://github.com/AnyaGl/oop/blob/1a4fe40392a892c3119fc503a50d5802b0c6d874/labs/lab5/MyString/MyString/MyString.cpp#L157-L160

• [ ] Если строки имеют разную длину, то смысла проверять содержимое нет. Сделайте так, чтобы строки разной длины проверялись на равенство за O(1)

[alexey-malov]

• [ ] После применения move-конструктора строка-донор должна остаться в валидном состоянии. В частности, строка должна по-прежнему быть 0-terminated. У вас же после перемещения внутренний указатель строки становится null и большинство операций будут приводить к undefined behavior

[alexey-malov]

• [ ] Тесты операторов <, >, <=, >= не проверяют работу операций сравнения, так как тесты проходя, даже если в реализации всех этих операторов написать return true;

[alexey-malov]

• [ ] Метод GetStringData() должен всегда возвращать non null указатель на массив символов. Даже после того, как строка подвергнется воздействию move-конструктора или move assignment operator