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

artyRay12 / OOP

0 stars 0 forks source link

Замечания по программе CStringList #16

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

https://github.com/artyRay12/OOP/blob/cf46ce8cb692832a86ced75432d5181181ab6846/lab6/StringList/CStringList.h#L77

[ ] Поля класса должны иметь префикс m_

alexey-malov commented 4 years ago

https://github.com/artyRay12/OOP/blob/cf46ce8cb692832a86ced75432d5181181ab6846/lab6/StringList/CStringList.cpp#L19-L23

[ ] Здесь происходит копирование данных. Перемещение следует реализовать за O(1)

alexey-malov commented 4 years ago

https://github.com/artyRay12/OOP/blob/cf46ce8cb692832a86ced75432d5181181ab6846/lab6/StringList/CStringList.cpp#L25-L35

[ ] Что это за конструктор? Для чего он?
[ ] Зачем происходит копирование данных во временный список?

alexey-malov commented 4 years ago

https://github.com/artyRay12/OOP/blob/cf46ce8cb692832a86ced75432d5181181ab6846/lab6/StringList/CStringList.cpp#L52-L57

[ ] Тут, похоже, сторожевая нода теряется

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

https://github.com/artyRay12/OOP/blob/cf46ce8cb692832a86ced75432d5181181ab6846/lab6/StringList/CStringList.cpp#L214-L217 https://github.com/artyRay12/OOP/blob/cf46ce8cb692832a86ced75432d5181181ab6846/lab6/StringList/CStringList.cpp#L224-L227

[ ] Один из этих методов реализован неверно