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

AnyaGl / oop

Лабы по ООП

0 stars 0 forks source link

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

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/33879160095cd87334917802b46ca18c41a19c1d/labs/lab7/MyList/MyList/MyList.h#L146-L156

[ ] Конструктор перемещения не может обеспечить гарантию noexcept, так как реализован с использованием default-конструктора, бросающего исключения. Просто смиритесь с фактом, что при наличии сторожевых узлов вам, скорее всего, придётся использовать аллокации при конструировании объекта/ Рекомендую почитать статью одного из разработчиков STL (для MS Visual c++) на эту тему https://isocpp.org/files/papers/N4055.html

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/33879160095cd87334917802b46ca18c41a19c1d/labs/lab7/MyList/MyList/MyList.h#L167-L183

[ ] В этой реализации у вас тоже не выйдет обеспечить noexcept для оператора перемещения, если вы используете внутри него default конструктор, выполняющий аллокации. Но оператор перемещения можно сделать noexcept, если вы оставите текущие сторожевые ноды у приёмника и источника на месте, но перекинете остальные узлы.

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/33879160095cd87334917802b46ca18c41a19c1d/labs/lab7/MyList/MyList/MyList.h#L158-L166

[ ] Тут рекомендую добавить в класс списка noexcept метод swap, выполняющий эффективный обмен данных двух списков. Тогда и std::swap вы сможете сделать noexcept (сейчас у вас не выйдет это сделать, так как default конструктор у вас может бросать исключения, но если реализация std::swap будет использовать метод swap, не создающий промежуточных объектов, а меняющий данные напрямую).

https://en.wikibooks.org/wiki/More_C%2B%2B_Idioms/Non-throwing_swap

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/33879160095cd87334917802b46ca18c41a19c1d/labs/lab7/MyList/MyList/MyList.h#L266-L287

[ ] Эти методы вполне можно сделать noexcept (убедитесь, что они вызывают только noexcept функции и методы)

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/33879160095cd87334917802b46ca18c41a19c1d/labs/lab7/MyList/MyList/MyList.h#L313-L320

[ ] Тоже можно объявить noexcept