在链表的第 n 个位置插入一个节点

Question

#include<iostream>
using namespace std;

struct node {
    int data;
    node *link;
};
node *head = NULL;

void insert(int data, int n)
{
    node *temp = new node();
    temp->data = data;

    if (n == 1)
    {
        temp->link = head;
        head = temp;
    }

    else 
    {
        node* ptr = head;
        for (int i = 1; i <= n - 1; i++)
            ptr = ptr->link;
        temp->link = ptr->link;
        ptr->link = temp;
    }
}

void print()
{
    cout << "list is: ";
    node *temp = head;
    while (temp != NULL)
    {
        cout << temp->data << " ";
        temp = temp->link;
    }
    cout << endl;
}

int main()
{
    insert(2, 1);
    insert(3, 2);
    insert(4, 3);
    insert(5, 4);
    insert(6, 5);

    print();
    return 0;
}

这是在第 n 个位置实现插入链表的代码。数据和位置是从主位置传递过来的。

我不知道我可能犯了什么错误，它与for loop 有关。

它没有执行，但是如果我进行以下更改：

for(int i=0;i<n-2;i++)

效果很好。

Answer 1

首先insert(2,1) 工作正常。所以你有这样的链表

(2)->NULL
 |
head

在第二个插入中，让我们跟随代码，

1. else 
2. {
3.    node* ptr = head;
4.    for (int i = 1; i <= n - 1; i++)
5.        ptr = ptr->link;
6.    temp->link = ptr->link;
7.    ptr->link = temp;
8. }

第 3 行，ptr 指向 head。 n 是 2

(2)->NULL
 |
head
 |
ptr

第 4 行，1 <= (2-1) 是 true，因为 1 == 1，所以 for 循环运行一次

第5行，ptr移动了一步，所以它指向NULL

(2)->NULL
 |    |
head  |
      |
     ptr

第 6 行，ptr->link 被调用，即NULL->link。所以它在这里崩溃了。

---

当你做for(int i=0;i<n-2;i++)时，n是2，所以0 < (2-2)是false，所以它运行良好。注意：仅当像您的示例一样按顺序进行插入调用时才有效。如果以错误的顺序调用它们，它将不起作用。

将第 6 行更改为 temp->link = ptr;，也应该在不更改循环的情况下工作。

Answer 2

“在链表的第n个位置插入一个节点”：

使用std::list 而不是自己滚动。然后使用std::list::insert。

还有；考虑只使用std:: vector。列表是一种在现代 CPU 上使用的可怕（指针追逐、缓存未命中诱导）数据结构。 std::vector 几乎总能胜过它（无论您的教科书如何评价理论性能）。