ポインタを使用して単一のリンクリストで作業する

Question

これは宿題の問題です。プログラムは正常に構築されましたが、実行できません。それはただ止まった。私はリストを作るために "構造体"を使ってみました。私の "挿入"機能に何が問題なのか分かりません。私がここで初めてのことですが、うまくいけば私はいくつかのアドバイスを得るでしょう。

//============================================================================ 
// Name  : test2.cpp 
// Author  : yan zeng 
// Version  : 
// Copyright : Your copyright notice 
// Description : Hello World in C++, Ansi-style 
//============================================================================ 


#include <iostream> 
#include <stdio.h> 
#include <stdlib.h> 
#define TRUE 1 
#define FALSE 0 
typedef int BOOLEAN; 
using namespace std; 

struct Node { 
    int value; 
    struct Node *next; 
}; 

void insert(int x, struct Node **pL); 

void insert(int x, struct Node **pL){ 

    if (*pL == NULL) { 
     struct Node **pL = (struct Node **) malloc(10 * sizeof(int *)); 

     (**pL).value = x; 
     (*pL)->next = NULL; 

    } 

    else 
     insert(x, &((*pL)->next)); 

} 



int main (int argc, char **argv) 
{ 

    // insert code here... 

    // make a list by declaring a pointer to a node 

    struct Node *NodePointer = NULL; 



    for (int i=3; i<20; i+=2) { 
     insert(i,&NodePointer); 
    } 



} 

Answer 1

私が与えることができる最良のアドバイスは、デバッガの使い方を学ぶことです。その他のアドバイス：malloc/freeをC++で使用しないでください。new/deleteを使用してください。

Answer 2

このコードにはかなりの問題がありますが、ここではデバッガを使用することは間違いありません。

あなたのプログラムは、それが

(**pL).value = x; 

でクラッシュしたので、宿題を修正するよう、または空の機能があなたに提供された場合には、このコードはあなたに提供された場合、私はわからない、とあなたが必要「だけで停止します」それを記入する。いずれにしても、その行は間違っています。

また、newを使用する必要がある場合は、他にも述べたように、C++プログラムでmallocを使用しています。