Замечания по CStringList

[alexey-malov]

• [ ] Не делайте флаг isReversed полем класса iterator, прямой и реверсированный итераторы должны быть разными типами. Посмотрите в сторону std::reverse_iterator. Как еще один вариант - сделать флаг параметром шаблона.

[alexey-malov]

• [ ] Нет возможности выполнить декремент end-итератора

[alexey-malov]

CMyStringList & CMyStringList::operator=(CMyStringList & other)
{
    if (this != &other) // защита от неправильного самоприсваивания
    {
        CMyStringList tmp;
        for (auto const & elem : other)
        {
            tmp.PushBack(elem);
        }
        std::swap(m_firstNode, tmp.m_firstNode);
        std::swap(m_lastNode, tmp.m_lastNode);
        m_size = tmp.m_size;
    }
    // по соглашению всегда возвращаем *this
    return *this;
}

• [ ] Это не оператор копирующего присваивание, который ищет преподаватель. С конструктором копирования та же фигня

void DoSomething(const CMyStringList & lst)
{
    CMyStringList lst2;
    if (true)
    {
        lst2 = lst;
    }
}

Код не компилируется

[alexey-malov]

• [ ] Реализовать оператор перемещающего присваивания и перемещающий конструктор

[alexey-malov]

int main()
{
    using namespace std;
    CMyStringList lst;
    lst.PushBack("one");
    lst.PushBack("two");
    lst.PushBack("three");
    lst.PushBack("four");
    lst.PushBack("two");
    lst.Erase(++lst.begin());
    for (auto & i : lst)
    {
        cout << i << endl;
    }
    return 0;
}

• [ ] программа упала и не отжалась