Удаление всего бинарное дерево поиска сразу

Question

Удаление всего бинарное дерево поиска сразу

голоса

-1

Я пытался реализовать функцию удаления BST, но я не знаю, почему это не работает, я думаю, что это логически правильно. Может ли тело пожалуйста, скажите мне, почему я получаю сообщение об ошибке во время выполнения и как я должен это исправить.

  #include <iostream>
  using namespace std;

class node{
public:
int data;
node *right;
node *left;
node(){
    data=0;
    right=NULL;
    left=NULL;
      }
};

class tree{
node *head;
int maxheight;
     public:
tree(){head=0;maxheight=-1;}
bool deletenode(int key,node* root);
int get_height(){return maxheight;}
void insert(int key);
void pre_display(node* root);
     void delete_tree(node *root);
     node* get_head(){return head;}
         };

void tree::insert(int key){
     node *current=head;
    node *newnode=new node;

    if(newnode==NULL)
    throw(key);

    newnode->data=key;
    int height=0;

if(head==0){
head=newnode;
     }
else
{
    while(1){
    if(current->right==NULL && current->data < newnode->data)
    {
        current->right=newnode;
        height++;
        break;
    }
    else if(current->left==NULL && current->data > newnode->data)
    {
        current->left=newnode;
        height++;
        break;
    }
    else if(current->right!=NULL && current->data < newnode->data)
    {
         current=current->right;
         height++;
   }
    else if(current->left!=NULL && current->data > newnode->data)
    {
               current=current->left;
          height++;
    }
         }
 }
 if(height>maxheight)
 maxheight=height;
 }

 void tree::pre_display(node *root){
 if(root!=NULL)
 {
 cout<<root->data<< ;
 pre_display(root->left);
 pre_display(root->right);
 }
 }

 void tree::delete_tree(node *root){
  if(root!=NULL)
 {
 delete_tree(root->left);
 delete_tree(root->right);
 delete(root);
 if(root->left!=NULL)
 root->left=NULL;
 if(root->right!=NULL)
 root->right=NULL;
 root=NULL;
 }
 }

int main(){
tree BST;
int arr[9]={17,9,23,5,11,21,27,20,22},i=0;

for(i=0;i<9;i++)
BST.insert(arr[i]);

BST.pre_display(BST.get_head());
cout<<endl;
BST.delete_tree(BST.get_head());
BST.pre_display(BST.get_head());
cout<<endl;

system(pause);
return 0;
}

Все остальные функции работают правильно, вам просто нужно проверить delete_treeфункцию, другой код предоставляется , чтобы дать представление о структуре моего BST.

c++binary-search-tree

Задан 09/11/2011 в 09:22
источник пользователем Zohaib

На других языках...

7 ответов

голоса

0

Вы не должны читать от корня после его удаления. Переместить delete(root)строку вниз.

Ответил 09/11/2011 в 09:26
источник пользователем David Grayson

голоса

1

Проблема здесь:

 delete_tree(root->left);
 delete_tree(root->right);
 delete(root);
 if(root->left!=NULL)
 root->left=NULL;
 if(root->right!=NULL)
 root->right=NULL;
 root=NULL;

Вы пытаетесь присвоить NULL члену корня:

root->left=NULL;

который уже был удален. Там нет необходимости делать это, так как вы уже освобождение памяти вdelete_tree(root->left);

Ответил 09/11/2011 в 09:28
источник пользователем Luchian Grigore

голоса

0

Рекурсивное удалить левое и правое вспомогательное дерево и ваше дерево будет удалено так просто, как:

void delete(node *root){
  if(root->left==NULL && root->right==NULL)  //leaf node, delete it!!
    free(root);
  delete(root->left);
  delete(root->right);
 }

Ответил 15/07/2014 в 18:12
источник пользователем kalimba

голоса

1

Вы можете удалить с помощью: // Это функция очистки:

void cleantree(tree *root){
 if(root->left!=NULL)cleantree(root->left);
 if(root->right!=NULL)cleantree(root->right);
 delete root;}

// Здесь мы называем функцию очистки:

cleantree(a);
a=NULL;

// Где «а» указатель на корень дерева.

Ответил 24/01/2015 в 06:39
источник пользователем Turdubek Joldoshov

голоса

0

Короткий ответ: ваша реализация дескриптора узла отсутствующего правильный явной реализации деструктора (по умолчанию, генерируемый компилятором будет использоваться вызовом удаления оператора: вызов деструктора и отпуская выделенное пространство в куче) -The одно, что четкие ссылки к братья и сестры

Ответил 05/06/2016 в 00:39
источник пользователем damirlj

голоса

-1

вот мое предложение, используя рекурсивность ..

 template <class T> void Tree<T>::destroy(Noeud<T> ** r){

            if(*r){
                if(!(*r)->left && !(*r)->right){  //a node having no child
                  delete *r; *r=NULL;
                }else if((*r)->left && (*r)->right){ //a node having two childs
                    destroy(&(*r)->left); //destroy the left tree
                    destroy(&(*r)->right); //destroy the right tree
                    destroy(r); //destroy the node
                }else if((*r)->left){ //a node has only left child
                    destroy(&(*r)->left); //destroy the left tree
                    destroy(r); //destroy the node
                }else if((*r)->right){ //a node has only right child
                    destroy(&(*r)->right); //destroy the right tree
                    destroy(r); //destroy the node
                }
            }
}

//in function main()
int main(){
Tree<int> a(5); // 'a' is a tree of int type with a root value equal 5
a.add(2);a.add(7);a.add(6);a.add(-1);a.add(10); // insert values into the tree 
a.destroy(&a.root);  //destroy the tree
}

Ответил 06/02/2017 в 17:28
источник пользователем Brahim KHOUY

parapura rajkumar · Accepted Answer · 2011-11-09 09:27

В вашем delete_tree

void tree::delete_tree(node *root){
    if(root!=NULL)
    {
        delete_tree(root->left);
        delete_tree(root->right);
        delete(root);
        if(root->left!=NULL)
            root->left=NULL;
        if(root->right!=NULL)
            root->right=NULL;
        root=NULL;
    }
}

вы обращаетесь к корневой переменной после того, как вы ее удалили

Также вы звоните

    BST.delete_tree(BST.get_head());
BST.pre_display(BST.get_head());

pre_display после удаления дерева. delete_tree после удаления дерева следует также установить BST.head на NULL

Также критика. BST имеет тип дерева. Это уже имеет переменную-член головки, указывающий корневой узел. Так delete_tree / pre_display не нужны никакие параметры на всех.