https://onlinegdb.com/ryscef5gU
程式新手入门debug，弄了两个小时还弄不对QQ
如上程式码连结
功能在于反转串行
这边仅贴上有问题的function code
void reverse_list(node *s)
{
node *x=new node();
node *p=new node();
node *q=new node();
p=x=s; //p, x都初始化为list头s
q=NULL; //q初始化为list的bottom=Null
while(p->link!=NULL)
{
x->link=q; //x指回前一个node q
q=p; //q变成p
p=p->link; //p再往后搜索一格
x=p; //x也是p
}
s=x; //把s这个头设为x即收工
}
但是最后因为这个function code拿到segmentation fault结果
谢谢赐教!!

猜测 x->link=q; 往回指了，p就拿不到下一个了

你误会 p=s 的 = 是 copy，其实只是 assign pointer所以把 x 的 link 指向 q 时，p 的 link 也一样指向 q了

x p q初始化之后又马上被assign不就都leak了吗...

楼上说得完全没错，不过那不是出槌的直接原因所以我和一楼才没有讲，但现在想想既然看见就应该要讲..回传用 return 的呀，不要 assign 参数 s参数是传 s 的值进来而已，不是直接让你存取 main 的 s

解决了 我的return type还通通写错 感谢热心指导!! ^^

看一下call by value的解释吧 s是copy传进function的

OK Thanks!!

void 不会有回传值 要用void 就设global variable吧