如何为自定义类实现迭代器？

Question

对于作业，我们必须创建自定义列表类。其中一个功能是将元素插入列表的中间。到目前为止，

 template<class T>
 class MyList : public list<T> {
 public:
   void addInMiddle(const T&);
   void insertInSortedOrder(const T&);
   void printList();
 };


template<class T>
void MyList<T>::addInMiddle(const T& x) {

 list<T>::iterator it = this->begin();

 int location = (this->size()) / 2;     //where we want to insert the new element
 for (int i = 0; i < location; i++) {
     it++;
 }
 this->insert(it,x);
}


 int main()
{
MyList<int> list1;
list1.push_back(1);
list1.push_back(2);
list1.push_back(3);
list1.push_back(4);
list1.addInMiddle(5);
list1.printList();
return 0;
}

程序已编译并显示 1,2,3,4。它实际上应该显示 1,2,5,3,4。我认为迭代器不起作用。有什么想法吗？

Answer 1

一旦我在 list::iterator 前面添加 typename ，您的代码就可以正常工作，因为迭代器的类型取决于 的类型>T 模板参数，例如：

typename list<T>::iterator it = this->begin();

在线演示

参见我必须在哪里以及为什么必须放置“template”和“typename”关键字？

或者，您可以（并且应该）只使用 < code>auto 改为：

auto it = this->begin();

在线演示

---

话虽这么说，标准 C++ 容器并不打算被继承来自（无虚拟析构函数等）。您应该使用封装而不是继承，例如：

template<class T>
class MyList {
private:
    list<T> m_list;
public:
    void addToBack(const T&);
    void addInMiddle(const T&);
    void insertInSortedOrder(const T&);
    void printList();
};

template<class T>
void MyList<T>::addToBack(const T& x) {
    m_list.push_back(x);
}

template<class T>
void MyList<T>::addInMiddle(const T& x) {

    list<T>::iterator it = m_list.begin();
    // or: auto it = m_list.begin();

    int location = m_list.size() / 2;     //where we want to insert the new element
    for (int i = 0; i < location; i++) {
        it++;
    }

    m_list.insert(it, x);
}

int main()
{
    MyList<int> list1;
    list1.addToBack(1);
    list1.addToBack(2);
    list1.addToBack(3);
    list1.addToBack(4);
    list1.addInMiddle(5);
    list1.printList();
    return 0;
}

在线演示