Замечания по HtmlEncode / HtmlDecode

Romasmi / cpp-practice

0 stars 0 forks source link

Замечания по HtmlEncode / HtmlDecode #15

Closed alexey-malov closed 5 years ago

alexey-malov commented 5 years ago

[x] Программа неверно будет работать, если встретит между & и ; что-то, не являющееся сущностью.
[x] Добавить тесты на обработку этой ситуации
[x] Добавить & и ; к сущностям в функции кодирования, тогда не надо будет на лету заниматься склейкой строк

alexey-malov commented 5 years ago

std::string HtmlDecode(const std::string str)

[x] объекты следует передавать по константной ссылке, если функция не модифицирует их состояние. Примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению

alexey-malov commented 5 years ago

    for (HtmlEntityCode item : HTML_ENTITIES_CODES)
    {
        if (code == item.second)
        {
            return item.first;
        }
    }

[x] Перебирать по константной ссылке

alexey-malov commented 5 years ago

        const string code = GetEntityCode(ch);
        if (!code.empty())
        {
            processedString.append(code);
        }
        else
        {
            processedString += ch;
        }

[x] Лучше сделать функцию HtmlEncode(ch), которая вернёт строку, содержащую html-представление символа, либо сам символ. Тогда не нужен будет if