別の関数に送信した後にポインタが更新されない

Question

リストから最初の要素をポップしようとしているときにも簡単なリンク済みリストプログラムを作成しようとしていましたが、ポップしていません。このエラーを解決するのを手伝ってください。

ここに必要なファイルとメインコードがあります。おかげさまで

stack.c stack.h

char pop(stack *s){ 
    if(s == NULL){ 
     isEmpty(s); 
     return -1; 
    } 
    char x=s->value; 
    s=(s->next); 
    return x; 
} 

Answer 1

エラーがgetsize機能に次の行にあります。ここ

while((temp->next) != NULL){

temp他のポインタの値で初期化されたポインタでありますs。 （tempとsは同じアドレスを指します）。しかし、getsizeに渡す前に、sにメモリを割り当てていません。したがって、nextメンバーにアクセスすると、segaultが発生します。

ソリューション：

ユーザーにあなたのstackを操作するための選択肢を与える前に、十分なメモリを初期化します。 stack.cには、スタックにメモリを割り当てるcreate関数があります。しかし、あなたはその機能を呼んでいません。エラーを修正するには、プログラムの最初にcreateに電話してください。

また、.cファイルを含めないでください。代わりにヘッダファイル* .hをインクルードする必要があります。 （良い習慣として）。