按子字符串搜索链表并创建一个包含所有包含该子字符串的结构的新链表

Question

假设我有以下结构：

struct _node {
    int id;
    char *name;
    struct node *next;
};

我想编写一个函数，它遍历链表并查找名称成员包含某个子字符串的结构，并将这些结构添加到新的链表并返回新的链表。

我尝试使用 strstr 执行此操作，但在某些时候我遇到了无限循环，我无法弄清楚为什么会发生无限循环。

这是我的功能：

struct _node *containSubstr(struct _node *head, char needle[]) {
    if (head == NULL) {
        printf("BLOOP BLEEP BOOP BEEP.\n");
        exit(EXIT_FAILURE);
    }

    struct _node *curr = head;
    struct _node *returnHead = createEmptyNode();

    while (curr != NULL) {
        char haystack[strlen(curr->name)];
        strcpy(haystack, curr->name);

        strToLower(needle);
        strToLower(haystack);

        if (strstr(haystack, needle) != NULL) {
            // this is where I get the infinite loop
            append(returnHead, curr);
        }

        curr = curr->next;
    }

    return returnHead;
}

函数 append 和 createEmptyNode 已经过测试并且工作正常。

我多次检查了这个逻辑，我认为这必须有效。我用打印语句填充了我的代码，似乎在它找到包含子字符串的所有节点之后它不断重复最后一个节点并进入无限循环。

这是我的附加功能：

void append(struct _node *head, struct _node *newNode) {
    if (head == NULL) {
        printf("BLOOP BLEEP BOOP BEEP.\n");
        exit(EXIT_FAILURE);
    }

    struct _node *curr = head;

    if(curr == NULL) {
        head = newNode;
    }
    else {
        while(curr->next != NULL) {
            curr = curr->next;
        }
        curr->next = newNode;
    }
}

Answer 1

想象一下这种情况：您有一个列表 L，它在您的链表中只包含 2 个节点（A 和 B）。想象一下，两个节点 A 和 B 都包含您要查找的子字符串：

原始列表L：

A->B->NULL

所以在第一次迭代之后，您的新列表 L2 应该如下所示：

A->NULL

但是在您的附加函数中，您不会创建新节点的深层副本。因此，您的新列表 L2 如下所示：

Empty_node->A->B->NULL

在下一步中，您将移动到节点 B。所以你把你的L2 附加到那里B。第二次迭代后您的 L2 如下所示：

Empty_node->A->B->B（B指向自己）。

由于您不创建深层副本，因此您实际上一直在使用列表 L 并且当您将节点 B 附加到您认为的列表 L2 时 你实际上将 B 附加到 L，然后 B 指向它自己（append 函数中的curr->next = newNode;）。因此，在下一次迭代中，您再次询问 B 是否包含您要查找的字符串。

结论

创建新列表时需要创建深拷贝。

在当前设置中，您的 append 函数会修改原始列表。