结构体单链表代码故障排查：为何这段代码无法正常运行？

咱们来一步步拆解这段代码里的问题，一眼就能看到几个致命的错误：

• 链表头被直接覆盖，原有数据完全丢失
  在printall()函数的第一行，你定义了一个静态结构体h，紧接着就把全局的head指针赋值为&h——这直接把filllist()里刚构建好的链表头给彻底覆盖了！原来由a、b、c、d组成的链表直接被丢弃，现在head指向的是一个未初始化的空结构体，自然没法遍历到任何有效数据。

• for循环存在语法错误+遍历逻辑完全错误
  代码里的for(int i = 0; i < 5; i...是不完整的语法，循环的增量部分和循环体都缺失了。更关键的是，用固定次数的for循环遍历链表是完全错误的思路：链表的正确遍历方式应该是借助临时指针从head出发，一直走到next为NULL为止，而不是假设链表有固定的5个节点——这既容易越界，也适配不了链表长度变化的情况。

• 未初始化的结构体带来的未定义行为
  printall()里的静态结构体h没有对任何成员（key、data、next）赋值，它的内存里都是垃圾值，就算你没有覆盖head，基于这个结构体的操作也会触发未定义行为，可能导致程序崩溃或者输出乱码。

修复后的printall()示例 #

void printall() {
    // 用临时指针遍历，不要修改原head指针
    struct item *current = head;
    while (current != NULL) {
        printf("Key: %d, Data: %d\n", current->key, current->data);
        current = current->next;
    }
}

内容的提问来源于stack exchange，提问作者e t