Замечания по HTML Decode

[alexey-malov]

string StringReplace(const string& line, const string& search, const string& replace)
{
    string newString(line);
    size_t position = newString.find(search, 0);
    while (position != string::npos)
    {
        newString.replace(position, search.length(), replace);
        position += replace.length();
        position = newString.find(search, position);
    }
    return newString;
}

• [ ] Функция имеет квадратичную зависимость количества вычислений от длины строки

[alexey-malov]

• [ ] Тесты должны проверять декодирование всех html-сущностей.

[alexey-malov]

• [ ] Функция не покрыта тестами и содержит ошибку. Покрыть, исправить

[alexey-malov]

• [ ] Функция не покрыта тестами и содержит ошибку. Покрыть, исправить

[alexey-malov]

• [ ] Функция не покрыта тестами. Кто знает, может, она содержит ошибку. Покрыть тестами

[alexey-malov]

Используйте расширение OpenCppCoverage для VisualStudio, чтобы обнаруживать строки, которые никогда не выполнялись

[alexey-malov]

• [ ] В заголовочных файлах using namespace писать не следует. И вот почему: