从链表中删除项目时出现分段错误

Question

我正在尝试创建程序，您可以在其中输入“+word”并添加单词，当您输入“-word”时，它会将单词从链接列表中取出。

插入单词对我来说很好，但删除它会导致分段错误。我不确定问题出在哪里。另外，有没有办法可以提示分段错误在哪里？

void
remove_from_list(struct linked_list *list, char *data)
{
    struct node *current_node = list->head;
    struct node *previous_node = NULL;

    while (current_node != NULL) {
        if (current_node->data == data) {
            break;
        }
        previous_node = current_node;
        current_node = current_node->next;
    }
    if (previous_node == NULL) {
        list->head = list->head->next;
    } else {
        previous_node->next = current_node->next;
    }
    free(current_node);
    if (list->tail == current_node)
        list->tail = previous_node;
}

int
main(void)
{
    struct linked_list list = { .head = NULL, .tail = NULL };
    char word[50];

    do {
        printf("Enter string: ");
        fgets(word, 50, stdin);
        if (word[0] == '+')
            add_to_list(&list, word);
        else if (word[0] == '-')
            remove_from_list(&list, word);
    } while (word[0] != '\n');

    print_list_rec(&list);
    free_list(&list);
    return 0;
}

Answer 1

您循环到链表的末尾，然后继续在此处取消引用 NULL 指针

} else {
    previous_node->next = current_node->next;
}

发生这种情况是因为您的比较实际上并未比较数据；

 if (current_node->data == data) {

你永远不会从那个 if 语句中得到真正的结果。

如果要比较字符串，请使用 strcmp()。

Answer 2

您遇到段错误的主要原因是您在尝试删除时没有处理列表中没有数据的情况。

if (previous_node == NULL) { 
    list->head = list->head->next;
} else { // ------------------------- If at the end of the list you go in here
    previous_node->next = current_node->next;
}

current_node 是 Null 所以 current_node->next 段错误。

您走到列表末尾的原因是您没有正确比较字符串的数据。使用strcmp() 像@this 建议正确比较。但是您应该处理列表中没有数据的情况。

---

您可以在 while 循环和第一个 if 语句之间添加一个检查，这将处理一个空列表和不在列表中的数据 -

if(current_node == NULL) // Empty list or wasn't found
    return;

---

另一个说明：

你先释放current_node，然后再检查它是否是尾巴。颠倒这个顺序。

if (list->tail == current_node)
    list->tail = previous_node;
free(current_node);

Answer 3

在其他人说的之上，如果列表为空，则此代码将导致分段：

if (previous_node == NULL) {
    list->head = list->head->next;
}

Answer 4

如果没有插入函数的代码，很难说它出了什么问题，因为删除的步骤很好。但是，插入可能有问题，因此这不起作用。

但是您的代码中有一个问题，如果您尝试删除一个不存在的节点，您最终仍然会删除最后一个节点。您需要在 while 循环中断之前设置一个标志，然后仅在标志为 true 时删除该节点。

Answer 5

函数应该如下所示

void
remove_from_list( struct linked_list *list, char *data )
{
    struct node *current_node = list->head;
    struct node *previous_node = NULL;

    while ( current_node != NULL && strcmp( current_node->data, data ) != 0 ) 
    {
        previous_node = current_node;
        current_node = current_node->next;
    }

    if ( current_node != NULL ) 
    {
        if ( previous_node != NULL ) 
            previous_node->next = current_node->next;
        else
            head = head->next; 

        if ( list->tail == current_node )
            list->tail = previous_node;

        free( current_node->data );
        free( current_node );
    }
}

我也会存储不带 + 或 - 的字符串。在这种情况下，main 中的 if 语句看起来像

    if ( word[0] == '+' )
        add_to_list( &list, word + 1 );
    else if ( word[0] == '-' )
        remove_from_list( &list, word + 1 );

否则，您将永远找不到添加了加号的字符串以将其从列表中删除。