Question

我试图在c中创建一个链接列表可以接受一个字符串作为数据，我已经实现了一个push，free和pop函数但我的问题是，在pop函数中它不会将该名称识别为成员试着释放它。

typedef struct list
{
    char* name;
    struct list* next;
} node;


void free_list(node*head)
{
    if(head==NULL){
        return;
    }
    node* temp=NULL;
    while (head!= NULL)
    {
        temp=head->next;
        free(head->name);
        free(head);
        head=temp;
    }
    head=NULL;
}

/*add elements to the front of the list*/
void push_list(node **head, char* name)
{
    node *temp=malloc(sizeof(node));
    if(temp==NULL)
    {
        fprintf(stderr,"Memory allocation failed!");
        exit(EXIT_FAILURE);
    }
    temp->name=strdup(name);
    temp->next=*head;
    *head=temp;
}


void pop_list(node ** head) {
    node * next_node = NULL;
    if (*head == NULL) {
        return;
    }
    next_node = (*head)->next;
    free(*head->name); //this line generating error
    free(*head);
    *head = next_node;
}

bool empty_list(node *head){
    return head==NULL;
}

我猜这与我使用指针指针错误有关？有点卡住了

Answer 1

您需要在(*head)周围加上括号来生成语句free((*head)->name);。 free(*head->name)被解释为free(*(head->name))，这就是编译器对你大吼大叫的原因。

引用this Stack Overflow帖子，原因是后缀运算符（->）的优先级高于一元运算符(*)。

错误：不是结构或联合的成员，导致内存泄漏

1 个答案: