Замечания по программе парсинга URL-ов

GreatHorizon / oop

Object Oriented Programming Course

1 stars 0 forks source link

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

[ ] Здесь у вас утечка памяти - выделенная в куче при помощи new переменная не освобождается нигде. Функции stoi вы передаете адрес переменной, куда она должна сохранить позицию, где остановился разбор строки. Этот указатель не обязательно должен указывать на память, выделенную в куче. Посмотрите пример: https://github.com/alexey-malov/oop/blob/1fd8fd85bd8b3ab50e1d4bd18906020236469ebb/samples/01-basics/numeric-string-conversion/main.cpp#L35-L40

alexey-malov commented 4 years ago

[ ] Исключения следует ловить по константной ссылке, а не по значению, чтобы избежать срезки (slicing). В редких случаях можно ловить по неконстантной ссылке, если требуется модифицировать пойманный объект исключений перед перевыбросом его.

alexey-malov commented 4 years ago

[ ] Неудачно имя константы. Что за "значения по умолчанию"? Значения чего? Не нужно объявлять эти константы в заголовочном файле так как используются они только внутри одного cpp файла

alexey-malov commented 4 years ago

[ ] Блок catch не покрыт тестами (должно быть 2 случая - ошибка парсинга числа и переполнение)