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

voronindim / OOP

1 stars 0 forks source link

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Операция swap должна быть noexcept. У вас это не обеспечивается. https://en.wikibooks.org/wiki/More_C%2B%2B_Idioms/Non-throwing_swap

alexey-malov commented 4 years ago

[ ] Если default-конструктор выбросит исключение, то деструктор класса MyList не будет вызван и вместо итеративного удаления (вызов метода Clear) произойдёт рекурсивный (за счёт деструктора unique_ptr). Что в случае длинного списка приведёт к переполнению стека

alexey-malov commented 4 years ago

[ ] Операция swap должна быть noexcept. У вас это не выполняется. В результате оператор = не может обеспечить строгую гарантию безопасности исключений

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Выход исключения внутри noexcept функции приведет к аварийному завершению программы