ポインタが関数に代入されないように渡されました

Question

char引数をとり、リンクリストにノードの値としてasciiコードを追加するリンクリストスタックを実装しようとしています。

私はpush関数にnstackポインタを渡し、新しいtopを作成するためにnew_nodeに再割り当てしますが、push関数はnstackノードを再割り当てしていないようです。 nstackの値。なぜnstackは再割り当てされないのですか？

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

struct list_node { 
    int element; 
    struct list_node * pnext; 
}; 

void push(struct list_node *operators, int e); 

int pop(struct list_node *operators); 


int main(int argc, char *argv[]) { 
    int newvalue = (int)argv[1][0]; 
    struct list_node * nstack = (struct list_node*)malloc(sizeof(struct list_node)); 

    nstack->element = newvalue; 
    nstack->pnext = NULL; 
    int i; 
     for (i = 2; i < argc; i++) { 
      push(nstack, (int)argv[i][0]); 
     } 
    printf("top: %d\n", nstack->element); 
} 

void push(struct list_node *nstack, int e) { 
    struct list_node * new_node = (struct list_node*)malloc(sizeof(struct list_node)); 
    new_node->pnext = nstack; 
    new_node->element = e; 
    nstack = new_node; 
} 

Answer 1

あなたはポインタのコピーを（値で）渡しているためです。次のようなものが必要です（ポインタへのポインタ）：

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

struct list_node { 
    int element; 
    struct list_node * pnext; 
}; 

void push(struct list_node **operators, int e); 

int pop(struct list_node *operators); 


int main(int argc, char *argv[]) { 
    //int newvalue = (int)argv[1][0]; 
    int newvalue = 1; 
    struct list_node * nstack = (struct list_node*)malloc(sizeof(struct list_node)); 

    nstack->element = newvalue; 
    nstack->pnext = NULL; 

    int i; 
     for (i = 2; i < 7; i++) { 
      //push(nstack, (int)argv[i][0]); 
      push(&nstack, i); 
     } 
    printf("top: %d\n", nstack->element); 
} 

void push(struct list_node **nstack, int e) { 
    struct list_node * new_node = (struct list_node*)malloc(sizeof(struct list_node)); 
    new_node->pnext = *nstack; 
    new_node->element = e; 
    *nstack = new_node; 
} 

Answer 2

Tarodの正しい点を拡大します。呼び出し元関数が行っていることは、ポインタ値のコピーを取得し、それをプッシュ関数で使用されるスタックまたはレジスタ（コンパイラ依存）に置くことです。ただし、プッシュ関数内では、実際にこのレジスタまたはスタック位置の値を変更してから戻します。しかし、呼び出し元関数を返すと、基本的にこの情報が破棄されます（スタックがポップされたか、またはコンパイラによっては別のもののためにレジスタが使用されます）。この周りの唯一の方法は、Tarodが示しているように、ポインタのアドレスを渡してそれに書き込む必要があるときにこれを尊重することです。