69,371
社区成员
发帖
与我相关
我的任务
分享
#include<stdio.h>
#include<stdlib.h>
typedef int datatype;
typedef struct node
{
datatype data;
struct node *next;
}linklist;
void creatlist1(linklist *head,int n)
{
linklist *p,*s;
int i;
head->next=NULL;
s=head;
for(i=0;i<n;i++)
{
p=(linklist*)malloc(sizeof(linklist));
scanf("%d",&p->data);
s->next=p;
s=p;
}
s->next = NULL; //last node's next pointer NULL
}
void displaylist(linklist *head)
{
linklist *p;
//int i=0;
p=head->next;
if(p==NULL)
{
printf("这是个空指针\n");
return; //exit from displylist
}
while(p)
{
printf("%d ",p->data);
p=p->next;
}
}
int main()
{
linklist *head;
head=(linklist*)malloc(sizeof(linklist));
creatlist1(head,5); //创建一个链式列表
displaylist(head); //输出列表中的元素
return 0;
}
while (p)
{
printf("%d ", p->data);
p = p->next;
}
在遍历链表的时候,最后一个元素时,理论上p->next应该指向空指针,但是由于createlist1()函数的for循环中分配内存时,并没有对p->next值进行显式地赋空指针值,导致最后一个元素的p->next不是空指针而是一个随机值,在我调试的VS2015中,这个值是0xcdcdcdcd。这样就导致在displaylist()函数的while()语句中会访问0xcdcdcdcd,而这个地址是不能访问的,就会直接报错。
解决的方法就是在createlist1()函数的for循环中
p = (linklist*)malloc(sizeof(linklist));
p->next = nullptr;