0

私は木を作るのが苦手で、再帰を完全に台無しにしています。しかし、ツリーにデータを挿入して表示するプログラムを作ってみました。

問題は、ルート ノードに挿入した後にクラッシュすることです。その理由はわかりません。木は大きすぎません。ちょうど10 int。 

#include <stdio.h>
#include <stdlib.h>
#define SIZE 10;
/* run this program using the console pauser or add your own getch, system("pause") or input loop */
struct node{
    int data;
    struct node * left;
    struct node * right;
};


void insert(struct node * root,int num){
    printf("Insert called for num:%d\n",num);
    if(root == NULL){
        root = (struct node *)malloc(sizeof(struct node));
        root->data = num;
    }else if(num > root->data){ // Number greater than root ?
        insert(root->right,num); // Let the right sub-tree deal with it
    }else if(num < root->data){// Number less than root ?
        insert(root->left,num);// Let the left sub-tree deal with it.
    }else{
        // nothing, just return.
    }
}


void display(struct node * root){ // Inorder traversal
    if(root->left!=NULL){ // We still have children  in left sub-tree ?
        display(root->left); // Display them.
    }

    printf("%d",root->data); // Display the root data

    if(root->right!=NULL){ // We still have children in right sub-tree ?
        display(root->right); // Display them.
    }

}

int main(int argc, char *argv[]) {
    int a[10] = {2,1,3,5,4,6,7,9,8,10};
    int i;
    struct node * tree;

    for(i = 0; i < 10;i++){
        insert(tree,a[i]);
    }
    printf("Insert done");
    return 0;
}

誰かが私が間違っていた場所を教えてもらえますか?

スタック上のコードをレビューするように人々に依頼するのは嫌われていることは知っていますが、ペアプログラミングがうまくいく場合もあります:p

更新:
を設定した後struct node * tree = NULL;、insert()メソッドはうまく機能します。これdisplay()により、プログラムがクラッシュします。

4

1 に答える 1

2

あなたの

int main(int argc, char *argv[]) {
    // ...
    struct node * tree;
    // what is the value of tree at this line?
    for(i = 0; i < 10;i++){
        insert(tree,a[i]);
    }
    // ...
}

マークされた行の「ツリー」は何を指していますか?

于 2013-10-02T04:57:49.537 に答える