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

[alexey-malov]

• [ ] В C++ для итератора, возвращаемого при помощи end(), должна иметься возможность выполнить декремент. Одно из возможных решений - добавить фиктивную ноду в конец списка

[alexey-malov]

• [ ] Направление итератора является частью типа, а не частью его состояния. То есть прямой и обратный итераторы должны быть разными типами. Рекомендую почитать про std::reverse_iterator

[alexey-malov]

• [ ] Для итераторов не заданы постфиксные формы инкремента и декремента

• [ ] Нет возможности пробежаться итератором по константному списку

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/b8668444aa95ef37f09f618aa5e3b9da0a379a7a/lab6/StringList/StringList/StringList.h#L53-L54

• [ ] Константный итератор - это не const CIterator, а итератор, при разыменовании возвращающий констатнтую ссылку. Сам итератор при этом можно инкрементировать и декрементировать как угодно

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/b8668444aa95ef37f09f618aa5e3b9da0a379a7a/lab6/StringList/StringList/StringList.cpp#L91-L103

• [ ] Почему бы здесь не воспользоваться конструктором копирования?

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/b8668444aa95ef37f09f618aa5e3b9da0a379a7a/lab6/StringList/StringList/StringList.cpp#L105-L117

• [ ] Программа упадёт из-за переполнения стеке, если в списке-приемнике перед вызовом оператора перемещающего присваивания будет содержаться полядка 20-30 тысяч элементом