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

[alexey-malov]

https://github.com/artyRay12/OOP/blob/160dfb139944a5edab0a47cc6bcab35724148ad6/lab2/htmlEncode/htmlEncode/htmlEncoder.cpp#L4

• [ ] Поиск по массиву будет быстрее, чем по дереву, когда количество элементов небольшое

• [ ] Что за test?

[alexey-malov]

https://github.com/artyRay12/OOP/blob/160dfb139944a5edab0a47cc6bcab35724148ad6/lab2/htmlEncode/htmlEncode/htmlEncoder.cpp#L18-L30

• [ ] Индексы не нужны. Для обработки строки достаточно использовать range-based for

• [ ] Функция не соответствует требованиям задания

• [ ] Сложность функции большая из-за сдвигов хвоста

[alexey-malov]

https://github.com/artyRay12/OOP/blob/160dfb139944a5edab0a47cc6bcab35724148ad6/lab2/htmlEncode/htmlEncode/htmlEncoder.cpp#L20-L24

• [ ] поиск выполняется дважды - один раз в find, второй раз операторе []. выполнить поиск за один раз

if (auto it = map.find(key); it != map.end()) // чтобы работал такой if надо включить в проекте поддержку C++17
{
    // найденное значение находится в it->second
    //можно использовать его внутри этого блока
}