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

KernelMrex / oop_labs

Лабораторные работы по ООП

0 stars 0 forks source link

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

[ ] Не подключен заголовочный файл <optional>. Программа не собирается под MSVC

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Зачем создавать динамический массив, а потом оборачивать его в shared_ptr? Разве предполагается возможность копирования и совместного использования массива? И чем обычный string не подошёл? Или даже string_view?

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

[ ] Зачем вообще вводить абстракцию Window, куда копировать символы, если есть string_view?

alexey-malov commented 4 years ago