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

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

[ ] Операция swap у вас будет использовать move конструктор, который вы не сможете сделать noexcept, если внутри него будет выделяться память. Лучше сделайте noexcept метод swap внутри класса списка.

alexey-malov commented 4 years ago

[ ] Если оператор new выбросит исключение оператор упадёт. Если хотите сделать перемещающий оператор присваивания noexcept, оставляйте сторожевые ноды на месте и перекидывайте указатели

alexey-malov commented 4 years ago

[ ] Реверсированный итератор должен быть отдельным типом, а не состоянием (иначе присваивание превратит реверсированный итератор в нереверсированный, а сравнивать реверсированный итератор с нереверсированным, по идее, должны быть тоже нельзя. Проще всего сделать один обычный итератор обернуть его в std::reverse_iterator

alexey-malov commented 4 years ago

[ ] у const списка должны возвращаться итераторы, позволяющие только читать данные. У вас же возвращается просто константный итератор, который можно скопировать в обычный итератор и выполнять любые действия над данными

alexey-malov commented 4 years ago

P.S. По шаблонному списку, если он сделан на основе списка строк, будут подобные замечания

voronindim / OOP