运行程序时出现分段错误（核心转储）

Question

#include<stdio.h> 
#include<string.h> 
#include<malloc.h> 
//#include<conio.h> 
struct list 
{ 
char *value; 
struct list *link; 
}; 
struct list *arr[12];int val; 
int hf(char *item) 
{ 
int sum,i=0; 
while(item[i]!='\0') 
{ 
    sum+=item[i]; 
    i++; 
} 
return sum%12; 
} 
void insert(struct list ** arr,char *item,int val) 
{ 
struct list *temp,*r; 
r=*arr; 
    temp=(struct list *)malloc(sizeof(struct list)); 
strcpy((temp->value),item); 
    if(strcmp((r->value),NULL)) 
    { 
     strcpy((r->value),(temp->value)); 
     (r->link)=NULL; 
    } 
    else 
    { 
     while(r->link!=NULL) 
     r=r->link; 
     r->link=temp; 
     r=r->link; 
     strcpy((r->value),(temp->value)); 
     r->link=NULL; 

    } 
*arr=r; 

} 
void main() 
{ 
    struct list *li[12];int i=0; 
    for(i=0;i<12;i++) 
    { 
     li[i]=NULL; 
    } 
    char *item;int ret; 
    strcpy(item,"Steve"); 
    ret=hf(item); 
    insert(&li[ret],item,ret); 
    strcpy(item,"raj"); 
    ret=hf(item); 
    insert(&li[ret],item,ret); 
    strcpy(item,"Notes"); 
    ret=hf(item); 
    insert(&li[ret],item,ret); 
} 

上面的程序是实现链表和数组，试图插入字符串 作为值。当我试图运行的程序，没有任何错误，但它告诉分段错误（核心转储） 所以请说明理由

Answer 1

代码

char *item;int ret; 
strcpy(item,"Steve"); 

尝试"Steve"复制字符串字面到未初始化的指针。您需要为item分配内存。这样做的最简单方法是硬编码合适尺寸的堆栈缓冲区

char item[50]; 

你也有内部insert类似的问题。你可以以同样的方式

struct list 
{ 
    char value[50]; 
    struct list *link; 
}; 

解决这个问题，或者当您释放该列表节点，你可以动态地分配缓冲区的正确大小内insert

temp->value = malloc(strlen(item) + 1); 
if (temp->value == NULL) { 
    /* handle oom error */ 
} 
strcpy(temp->value, item); 

在这后一种方法，确保free(node->value)。还要注意，当前程序中没有释放所有动态分配的内存，这意味着您泄漏了使用malloc分配的所有内存。

。在你的代码多了一个错误 - insert假定arr是指向一个有效的list*但它始终是NULL。您需要更新main或insert这里的假设。

Answer 2

变化以下

在插入件（）函数改变，如果环路

if(r==NULL){ 
    r = temp; 
} 

变化的结构。变量项目改变结构的大小以适合您的需要

struct list 
{ 
char value[25]; 
struct list *link; 
}; 

更改为

焦炭项目[25]。

编辑： 没有必要强制转换的malloc