Question

数据结构串的链式存储赋值问题！

#include"stdio.h"
#include"stdlib.h"
typedef struct Node
{
char ch;
struct Node * next;
}Cstring,*Pstring;
void initstr(Pstring s)//初始化
{
s=(Pstring)malloc(sizeof(Cstring));
s->next=NULL;
}
void strval(Pstring s,char ch[])//赋值
{
int i=1;
if(s->next!=NULL)
s->next=NULL;
s->ch=ch[0];
Pstring s2;
s2=s;
for(i=1;ch[i]!='\0';i++)
{
Pstring s1;
initstr(s1);();//执行到这里就内存不能读！！
s1->ch=ch[i];
s2->next=s1;
s2=s1;
}
}
void main()
{
Cstring L;
char ch[4]="abc";
initstr(&L);
strval(&L,ch);
}

Answer 1

Pstring s1;
initstr(s1);();//执行到这里就内存不能读！！
s1->ch=ch[i];
这里s1,没有分配内存，因为你的Initstr函数写的有问题，不能改变实参的值。
修改Initstr函数，s1正确分配内存就不会造成内存的非法访问即段错误了。

void initstr(Pstring s)//初始化
{
s=(Pstring)malloc(sizeof(Cstring));
s->next=NULL;
}

这里涉及到了对指针的修改，c中函数参数是副本，这样即使修改了只是改变参数的副本，对于实参是没有影响的。应该修改为：
Pstring initstr(void)//初始化
{
Pstring s;
s=(Pstring)malloc(sizeof(Cstring));
s->next=NULL;
return s;
}

或者：
void initstr(Pstring *s)//初始化
{
*s=(Pstring)malloc(sizeof(Cstring));
*s->next=NULL;
}

Answer 2

initstr(s1);();//执行到这里就内存不能读！！你这里函数initstr（s1）；后面那个（）干什么的啊？