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

ASTeterin / OOP

0 stars 0 forks source link

Замечания по HtmlEncode #6

Closed alexey-malov closed 5 years ago

alexey-malov commented 5 years ago

1>c:\teaching\ips\2019\teterin\oop\lab2\htmlencode\main.cpp(18): warning C4326: return type of 'main' should be 'int' instead of 'void'

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

alexey-malov commented 5 years ago

[x] HTML_ESSENCE -> HTML_ENTITIES

alexey-malov commented 5 years ago

        string htmlCode = GetHtmlCode(ch);
        if (htmlCode.empty())
        {
            processedLine += ch;    
        }
        else
        {
            processedLine.append(htmlCode);
        }

[x] Вместо GetHtmlCode сделайте HtmlEncodeChar(ch), которая бы возвращал либо строку, содержащую либо сущность, либо сам символ. Тогда здесь не нужен будет if

alexey-malov commented 5 years ago

string HtmlEncode(string const& encodingString)
{

    char ch;
    string processedLine;

    if (encodingString.empty())
    {
        return "";
    }

    istringstream sourceString(encodingString);
    while (sourceString.get(ch))

[x] Проверка на пустоту - избыточна.
[x] не нужно созадавать stringstream. Символы строки можно перебирать при помощи for:
```
for (auto ch : encodingString)
{
}
```

alexey-malov commented 5 years ago

[x] Имена и структура тестов должны играть роль спецификации на тестируемую функцию или класс, а не просто перечислить ситуации, в которых вы проводите испытание класса или функции.

В вашем случае можно сделать по одному тесту для преобразования каждой сущсности + один тест на преобразование строки, содержащей все сущности и обычные символы. Можно добавить тест на пустую строку