Замечания по HttpURL about oop HOT 9 OPEN

• методы класса, не требующие обращения к данным класса, следует сделать статическими

from oop.

• mistake - это ошибка, совершаемая человеком, а не компьютером

from oop.

https://localhost:80
URL: https://localhost/
Protocol: HTTPS
Domain: localhost
Port: 80
Document path: /

http://localhost:443
URL: http://localhost/
Protocol: HTTP
Domain: localhost
Port: 443
Document path: /

• Порт не должен выводиться, только если он соответствует дефолтному порту протокола

from oop.

void CHttpUrl::ParseUrl(std::string const & url)
{
	std::string urlStr(url);

	auto protocolStr = ParseProtocol(urlStr);
	m_protocol = StrToProtocol(protocolStr);
	urlStr = urlStr.substr(protocolStr.size() + 3);
	
	m_domain = ParseDomain(urlStr);
	urlStr = urlStr.substr(m_domain.size());

	if (urlStr[0] == ':')
	{
		m_port = std::stoi(ParsePort(urlStr));
		urlStr = urlStr.substr(std::to_string(m_port).size() + 1);
	}
	else
	{
		m_port = (m_protocol == Protocol::HTTP) ? 80 : 443;
	}
	
	m_document = ParseDocumentPath(urlStr);
}

• Реализовать без отрезания головы. Чтобы не передавать постоянно начальный индекс, можно воспользоваться boost::string_ref

from oop.

• ProtocolToStr сделайте свободной функцией ,принимающей протокол

from oop.

• Конкретизируйте имена тестов, чтобы они читались как спецификация

from oop.

• Не покрыты тестами ветки кода. Внесенная ошибка не обнаружена

from oop.

https://ya.ru:80
Invalid URL port.
http://ya.ru:80
Invalid URL port.

• ошибка. Воспроизвести в тестах

from oop.

k=0.8

from oop.