我在挣扎。我已经得到了它来显示大部分列表,但其中一个1没有显示出来,我不能为我的生活弄清楚如何解决它。

这里是相关的代码,我想。

我的插入功能:

 template <class T>
 void DoublyLinkedList<T>::insert(T data)
 {
    DoublyLinkedList<T> *newNode, *tmp, *oneBefore;

    newNode = new DoublyLinkedList(data);

if (mNext == NULL)
    mNext = newNode;

else
{
   oneBefore = mNext;
   tmp = mNext;

    while (tmp != NULL && tmp->mData < data)
    {
        oneBefore = tmp;
        tmp = tmp->mNext;
    }

    if (tmp == mNext)
    {
        newNode->mNext = mNext;
        mNext = newNode;
    }
    else
    {
        oneBefore->mNext = newNode;
        newNode->mNext = tmp;
        newNode->mPrevious = oneBefore;
    }

   }
 }

我的displayBackwards功能:

 void displayBackward(DoublyLinkedList<int> *ptr)
 {
    DoublyLinkedList<int> *tmp;

    tmp = ptr;
    while (tmp != NULL)
    {
       cout << tmp->getData() << endl;
       tmp = tmp->getPrevious();
    }
 }

和我的主要功能的相关部分:

    DoublyLinkedList<int> *ptr, *head, *tail;

    ptr = new DoublyLinkedList<int>;

    cout << "Testing Insert\n";
    ptr->insert(1);
    ptr->insert(2);
    ptr->insert(3);
    ptr->insert(1);

    tail = ptr;
    while (tail->getNext() != NULL)
       tail = tail->getNext();

    cout << "\n\nTesting displayBackward\n";
    displayBackward(tail);

我目前的输出是:

 Testing displayBackward
 3
 2
 1
有帮助吗?

解决方案

这段代码是问题所在(在insert函数中)

    if (tmp == mNext)
    {
        newNode->mNext = mNext;
        mNext = newNode;
    }

你需要

    if (tmp == mNext)
    {
        newNode->mNext = mNext;
        mNext->mPrevious = newNode;
        mNext = newNode;
    }

您的原始代码将在某些情况下工作,除非您尝试插入小于或等于当前第一个节点中的数据的数据。

另外,我假设你的构造函数将mNext&mPrevious初始化为NULL。如果没有,你会有其他问题。

其他提示

 template <class T>
 void DoublyLinkedList<T>::insert(T data)
 {
    DoublyLinkedList<T> *newNode, *tmp, *oneBefore;

    newNode = new DoublyLinkedList(data);

if (mNext == NULL)
    mNext = newNode;

else

这不是连接新节点的后指针。

你的代码也可能有更多的错误。


编写双链表的一个简单方法是

  • 区分 名单节点 类型,以及
  • 使每个列表始终有一个虚拟节点,称为 头节点.

这样你就没有了 NULL 指针来处理。

它确实简化了事情。

许可以下: CC-BY-SA归因
不隶属于 StackOverflow
scroll top