Question

我正在用C编写一个基本的链表程序，并且在删除时遇到了一些麻烦。这就是我所拥有的：

#include <stdio.h>

struct node * delete(struct node * head, struct node * toDelete);
void print(struct node * head);

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

int main(int argc, const char * argv[]) {

    struct node node1, node2, node3;
    struct node *head = &node1;

    node1.value = 1;
    node1.next = &node2;

    node2.value = 2;
    node2.next = &node3;

    node3.value = 3;
    node3.next = (struct node *) 0;

    print(head);

    delete(head, &node3);

    print(head);

    return 0;
}

struct node * delete(struct node * head, struct node * toDelete) {
    //if to delete is head
    if (head == toDelete) {
        head = head->next;

    } else {
        //find node preceding node to delete
        struct node *current = head;
        while (current->next != toDelete) {
            current = current->next;
        }
        current = current->next->next;
    }
    return head;
}

void print(struct node * head) {
    struct node *current = head;

    while (current != (struct node *) 0) {
        printf("%i\n", current->value);
        current = current->next;
    }
}

问题＃1： 所以我试着写：

delete(head, node3);

但xCode希望我添加“＆amp;”在“node3”前面。通常情况下，当我定义一个函数来获取指针时，我需要传入内存地址吗？

问题＃2：

我的打印功能用于打印3个节点的值。在调用delete并尝试删除node3之后，它仍然打印出3个节点。我不确定我哪里出错了。我找到了我要删除的节点之前的节点，并在节点之后设置了它的下一个指针节点（非正式地：node.next = node.next.next）。

有什么想法吗？

感谢您的帮助， bclayman

Answer 1

but xCode wanted me to add "&" in front of "node3". Is it generally true that
when I define a function to take a pointer, I need to pass in the memory 
address?

是的，如果你声明函数采用指针，你必须传递一个指针。

同样，当从链表中删除值时，您将要更改

current->next = current->next->next

Answer 2

你应该传递它&node3。要删除，请更改您的代码 current = current->next->next;至current->next = current->next->next;

Answer 3

尝试将current = current->next->next;更改为current->next=current->next->next。如果它不起作用，请告诉我。

Answer 4

通常情况下，当我定义一个函数来获取指针时，我需要传入内存地址吗？

是的，xCode是对的。 node3是struct node，但您的函数delete将struct node *作为第二个参数，因此您必须将指针传递给node3，而不是变量本身。

在调用delete并尝试删除node3之后，它仍打印出3个节点。

因为您没有更改next的值。另外，为了保护内存，请不要忘记检查指针是否为NULL：

while ((current->next != toDelete) && (current->next != NULL)) {
    current = current->next;
}
if (current->next != NULL)
    current->next = current->next->next;

C中的基本链接列表

4 个答案: