我试图使用递归方法在二叉树中插入节点。但是一旦它离开方法,根就变成了新节点,而左子节点和右子节点都是空的。有了这个,我试图了解递归是如何工作的。此外,递归方法总是必须返回一些东西。以下是代码。
public class BinaryNode {
int key;
BinaryNode left;
BinaryNode right;
public BinaryNode( int key){
this.key = key;
// this.left = left;
//this.right = right;
}
}
public class BinaryTree {
BinaryNode root;
public void insert(int key){
BinaryNode newNode = new BinaryNode(key);
if(root == null){
root = newNode;
}else{
BinaryNode focusNode = root;
BinaryNode parent;
while(true){
parent = focusNode;
if(key<focusNode.key){
focusNode = focusNode.left;
if(focusNode==null){
parent.left= newNode;
return;
}
}else{
focusNode = focusNode.right;
if(focusNode==null){
parent.right= newNode;
return;
}
}
}
}
}
public BinaryNode recursiveInsert(int key, BinaryNode node){
BinaryNode newNode = new BinaryNode(key);
if (node == null){
root = newNode;
}
else{
if(key < node.key){
root.left = recursiveInsert(key, node.left);
}
else{
root.right = recursiveInsert(key, node.right);
}
}
return root;
}
public String toString(){
String toTree = null;
return toTree;
}
public static void main(String[]args){
BinaryTree tree = new BinaryTree();
tree.recursiveInsert(7, tree.root);
tree.recursiveInsert(6, tree.root);
tree.recursiveInsert(4, tree.root);
tree.recursiveInsert(8, tree.root);
tree.recursiveInsert(9, tree.root);
tree.recursiveInsert(5, tree.root);
}
}
我正在尝试的方法是recursiveInsert。谢谢!
答案 0 :(得分:2)
我认为问题来自
if (node == null){
root = newNode;
}
您正在遍历树,在最后一步中,您正在询问叶节点的左/右子节点。这个没有,所以它的孩子是空的。 这是递归调用返回的值,最后,它被分配给root。
要解决此问题,请在降级到节点之前,确保子节点存在。
这也有点奇怪
root.left = recursiveInsert(key, node.left);
应该是node
而不是root
。
答案 1 :(得分:0)
您的递归代码应该是这样的:
public BinaryNode recursiveInsert(int key, BinaryNode node) {
if (node == null) {
return root = new BinaryNode<>(key);
} else {
if (key < node.key) {
if (node.left == null) {
return node.left = newNode;
} else {
return recursiveInsert(key, node.left);
}
} else {
if (node.right == null) {
return node.right = newNode;
} else {
return recursiveInsert(key, node.right);
}
}
}
}
您在递归函数中作用于root
节点,您应该对传递给递归函数的node
采取行动。
答案 2 :(得分:0)
您的代码存在一些问题,我不一定会这样做,因为我认为它们会分散您的核心问题。
首先回答你的第二个问题:不,没有规则说递归方法必须返回任何东西。他们知道何时终止更重要。
至于你的错误,我认为这可能是因为你的insert
方法始终返回并在root
上运行。您可能需要修改并返回newNode
。
答案 3 :(得分:0)
这是因为您的recursiveInsert
方法始终在root
上运行。试试这个 -
public Node recursiveInsert(Node currentParent, Node newNode) {
if (currentParent == null) {
return newNode;
} else if (newNode.key > currentParent.key) {
currentParent.right = recursiveInsert(currentParent.right, newNode);
} else if (newNode.key < currentParent.key) {
currentParent.left = recursiveInsert(currentParent.left, newNode);
}
return currentParent;
}