c/c++语言开发共享为什么这个C链表程序会出现“分段错误”？

一个明显的问题就在这里：

 FILE * file = NULL; fopen(fname, "r"); 

为了使fopen能够完成很多工作，你需要将fopen的结果赋给你的FILE * ：

 file = fopen(fname, "r"); 

编辑：由于您在C中工作，因此无法通过引用传递指针。 作为替代方法，您可以将指针传递给指针：

 int lcreate(char *fname, element **list) { // ... *list = malloc(sizeof(element)); (*list)->next = null; (*list)->val = c; // ... } 

基本上， lcreate所有代码都需要引用*list而不仅仅是list 。 或者，您可以将指向现有列表的指针作为输入，并返回指向列表的指针，因此在main您可以使用以下内容： list = lcreate("list.txt", list);

file为NULL ，您永远不会为其分配文件句柄。

在main函数中，您还将按值传递给lcreate 。 在lcreate()函数中，您将覆盖list的本地副本，而不是更改main函数中list的值。 由于list初始化为NULL ，因此在调用list->val 。

是的 – 其他人对FILE指针的说法，并且按值传递而不是引用lcreate() ，是真的。

你也没有从lcreate()返回列表的大小 – 你应该通过返回值或指针参数返回它。

您试图在main()函数中迭代列表4次，但列表中可能少于4个项目。 如果list为NULL，最终printf()将导致分段错误。

如果在进行这些更改后仍然存在问题，我建议您在代码中添加跟踪，以确定何时发生分段错误。

更新：

还要记得在遍历列表后释放你已经分配的内存，否则你最终会出现内存泄漏（虽然实际上这对你来说不是一个问题，因为程序结束了，但释放内存是进入的好习惯）。

我也可以看到另外一个问题。 在lcreate()的while语句中，if语句malloc的true子句是一些内存并将其分配给list但是elem没有更新。

 while ((c = getc(file)) != EOF) { if(list == NULL) { list = (element*)malloc(sizeof(element)); if(list == NULL) { return 0; } list->val = c; } else { 

下一次通过while循环list不会是非null但是elem仍然是null所以elem-> next的赋值会尝试依赖空指针，从而导致分段错误（顺便说一句，这意味着你试图访问内存，尚未分配给您的流程）： –

 else { elem->next=(element*)malloc(sizeof(element)); 

正如其他人指出的那样，你也没有将list返回到main，所以当你点击printf（）循环时它仍然是NULL。

最后，在查看这些问题时，调试器是您的朋友。 您将确切地看到哪条线触发了seg故障以及变量的状态。

通过检查非null pinter来检查malloc是否成功会很好。 此外，您可能希望在while之外分配head / first链接，以避免每次在while循环中对头进行空检查。 当然，这些都是优化，万一你的链表变得非常大！