C pointer shenanigans

Question

私は、データへのポインタを格納するためにC（Linked Listを使用）でQueueを実装しようとしています。 en-queuingは正常に動作しているようですが、de-queuing時のポインタに関するいくつかの問題があります。

void* data = malloc(sizeof(int)); 
dequeue(&Q, data); 
printf("(%d) %d\n", k, *(int*)data); 

デキュー（）：：私のメイン（）で

は私のデバッグに基づき

int dequeue(struct queue *q, void *value) 
{ 
    struct queue_node *tmp; 

    if (!q->first) { 
     value = 0; 
     return 1; 
    } 
    value = q->first->data; 
    tmp = q->first; 
    if (q->first == q->last) 
     q->first = q->last = NULL; 
    else 
     q->first = q->first->next; 

    free(tmp); 
    return 0; 
} 

、それが思われるループのメインで*データポインタの値（） dequeue（）に設定されている値は保持されません。私は何が欠けていますか？

編集：

struct queue_node 
{ 
    struct queue_node *next;  
    void* data; 
}; 

struct queue 
{ 
    struct queue_node *first; 
    struct queue_node *last; 
}; 

Answer 1

queue_nodeのdataは、この値があったため、いくつかの値へのポインタ（ここではそれがintですが、それがそうでなければあなたの代わりにintを使用したい、常に真ではないかもしれない...）

を保持していますmallocで割り当てられ（ローカル変数ではありません）、ある時点でそれを解放する必要があります。だから、

、void**を受け入れるために、関数のシグネチャを変更、main()にintに領域を割り当てるが、void * data = 0パラメータとして&dataでdequeueを呼び出すことはありません。完了したらfreeのデータを忘れないでください。 dequeueには、*value = q->first->dataと設定します。

Answer 2

あなたが関数にローカルポインタである「値」変数の値を設定しています。

あなたはそれが指し示すための値を設定したい場合は、使用：

*value = 0; 

と：

*value = q->first->data; 

編集（質問編集後）：queue_node.dataは、それ自体であるので、上記の@Andreiの注記のように、void **を渡す方が合理的です。

Answer 3

（ボイド*）データをされると仮定して、（int型）値は、データが指すたい、あなたは

*(int *)value = *(int *)q->first->data;

で

value = q->first->data;

を交換する必要があります関数deque（）の中で。