释放一个链表

Question

我有问题释放我的链表正确。我真的不知道，如何制作头部和电流…

typedef struct list{
   int info;
   struct list *link;
} Node;
Node *_getnode (){
    return((Node *) malloc (sizeof (Node)));
}
int main() {
    Node *s = _getnode();
    s -> link = NULL;
    Node *t = s;
    int c;
    while (1) {
        printf("nt Enter number : ");
        scanf ("%d", &s -> info);   
        printf ("nt Continue? ");
        __fpurge (stdin);
        c = getchar ();
        if (c == 'N' || c == 'n'){
            s -> link = NULL;
            break;
        }
        s -> link = _getnode ();
        s = s -> link;
    }
    s = t;
    free(s);
    free(t);
    putchar('n');
    return (0);
}

当我只输入一个数字时，valgrind说，没有内存泄漏。但除此之外还有。我明白，我必须释放的不仅仅是第一个节点，但我不知道如何。

Answer 1

首先，

free(s);
free(t);

是多余的，因为您已经在上一行中分配了s = t;。

发生内存泄漏是因为您只在头指针上使用free，而不是链表中的所有节点。因此，理想情况下，应该循环遍历所有节点并释放分配给每个节点的内存，您的代码应该是这样的:

while(s != NULL)
{
     t = s->link;
     free(s);
     s = t;
}

Answer 2

试一下

void free_list(Node *head)
{
    Node *s;
    Node *n;
    if (head == NULL) /* passing `NULL' should be ok */
        return;
    for (s = head ; s != NULL ; s = n)
    {
        n = s->link;
        free(s);
    }
 }

你可以看到你创建列表的方式有一个明显的缺陷

Answer 3

首先，请初始化节点，以便程序能够正确判断列表的结束。

Node *_getnode (){
    Node* buffer = malloc (sizeof (Node));
    if (buffer == NULL) exit(1);
    buffer->info = 0; /* not important */
    buffer->link = NULL; /* this is important */
    return buffer;
}

然后，逐个释放节点。

void free_nodes (Node* head){
    while(head != NULL){
        Node* next = head->link;
        free(head);
        head = next;
    }
}

未测试，通过t到这个free_nodes应该工作