C - 単一リンクリストから要素を削除する

Question

問題がどこにあるのかわかりません。

そして、del関数が期待どおりに機能しないのはなぜですか？これを実行した後

#include<stdio.h> 
#include<malloc.h> 

typedef struct list List; 

struct list 
{ 
    int data; 
    List* next; 
}; 

void prl(List* head); 
void ins(List** head, int value); 
void del(List** head, int value); 

int main() 
{ 
    List* head = NULL; 

    ins(&head, 10); 
    ins(&head, 50); 
    ins(&head, 20); 
    ins(&head, 150); 
    ins(&head, 120); 

    del(&head, 150); 

    prl(head); 

    //freeing dynamically allocated memory for each nodes 

    while(head!=NULL) 
    { 
     List* t = head; 
     head = head->next; 
     free(t); 
    } 

    return 0; 
} 

void prl(List* head) 
{ 
    if(head == NULL) 
     printf("List is empty\n"); 
    else 
    { 
     while(head != NULL) 
     { 
      printf("%d ", head->data); 
      head = head->next; 
     } 
    } 
} 

void ins(List** head, int value) 
{ 
    List* node = malloc(sizeof *node); 

    node->data = value; 
    node->next = NULL; 

    node->next =*head; 
    *head = node; 

} 


void del(List** head, int value) 
{ 
    List* p,*q; 
    p=q=*head; 

    if((*head)->data == value) 
    { 
     *head = (*head)->next; 
     free(p); 
     return; 
    } 
    else 
    { 
     while(p->next != NULL) 
     { 
      if(p->data == value) 
      { 
       q->next = p->next; 
       free(p); 
      } 
      else 
      { 
       q = p; 
       p = p->next; 
      } 
     } // while loop ends 

    } // outer else ends 

} // del function ends 

、出力は空白で、私は何か（論理的には）デル機能アウター他のループ内で間違っていると思います、しかし、最初の値は、この機能を使用して削除することができます。

Answer 1

私はあなたがここにreturn文を追加していないと思う：

if(p->data == value) 
    { 
     q->next = p->next; 
     free(p); 
     return; 
    } 

Answer 2

私のコメントにもかかわらず、私はあなたを助けることにしました。

これらの行を見てみましょう：？

while(p->next != NULL) 
{ 
    if(p->data == value) 
    { 
     q->next = p->next; 
     free(p); 
    } 
    else 
    { 
     // Irrelevant... 
    } 
} 

は、ループ、p->data == value内部の状態は、その後、何が起こるか、真実であることを起こることを言うことができますかq->nextがp->nextを指すようにすると、それはおそらく問題ありません。pを他の場所に指定しなくてもループを続けることができます。

pは、freeと呼ばれているデータを指しているので、参照番号pなどを参照してください。ループ状態のp->nextは、の未定義の動作になります。

解決策はpとqを適切に更新することです。