Замечания по программе HtmlDecode

Curpiq / oop_labs

лабы по ооп

GNU General Public License v3.0

0 stars 0 forks source link

Замечания по программе HtmlDecode #5

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

2>C:\teaching\oop-labs\semenova\oop_labs\lab02\HTMLDecode\HTMLDecode\StringDecoder.cpp(49,68): warning C4267: '+=': conversion from 'size_t' to 'int', possible loss of data

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

alexey-malov commented 4 years ago

https://github.com/Curpiq/oop_labs/blob/f689fed4a05d718f4702bb689b81e8ca78981a56/lab02/HTMLDecode/HTMLDecode/StringDecoder.cpp#L26-L44

[ ] Что такое marker?
[ ] Зачем нужен аргумент entityNumber?
[ ] Для чего нужна промежуточная строка str? В классе string есть метод compare, позволяющий сравнить часть одной строки с другой без каких либо промежуточных строк

alexey-malov commented 4 years ago

https://github.com/Curpiq/oop_labs/blob/f689fed4a05d718f4702bb689b81e8ca78981a56/lab02/HTMLDecode/HTMLDecode/StringDecoder.cpp#L7

[ ] У вектора есть метод size(), возвращающий длину.

alexey-malov commented 4 years ago

[ ] программа неудачно разбита на функции. подумайте, как можно сделать лучше