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

[alexey-malov]

BOOST_AUTO_TEST_CASE(create_exception_error)
{
    BOOST_CHECK_THROW(throw CUrlParsingError("error text"), std::invalid_argument);
}

• [x] Не надо тестировать оператор throw

[alexey-malov]

BOOST_AUTO_TEST_CASE(create_exception_error_if_url_is_empty)
{
    BOOST_CHECK_THROW(CHttpUrl(""), std::invalid_argument);
}

• [x] Выбрасывается исключение типа CUrlParsingError, поэтому проверять следует его

[alexey-malov]

BOOST_AUTO_TEST_CASE(create_exception_if_port_is_incorrect)
{
    BOOST_CHECK_THROW(CHttpUrl("http://ab.ru:as6"), std::invalid_argument);
    BOOST_CHECK_THROW(CHttpUrl("http://ab.ru:0"), std::invalid_argument);
}

• [x] Следует проверять оба граничных значения порта в одном тесте

[alexey-malov]

BOOST_AUTO_TEST_CASE(create_exception_error_if_url_has_wrong_protocol)
{
    BOOST_CHECK_THROW(CHttpUrl("httd://last.fm"), std::invalid_argument);
}

BOOST_AUTO_TEST_CASE(create_exception_error_if_url_has_wrong_parameters)
{
    BOOST_CHECK_THROW(CHttpUrl("httd://last.fm/image.jpeg/image2.jpeg"), std::invalid_argument);
}

• [x] Второй тест не привносит ничего нового по сравнению с первым

[alexey-malov]

• [x] Тесты должны проверять оба значения порта вблизи каждой границы, а не только невалидные

[alexey-malov]

https://k:65537
bad lexical cast: source type value could not be interpreted as target

• [ ] Должно выводиться человекочитаемое сообщение об ошибке. Обрабатывайте bad_lexical_cast внутри HttpUrl и перевыбрасывайте исключение с понятным сообщением

[alexey-malov]

http://google.com:80
Protocol: http
Domain: google.com
Port: 80
Document:
Url: http://google.com:80

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

[alexey-malov]

• [ ] При встрече символа конца файла не нужно выводить сообщение Url is empty

[alexey-malov]

• [ ] Метод GetURL не возвращает строковое представление URL-а, если URL был сконструирован из отдельных компонентов

[alexey-malov]

• [ ] stringPort переводится как "порт сроки"

[alexey-malov]

[alexey-malov]

            if ((port < MIN_PORT_VALUE) || (port > MAX_PORT_VALUE))
            {
                throw CUrlParsingError("Invalid port value");
            }

• [ ] Условие (port > MAX_PORT_VALUE) будет false на компах, где unsigned short имеет разрядность 16 бит

[alexey-malov]

    std::regex urlRegex(R"(^(\w+):\/\/([^\s:\/]+)(?::(\d+))?(?:\/(\S*))?$)");

• [ ] Разобраться и уметь объяснить, для чего стоит буква R перед строковым литералом

• [ ] Разобраться и уметь объяснить, что означает каждый символ в этом регулярном выражении. Для экспериментов рекомендую сайт https://regexr.com/

[alexey-malov]

CHttpUrl::CHttpUrl(std::string const& domain, std::string const& document, Protocol protocol)
    : m_protocol(protocol)
    , m_document(document)
{
    if (!domain.empty())
    {
        m_domain = domain;
    }
    else
    {
        throw CUrlParsingError("Empty domain");
    }
}

• [ ] Порт остаётся неинициализированным. Проверить в тестах и исправить

[alexey-malov]

CHttpUrl::CHttpUrl(std::string const& domain, std::string const& document, Protocol protocol, unsigned short port)
    : m_protocol(protocol)
    , m_document(document)
{
    if (!domain.empty())
    {
        m_domain = domain;
    }
    else
    {
        throw CUrlParsingError("Empty domain");
    }
    m_port = ParsePort(std::to_string(port), m_protocol);
}

• [ ] Зачем преобразовывать порт в строку, а потом обратно из строки в число?

[alexey-malov]

• [ ] Сделайте вспомогательную функция в тестах, сравнивающую параметры CHttpURL с ожидаемыми. Используйте её вместо ручных проверок отдельных свойств. Это поволит не пропустить неправильно проинициализированные поля класса