我当前实现的双链表类是否需要重新构造迭代器end（）功能？

Question

恐怕我当前的双向链表实现不能适当地容纳迭代器。我的迭代器end（）函数需要开始指向尾后的节点，以便通过列表递减。当然，因为我有我的列表当前设计的尾-

我写这篇文章时是否理解我的第一个指针和尾部指针是“虚拟节点”？

我尝试过制作第三个指针，但考虑到与我的pushBack函数非常相似，所有东西都是为了在我的doublyLinkedList类中将first和tail构造为空ptr而设计的，因此我认为找不到一个可行的实现。

我一直在试图弄清楚如何实现一个额外的虚拟节点——任何解决方案都可以。

我将只显示我认为可能相关的代码部分。

struct node
{
    T data;
    node<T> * next;
    node<T> * prev;

    node() {
        next = nullptr;
        prev = nullptr;
    };
    node(T data, node<T> * next = nullptr, node<T> * prev = nullptr) 
        : data(data), next(next), prev(prev) {};
};

template <class T>
class LinkedList
{
private:
    node<T> * first;
    node<T> * tail;
    int size;
public:
    LinkedList();
    LinkedList(const LinkedList&);
    void copy(const LinkedList&);
    void operator=(const LinkedList&);
    //void insert_end(T dat);
    void pushBack(T data);
    void pushFront(T data);
    void display();
    void displayRev();
    int getSize();

    class Iterator
    {
    private:
        node<T> * itr;
    public:

        Iterator(node<T> * itr = nullptr) : itr(itr){};

        Iterator& operator++();
        Iterator operator++(int);
        Iterator& operator--();
        Iterator operator--(int);

        T operator*();

        bool operator!=(LinkedList<T>::Iterator other);
        bool operator==(LinkedList<T>::Iterator & other);

        ~Iterator() {}
    };
/*BEGIN AND END*/
    Iterator begin() { return Iterator(first); }
    Iterator end(tail->next) 
    {
        return Iterator(tail->next); 
    }
    ~LinkedList();
};

定义

template<class T>
typename LinkedList<T>::Iterator& LinkedList<T>::Iterator::operator--()
{

    itr = itr->prev;
    return *this;
}

template<class T>
typename LinkedList<T>::Iterator LinkedList<T>::Iterator::operator--(int)
{
    auto tmp = *this;

    itr = itr->prev;
    return tmp;
}

//constructor
template <class T>
LinkedList<T>::LinkedList()
{
    first = tail = nullptr;
    size = 0;
    /*node<T>* first = new node<T>;
    node<T>* tail = new node<T>;
    first->next = tail;
    tail->prev = first;*/

}

template<class T>
void LinkedList<T>::pushBack(T data)
{

    if (first == nullptr)
    {
        first = tail = new node<T>(data);
        size++;
    }
    else
    {
        if (tail == nullptr)
        {
            tail = new node<T>(data);
            first->next = tail;
            tail->prev = first;
            size++;
        }
        else
        {
            node<T> *prevTail = tail;
            node<T> *newTail = new node<T>(data, nullptr, prevTail);
            prevTail->next = newTail;
            tail = newTail;
            size++;
        }
    }

}

主要的

#include "pch.h"
#include "LinkedList.h"
#include <iostream>
typedef LinkedList<int>::Iterator iter;
int main()
{
    LinkedList<int> l1;
    l1.pushBack(10);
    l1.pushBack(20);
    l1.pushBack(30);

    for (auto itr = l1.begin(); itr != l1.end(); itr++)
    {
        std::cout << (*itr) << "  ";
    }
}

在过去的三个小时里，我一直在做这件事，但毫无结果，我又回到了我的起点。如果有人能给我指路，我将万分感激。

谢谢你。

Answer 1

我可以看到两种选择。两者都避免使用虚拟节点，尽管这也是一种可行的方法。

构造结束迭代器很困难，因为自然的方法是实例化迭代器{}，这意味着迭代器：：itr节点指针将为空。但是，在这种情况下，减量运算符将无法实现。作为解决方案，这些操作可以删除。

template <class T>
class LinkedList
{
    /* ... */
    class Iterator
    {
    private:
        node<T> * itr;
    public:

        Iterator(node<T> * itr = nullptr) : itr(itr){};

        Iterator& operator++();
        Iterator operator++(int);

        T operator*();

        bool operator!=(LinkedList<T>::Iterator other);
        bool operator==(LinkedList<T>::Iterator & other);
    };
    Iterator begin() { return Iterator{first}; }    
    Iterator end() { return Iterator{}; }  
};

我认为您可以向迭代器添加一个prev指针来保留双向操作。

template <class T>
class LinkedList
{
    /* ... */
    class Iterator
    {
    private:
        node<T> * current;  // null if and only if this is end iterator.
        node<T> * prev;     // may be null (at head of list)
    public:
        Iterator() : current{nullptr}, prev{nullptr} {}
        Iterator(node<T> * current) : current{current}, prev{current ? current->prev : nullptr} {}
        Iterator(node<T> * current, node<T>* prev) : current{current}, prev{prev} {}

        Iterator& operator++();    // implemented using `curr`
        Iterator operator++(int);  // implemented using `curr`
        Iterator& operator--();    // implemented using `prev`
        Iterator operator--(int);  // implemented using `prev`

        T operator*();  // implemented using `curr`

        bool operator!=(LinkedList<T>::Iterator other);
        bool operator==(LinkedList<T>::Iterator & other);
    };

    Iterator begin() { return Iterator{first}; }    
    Iterator end() { return Iterator{nullptr, tail}; }
};