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

[alexey-malov]

https://github.com/venera13/oop/blob/38d55b33015d47cd336b578aa77fe3d9fcf04688/lab6/HTTP_URL/HTTP_URL/CHttpUrl.h#L27-L28

• [ ] Методы, которые не используют состояние текущего объекта, должны быть объявлены константными

[alexey-malov]

https://github.com/venera13/oop/blob/38d55b33015d47cd336b578aa77fe3d9fcf04688/lab6/HTTP_URL/HTTP_URL/CHttpUrl.cpp#L99-L120

• [ ] Исключения следует ловить по ссылке

• [ ] stoi может выбросить и исключения другого типа

• [ ] Программа принимает URL-ы с номером порта, выходящим за пределы 1-65536, например, 70000

• [ ] Для чего протокол передается по ссылке?

[alexey-malov]

• [ ] Не выполнено требование задания:

  На основе данного класса напишите приложение, считывающее со стандартного потока ввода URL-ы и выводящее информацию об URL-е в стандартный поток вывода, пока не встретит символ конца файла.

[alexey-malov]

https://github.com/venera13/oop/blob/38d55b33015d47cd336b578aa77fe3d9fcf04688/lab6/HTTP_URL/HTTP_URL/CHttpUrl.cpp#L122-L127

• [ ] Метод будет работать некорректно для урлов https://google.com:80 и http://google.com:443

[alexey-malov]

https://github.com/venera13/oop/blob/38d55b33015d47cd336b578aa77fe3d9fcf04688/lab6/HTTP_URL/HTTP_URL/CHttpUrl.cpp#L78-L97

• [ ] Как-то странно, что мы сравниваем строку то с http, то с http://

[alexey-malov]

• [ ] В тестах следует проверить парсинг портов вблизи граничных значений