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

AnyaGl / oop

Лабы по ООП

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\gladysheva\oop\labs\lab2\task2\HTMLDecode\Decoder.cpp(39,24): warning C4244: 'initializing': conversion from '__int64' to 'int', possible loss of data
2>C:\teaching\oop-labs\gladysheva\oop\labs\lab2\task2\HTMLDecode\Decoder.cpp(43,26): warning C4244: 'initializing': conversion from '__int64' to 'int', possible loss of data

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

alexey-malov commented 4 years ago

[ ] Программа неэффективно работает на последовательностях вида.
```
&&&&&&....&;
```
Подумайте, как улучшить ее работу при обработке подобных ситуаций. Например, при помощи https://ru.cppreference.com/w/cpp/string/basic_string/compare

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/bab816190c763d768700f93a07b4191a7d6ce869/labs/lab2/task2/HTMLDecode/Decoder.cpp#L4

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

alexey-malov commented 4 years ago

[ ] Код HtmlDecode выглядит сложно. Упростить