Question

我很难理解为什么这门课没有运作。它是数据结构课程作业的一部分（编辑：作业的截止日期已经过去，我只是想弄明白......）。该节点是构建在BST上的AVL树的一部分，我选择实现它的方法是在Node类中创建方法以找到Balance因子和高度。

该课程的结构如下：

public class Node<T extends Comparable<? super T>> {

public T data;
public Node left;
public Node right;

public Node(T IN) {
    data = IN;
}

public Node(T IN, Node L, Node R) {
    this(IN);
    left = L;
    right = R;
}

@Override
public String toString() {
    return data.toString();
}

@Override
public Node clone() {
    return new Node(this.data) ;
}

public int getHeight() {
    return getHeight(this) ;
}

public int getBF() {

        //Calculate BF
        int balanceFactor = 0;
        if (right != null && left != null)
            balanceFactor = getHeight(right) - getHeight(left);
        else if (left != null) {
            balanceFactor = 0 - getHeight(left) ;
        }
        else if (right != null) {
            balanceFactor = getHeight(right) ;
        }
        else
            balanceFactor = 0 ;
        return balanceFactor ;
}

private int getHeight(Node p) {
    if (p.left == null && p.right == null ) {
        return 0 ;
    }
    else if (p.left != null && p.right != null) {
        return 1 + max(p.left.getHeight(), p.right.getHeight());
    }
    else if (p.left != null) {
        return 1 + p.left.getHeight() ;
    }
    else if (p.right != null) {
        return 1 + p.right.getHeight() ;
    }
    else {
        return 0;
    }
}

private int max(int x, int y) {
    if (x >= y) {
        return x;
    } else {
        return y;
    }
}

}

并且调用该方法的函数是：

@Override
public boolean insert(T el) {
    boolean test = super.insert(el) ;
    if (test) {
        return checkBalance(root) ;
    }
    else
        return false ;
}

我接受的例外是重复：

Exception in thread "main" java.lang.StackOverflowError
at Node.getHeight(Node.java:54)
at Node.getHeight(Node.java:33)
at Node.getHeight(Node.java:58)

Answer 1

我建议您的树变形或非常大。代码似乎没有问题。

如果您的树变形使得您在同一棵树中插入了两次Node，那么此代码将会中断。

已添加 - 您正在吃多于您需要的堆栈 - 将p.left.getHeight()替换为getHeight(p.left)等将避免每次递归一次堆栈推送。如果您的问题仅仅是大树，那么这可能会让您大吃一惊，但这只会推迟问题。

Answer 2

通过查看两个getHeight方法，您似乎没有树而是循环图。您应该开始使用仅由根组成的树进行测试，然后添加节点，直到您观察到无限递归。您可能在重新平衡树的函数中出错。

编辑：你应该将属性（至少左和右）设为私有。

为什么这个方法会导致无限递归调用？

2 个答案: