Binarne drzewo poszukiwań w C

Question

Binarne drzewo poszukiwań w C

głosy

5

Jestem Python facet. Uczenie się języka C, a ja starałem się wdrożyć binarne drzewo poszukiwań w C. I zapisany kod, a ja już próbuje od kilku godzin, ale nie są w stanie uzyskać dane wyjściowe, jak oczekiwano. Proszę pomóż!

Proszę mnie poprawić.

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

typedef int ElementType;

typedef struct TreeNode {
  ElementType element;
  struct TreeNode *left, *right;
} TreeNode;

TreeNode *createTree(){
    //Create the root of tree
    TreeNode *tempNode;
    tempNode = malloc(sizeof(TreeNode));
    tempNode->element = 0;
    tempNode->left = NULL;
    tempNode->right = NULL;
    return tempNode;
}

TreeNode *createNode(ElementType X){
    //Create a new leaf node and return the pointer
    TreeNode *tempNode;
    tempNode = malloc(sizeof(TreeNode));
    tempNode->element = X;
    tempNode->left = NULL;
    tempNode->right = NULL;
    return tempNode;
}

TreeNode *insertElement(TreeNode *node, ElementType X){
    //insert element to Tree
    if(node==NULL){
        return createNode(X);
    }
    else{
        if(X < node->element){
            node->left = insertElement(node->left, X);
        }
        else if(X > node->element){
            node->right =  insertElement(node->right, X);
        }
        else if(X == node->element){
            printf(Oops! the element is already present in the tree.);
        }
    }
}

TreeNode *displayTree(TreeNode *node){
    //display the full tree
    if(node==NULL){
        return;
    }
    displayTree(node->left);
    printf(| %d , node->element); 
    displayTree(node->right);
}

main(){
    //pointer to root of tree #2
    TreeNode *TreePtr;
    TreeNode *TreeRoot;
    TreeNode *TreeChild;

    //Create the root of tree
    TreePtr = createTree();

    TreeRoot = TreePtr;

    TreeRoot->element = 32;
    printf(%d\n,TreeRoot->element);

    insertElement(TreeRoot, 8);
    TreeChild = TreeRoot->left;
    printf(%d\n,TreeChild->element);  

    insertElement(TreeRoot, 2);
    insertElement(TreeRoot, 7);
    insertElement(TreeRoot, 42);
    insertElement(TreeRoot, 28);
    insertElement(TreeRoot, 1);
    insertElement(TreeRoot, 4);
    insertElement(TreeRoot, 5);

// the output is not as expected :(
    displayTree(TreeRoot);
}

c binary-search-tree

Utwórz 24/03/2010 o 11:42
źródło użytkownik heapzero

W innych językach...

2 odpowiedzi

głosy

2

Twój insertElementnie zawsze zwraca wartość. To dlatego twoje rekurencyjne wywołania udać. Należy poinformować kompilator, aby ostrzec o błędach, takich jak że (na przykład na gcc używać -Wall).

displayTreema podobny błąd, wracając nic, gdy jest on określony do zwróci TreeNode*.

mainNależy również zwrócić wartość (lub należy ją zadeklarować void).

Odpowiedział 24/03/2010 o 11:49
źródło użytkownik Thomas

codaddict · Accepted Answer · 2010-03-24 11:53

Problemem jest wkładania. Jeśli nodejest NULLutworzenie nowego węzła i zwraca go. Ale co, jeśli węzeł nie jest NULL. Robicie prawidłowych zmian w prawo / lewo poddrzewa, ale nie masz nic powrocie.

Zmiana

if(X < node->element){
    node->left = insertElement(node->left, X);
}
else if(X > node->element){
    node->right =  insertElement(node->right, X);
}

do:

if(X < node->element){
    node->left = insertElement(node->left, X);
    return node; // add this.
}
else if(X > node->element){
    node->right =  insertElement(node->right, X);
    return node; // add this.
}