请大家帮看看malloc出错的问题。

[byron1107]

写了个建表的东东。但一malloc完进行p->next = q;时就死掉了。不知道是为什么。
请大虾指点。谢谢。
typedef struct list *pointer;
typedef struct list
{
int data;
struct list *next;
} Klist;
typedef pointer plist;
plist createlist(plist L,int n)
{
plist p = L;
while(n > 0 )
{
plist q = (plist)malloc(sizeof(Klist));

p->next = q;
//运行到这儿就死掉了
q->next = NULL;
q->data = n;
p = p->next;

n--;
}

return L;
}

[yingge]

单这个函数没有什么问题啊，代入的时候L指向的空间初始化了没有？

[m_nNightmire]

如果传进来的L没有经过初始化，那的确会死掉

[byron1107]

plist L;
L->next = NULL;
L->data = 0;
straightlistsort(L);

调用时是这样的，但是不行。

[冷血孤剑]

不是初始化的问题，而是有没有确保指针指向的内存。
比如，不能这样定义：plist l，然后createlist(l,n)；
而要，定义Klist l，然后createlist(&l,n)；

[yingge]

plist L;
L->next = NULL;
L->data = 0;

不行的，你只申请了指针，没有申请指针指向的内存空间，应该

plist L;
L=(plist)malloc(sizeof(Klist));
L->next = NULL;
L->data = 0;

或者像楼上那样。

[小水]

plist L;
L->next = NULL;
L->data = 0;
straightlistsort(L);

L没有初始化就使用了。


Klist L;
L.next = NULL;
L.data = 0;
straightlistsort(&L);
createlist(&L, n);

或者：
plist L;
L=(plist)malloc(sizeof(Klist));
L->next = NULL;
L->data = 0;
straightlistsort(L);
createlist(L, n);

对楼上的总结

[byron1107]

谢谢楼上两位。
但这个问题解决了，却变成无限死循环了。。怎么会这样。。
请大虾再帮我看看。

plist createlist(plist L,int n)
{
plist p = L;
while(n > 0 )
{
plist q = (plist)malloc(sizeof(Klist));
q->next = NULL;
q->data = n;
p->next = q;

p = q;

n--;
}
p->next = NULL;

return L;
}
void straightlistsort(plist L)
{
createlist(L,10);

while(L ->next!= NULL));//在这个地方是死循环了
{
printf("%d\n",L->data）;
}



}

plist L = (plist)malloc(sizeof(Klist));
L->next = NULL;
straightlistsort(L);

[muroachanf]

while(L ->next!= NULL));//在这个地方是死循环了
{
printf("%d\n",L->data）;
L = l->next;

}

[byron1107]

谢谢楼上。。。
丢三落四的。。 惭愧！