循环链表插入两个元素后只有一个元素

Question

我正在尝试实现一个循环链表，但它没有像我预期的那样工作。即使我用insertAfter 插入两个元素，printList 也只打印一个节点。这是一个最小的例子：

#include <stdio.h>
#include <stdlib.h>
#include <limits.h>

struct dnode_elm {
    int item;
    struct dnode_elm *next, *prev;
};

struct dnode_elm *
insertAfter(struct dnode_elm *a, int value) {
    struct dnode_elm *v= malloc(sizeof(struct dnode_elm));
    v->item=value;
    a->next=v;
    v->prev=a;
    v->next=a->next;
    a->next->prev=v;
    return v;
}

void
printList(struct dnode_elm *h) {
    while (h != h->next) {
        h = h->next;
        printf("%d --> ",h->item);
    }
}

int
main(void) {
    struct dnode_elm h = { INT_MAX, &h, &h };
    insertAfter(&h, 1);
    insertAfter(&h, 2);
    printList(&h);
}

Answer 1

你的插入逻辑错了。

a->next=v;
v->prev=a;
v->next=a->next;
a->next->prev=v;

在这段代码之后，v->next 等于v，这可能不是你想要的。

一种可能的解决方法是先分配v 的指针，然后再修复v 周围的节点。

v->prev = a;
v->next = a->next;
v->next->prev = v;
v->prev->next = v;

举例说明：

---

在a 之后插入v。

---

设置v->next 和v->prev。

---

设置v->next->prev 和v->prev->next。

---

不过，您也可以重新排列代码中的分配，方法是将第一个分配移动到最后一个分配。

v->prev=a;
v->next=a->next;
a->next->prev=v;
a->next=v;

这允许分配给a->next->prev 的工作如您所愿。

---

此外，您的打印逻辑有缺陷。您需要记住初始列表指针是什么，以便正确检测何时到达末尾。

void *start = h;
while (start != h->next) {
    h = h->next;
    printf("%d --> ",h->item);
}