当我试图在双向循环链表之间输入一个节点时，它给出了错误的输出

Question

这里我创建了一个双向循环链表，并尝试在链表之间添加一个节点，但它给出了错误的输出

我先创建了一个双向循环链表，然后显示出来，然后在它们之间添加一个节点，然后再次显示出来

#include<iostream>
using namespace std;
class node                                  //node class
{
public:
    int data;
    node *next;
    node *prev;
    node(int a)
    {
        data=a;
        next=nullptr;
        prev=nullptr;
    }
};
class linkedlist                            //linkedlist class
{
    node *head,*tail;
public:
    linkedlist()
    {
        head=nullptr;
        tail=nullptr;
    }
    void addnode(int val)                   //creating node function
    {
        node *newnode;
        newnode=new node(val);
        if(head==0)
            head=tail=newnode;
        else
        {
            tail->next=newnode;
            head->prev=newnode;
            newnode->prev=tail;
            newnode->next=head;
            tail=newnode;
        }
    }
    void disp()                             //display function
    {
        node *temp=head;
        while(temp->next!=head)
        {
            cout<<temp->data<<"   ";
            temp=temp->next;
        }
        temp=temp->next;
        cout<<"   "<<temp->data;
    }
    void addin(int val,int pos)             //addin function
    {
        node *newnode=new node(val);
        node *temp=head;
        for(int i=0;i<pos;i++)
            {
                temp=temp->next;
            }
        newnode->next=temp->next;
        newnode->prev=temp;
        temp->next->prev=newnode;
        temp->next=newnode;
    }
};
int main()
{
    linkedlist l1;
    int s,val,val1,pos;
    cin>>s;
    for(int i=0;i<s;i++)
    {
        cin>>val;
        l1.addnode(val);
    }
    l1.disp();                              //display function
    cout<<"\n\n";
    cin>>pos;
    cin>>val1;
    l1.addin(val,pos);                      //calling addin function
    l1.disp();
}

输入

预期输出

1  2  3

1 2 0 3

当前输出

1  2  3   1

我不知道自己犯了什么错误，而且我是初学者，所以任何提示对我来说也太有帮助了

Answer 1

由于 disp() 函数中循环后的额外 temp=temp->next;，您正在打印头节点而不是尾节点。
您在 main() 函数的循环后添加 val 而不是 val1。

    void disp()                             //display function
    {
        node *temp=head;
        while(temp->next!=head)
        {
            cout<<temp->data<<"   ";
            temp=temp->next;
        }
        // remove this
        // temp=temp->next;
        cout<<"   "<<temp->data;
    }

    cin>>pos;
    cin>>val1;
    // add val1, not val
    //l1.addin(val,pos);                      //calling addin function
    l1.addin(val1,pos);                      //calling addin function
    l1.disp();

Answer 2

在主函数中您错误地添加了 val 而不是 val1

cin>>val1;
l1.addin(val1,pos);

然后在disp()函数中，去掉循环外的temp=temp->next

void disp()                             //display function
    {
        node *temp=head;
        while(temp->next!=head)
        {
            cout<<temp->data<<"  ";
            temp=temp->next;
        }
        // temp=temp->next;
        cout<<"  "<<temp->data;
    }

最后在 addin() 函数中，将 pos 更改为 pos-1

for(int i=0;i<pos-1;i++)
            {
                temp=temp->next;
            }

就这些了。

Answer 3

@MikeCAT 说对了，您正在使用 val1 并传递 val。但是，我认为 disp 中的 while 循环将跳过最后一个元素而不打印它。并且插件中的逻辑不会在新位置添加新元素。这是我的代码版本

#include<iostream>
using namespace std;
class node                                  //node class
{
public:
    int data;
    node *next;
    node *prev;
    node(int a)
    {
        data=a;
        next=nullptr;
        prev=nullptr;
    }
};
class linkedlist                            //linkedlist class
{
    node *head,*tail;
public:
    linkedlist()
    {
        head=nullptr;
        tail=nullptr;
    }
    void addnode(int val)                   //creating node function
    {
        node *newnode;
        newnode = new node(val);
        if(head == NULL)
            head = tail = newnode;
        else
        {
            tail->next = newnode;
            head->prev=newnode;
            newnode->prev=tail;
            newnode->next=head;
            tail=newnode;
        }
    }
    void disp()                             //display function
    {
        node *temp = head;
        if (temp == NULL) 
        {
            cout << "List is empty " << endl;
            return;
        }
        do
        {
            cout << temp->data << endl;
            temp = temp->next;
        } while (temp != head);

    }
    void addin(int val,int pos)             //addin function
    {
        node *newnode=new node(val);
        node *temp=head;
        for(int i=0;i<pos;i++)
        {
            temp=temp->next;
        }
        //newnode->next=temp->next;
        newnode->next=temp;
        newnode->prev=temp->prev;
        temp = temp->prev;
        temp->next = newnode;
    }
};
int main()
{
    linkedlist l1;
    int s,val,pos;
    cin>>s;
    cout << "val of size is " << s << endl;
    for(int i=0;i<s;i++)
    {
        cin>>val;
        l1.addnode(val);
    }
    l1.disp();                              //display function
    cout<<"\n\n";
    cin>>pos;
    cin>>val;
    l1.addin(val,pos);                      //calling addin function
    l1.disp();
}

当我试图在双向循环链表之间输入一个节点时，它给出了错误的输出

When I am trying to enter a node between doubly circular linked list it gives wrong output

c++

doubly-linked-list