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

[alexey-malov]

• [x] Вместо одного большого теста сделать несколько маленьких, проверящих по одному аспекту поведения тестируемой функции.
• [x] Написать тесты в bdd-стиле

[alexey-malov]

• [x] входные данные тестов следует сделать более человекочитаемыми

[alexey-malov]

• [ ] BOOST_CHECK_EQUAL Почитайте документацию boost.test library на boost.org, почитайте описания макросов, служащих для проверки

[alexey-malov]

std::string FindAndReplace(const std::string & subject, const std::string & search, const std::string & replace)
{
    std::string result;
    std::string window = "";
    for (auto ch : subject)
    {
        window += ch;
        if (window.size() == search.size())
        {
            result += window == search ? replace : window;
            window = "";
        }
    }
    result += window;
    return result;
}

• [ ] Use string::find method, Luke. А еще лучше, алгоритм KMP,

[alexey-malov]

• [x] Алгоритм не всегда работает на строках длиннее одного символа. Пример теста, который не проходит.

    BOOST_AUTO_TEST_CASE(search_for_multi_char_string)
    {
        BOOST_CHECK_EQUAL(FindAndReplace("hello world", "world", "world-world"), "hello world-world");
    }