如何修复数组,使我没有IndexOutOfBoundsException?

时间:2019-10-31 18:29:53

标签: java binary-search-tree tree-balancing

对于HW分配,我受命将一堆方法添加到BinarySearchTree类中。我有两个方法是balance和InsertTree(我认为它应该命名为InsertNode)。教科书的作者提供了该方法的外观的伪代码。两种方法相互配合。平衡应该采用不平衡树并将每个元素插入数组。我相信InsertTree应该从数组中获取元素,然后将它们放回到新形成的树中。

BST类本身很大,所以我认为发布它不是一个好主意。但是您可以在示例材料下找到源代码here。参考中的代码在ch07.trees包中。

这是我到目前为止对作者的伪代码的解释:

ArrayList<T> array = new ArrayList<T>();

public void balance()
// Will read, store, and recreate the tree
{
      Iterator<T> iter = this.iterator();
      int index = 0;
      while(iter.hasNext())
      {
          array.add(iter.next());
                  index++;
      }
      System.out.println(array.toString());
      System.out.println(index);

      tree = new BinarySearchTree<T>();
      tree.InsertTree(0, index -1);
  }

public void InsertTree(int low, int high)
// Will find the mid-point and insert other elements into left and right subtrees
  {
      if (low == high)
      {
          tree.add(array.get(low));
      }
      else if((low + 1) == high)
      {
          tree.add(array.get(low));
          tree.add(array.get(high));
      }
      else
      {
            int mid = (low + high)/2; 
            tree.add(array.get(mid));
            tree.InsertTree(low,mid-1);
            tree.InsertTree(mid+1,high);
      }
  }

我必须使用ArrayList,因为所有方法都是T类型的泛型。在驱动程序类中,我只是添加了一组不平衡的元素[A,B,C,D,E,F],并且索引将正确显示我将索引增加到6。但是,当新树调用InsertTree(0,index-1)时,我得到了:

Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 2, Size: 0
at java.util.ArrayList.rangeCheck(Unknown Source)
at java.util.ArrayList.get(Unknown Source)
at ch07.trees.BinarySearchTree.InsertTree(BinarySearchTree.java:180)
at ch07.trees.BinarySearchTree.balance(BinarySearchTree.java:163)
at ch07.trees.HWDriver.main(HWDriver.java:67)

第163行是tree.InsertTree(0, index -1);,第180行是tree.add(array.get(mid));

问题似乎与中点有关,但我不确定问题可能是什么。我不是使用ArrayLists的专家,因此对解决此问题的任何帮助将不胜感激。

编辑:

我相信问题已经解决。我将创建的数组放回balance方法中,而不是放到方法外部,然后将数组添加到InsertTree方法的参数中。然后,我必须将每个条件输出从this.tree.add更改为this.add。我也将BinarySearchTree树移回了balance方法,因为在获得NullPointerException之前。

我的方法是否按预期工作尚待确定。

2 个答案:

答案 0 :(得分:1)

看看有一个空集合时会发生什么...

int index = 0;
[...]
tree = new BinarySearchTree<T>();
tree.InsertTree(0, index -1);

您正在尝试在索引(-1)处插入内容。那是不合法的。

答案 1 :(得分:0)

以下是您的答案:

this.tree = new BinarySearchTree<T>();
this.tree.InsertTree(0, index-1);

因此,您已经创建了一个新的空树并将其存储在成员变量“ tree”中。然后,您尝试告诉您的新的空树insertTree(0,5)。