Question

我在不使用递归的情况下实现预先建立的二叉树遍历。这是我的代码：

#include<iostream>
#include<stack>
using namespace std;

struct node{

int data;
node *left;
node *right;

};

node *getNewNode(int data){   //method for creating new node

node *newNode = new node();
newNode->data=data;
newNode->left=newNode->right = NULL;
return newNode;

}

node *Insert(node *root , int data){     //Method for insert new data in tree

if(root == NULL){
    root = getNewNode(data);
}
else if(data>root->data){
    root->right = Insert(root->right,data);
}
else{
    root->left = Insert(root->left,data);
}

return root;

}

void Print(node *root){  //Method for preorder traversal with recursion

if(root == NULL){
    return;
}

 cout<<root->data<<" ";
Print(root->left);

Print(root->right);

}

void preOdr(node *root){  //Without recursion

stack<node*> stk;
cout<<root->data<<" ";

do{
    a:
    if(!(root->right==NULL&&root->left==NULL)){
        if(root->right!=NULL){
             stk.push(root->right);
        }
        if(root->left!=NULL){
             stk.push(root->left);
        }
    }
    cout<<stk.top()->data<<" ";
    root=stk.top();
    stk.pop();
    goto a;
}
while(!stk.empty());



}

int main(){

node *root = NULL;
root = Insert(root,10);
root = Insert(root,6);
root = Insert(root,15);
root = Insert(root,3);
root = Insert(root,9);
root = Insert(root,11);
root = Insert(root,17);
root = Insert(root,11);
root = Insert(root,62);
root = Insert(root,135);
root = Insert(root,30);
root = Insert(root,98);
root = Insert(root,117);
root = Insert(root,176);

Print(root);
cout<<endl;
preOdr(root);

return 0;
}

在我的程序中，我还使用递归创建了前序遍历的方法，以验证非递归方法Print()给出的输出。

在非递归方法中，首先我打印根，而不是分别将该节点的左右子节点（如果有）推送到堆栈。在此之后，我从堆栈中弹出项目并重复上述过程，直到堆栈不为空。

当我运行此代码时，它正确地提供输出，但在此之后崩溃。我无法理解名为preOrd()的方法中的问题。我已附上完整的代码以便更好地理解。

Answer 1

您的preOrd()功能被搞砸了。

1.不要使用goto

首先，你在循环结束时有goto，在循环开始时向右跳。这可以防止while条件被验证并导致循环。一旦堆栈为空，它将尝试pop / top导致UB（这里是崩溃）！

每当您想要使用goto时，请三思而后行！首选break来中断循环，或continue正确循环。这样可以避免将来出现这类问题。

当天的引用：＆＃34; goto - 臭名昭着的goto。＆＃34; - Bjarne Stroustrup

2.然后修改循环逻辑

如果您只是注释掉goto，它会更好用，但会在第二级（此时没有崩溃）后停止。显然，你不会推动堆栈上所需的一切。

这是修订版：

void preOdr(node *root){  //Without recursion

    stack<node*> stk;
    stk.push(root);           // put the root not on th stack and 
                              // then process it like the others
    while (!stk.empty()) {
        root=stk.top();
        stk.pop();
        cout<<root->data<<" ";  // get and show top of stack
        if(root->right)         // then push the childern 
            stk.push(root->right);
        if(root->left)    
            stk.push(root->left);
    }
}

Live demo

Answer 2

您已经获得了一个goto关键字，可以将您带回到该代码块的开头，这是最后一段时间的目的。它是你循环中没有检查的循环。

输出正确但每次打印输出后程序崩溃

2 个答案:

1.不要使用goto

2.然后修改循环逻辑