Замечания по HtmlEncode (л.2)

[sergey-shambir]

• в HtmlEncode.h пропущен пробел у PrintString.
• Также вместо #include "string" следует писать #include <string>, чтобы компилятор не искал файл string в текущей папке, аналогично для всех файлов из стандартной библиотеки.
• Также забыт #pragma once.
• Функция ReadInString наверное должна называться ReadIntoString
• Пусть лучше функция ReadIntoString возвращает std::string, вместо переданного по ссылке out-параметра. Начиная с c++ 2011, оптимальнее возвращать строку значением и тем самым позволить компилятору переместить строку через move-конструктор. Есть только одна причина передавать строку как out-параметр по ссылке: если надо переиспользовать уже выделенную в строке память, например при чтении ввода построчно.
• в теле функции HtmlEncode следует использовать switch вместо if, т.к. компилятору тогда проще применить оптимизации (таблица jmp или binary search вместо прямого перебора)
• файлу stdafx.h пусто и одиноко, нужно добавить в него <string>

[sergey-shambir]

Ах да, тестов мало и граничные случаи не проверены. Пока без коэффициента.