c：链表：如何编写节点交换函数？

Question

这是我的功能：

void switchnodes(NODE **A)
{
    if((*A)->next)
    {
        NODE *first = pop(A);
        NODE *second = pop(A);
        push(A,first);
        push(A,second);
    }
}

推送和弹出：

void push(NODE **top,NODE *nnew)
{
    nnew->next=*top;
    *top=nnew;
}
NODE * pop(NODE **top)
{
    NODE *remove = *top;
    *top=(*top)->next;
    remove->next=0;
    return remove;
}

我的困惑：

节点：

typedef struct node {
    int value;
    struct node *next;
} NODE;

NODE * top 我的链表持有值 1...10 NODE * holder是另一个指针，分别指向链表的第5个和第4个节点

switchnodes(&holder); holder 指向第 5 个节点。第 6 个节点丢失 switchnodes(&(holder->next));holder 指向第四个节点。第 5 和第 6 个节点交换成功。

为什么会这样？

以及如何编写这个函数，这样我就不必传递指针->next？我不能在所有情况下都这样做，比如我需要交换的顶级节点。

Answer 1

这就是您的列表的工作方式：您只能通过next 指针交换节点； holder 是列表外的指针。它可能会更新，但列表不会知道。

说holder 指向第四个节点。你交换它，它现在指向新的第 4 个节点，以前的第 5 个节点。但重要的是：列表第三个节点的next 指针仍然指向第四个节点——交换前的第四个节点。

您可以通过为每个节点添加一个 haed 和一个 tail 以及一个额外的 prev 指针来创建双向列表。然后，您可以使用该信息来更新交换节点的邻居（如果有）。

Answer 2

您必须在 switchnodes 函数中更新 first 的“next”。这是一个工作函数：

void switchnodes(NODE **A)
{
    if((*A)->next)
    {
        NODE *first = pop(A);
        NODE *second = pop(A);  
        push(A,first);
        first->next = second->next; //the missing line
        push(A,second);
    }
}