我是树的初学者,我只是尝试第一次实现一个并且正在生成stackoverflower。我知道它可能与一个糟糕的递归调用相关但是我没有看到递归的任何错误我可以得到一点帮助吗?错误在此代码中。
public void insert(Node node, String value)
{
if((value.compareTo(node.getValue())) < 0)
{
if (node.left != null)
{
insert(node.left, value);
}
else
node.left = new Node(value);
}
else if((value.compareTo(node.getValue())) > 0)
{
if(node.right !=null)
{
insert(node.right, value);
}
else
node.right= new Node(value);
}
}
我在这里称呼该方法
public static void main(String[] args) throws FileNotFoundException, IOException {
Tree dataTree = new Tree(new Node("m"));
File file = new File("C:\\randomdata.csv");
BufferedReader br = new BufferedReader(new FileReader(file));
String line;
while((line = br.readLine()) != null){
if(line.toString().contains("zogeti"))
break;
else
{
dataTree.insert(dataTree.getRootElement(),line);
}
}
br.close();
}
答案 0 :(得分:1)
如果文件最初是排序的,那么这个函数看起来会递归N次 一个有N行的文件。 Java没有实现尾递归,所以这肯定是真实的 递归。将其重写为while循环而不是递归函数。
答案 1 :(得分:0)
如果您的树中有node.left == node
或node.right == node
或其他更长的周期,则很可能会发生这种情况。
在当前形式中,如果值相等,则不会触发if块,并且只返回(并返回跟踪),不添加任何内容。这意味着循环可能在此方法之外发生。
您可能会在插入nethod之外的其他地方发现此错误:您的Tree类的构造函数。
答案 2 :(得分:0)
这个CSV文件有多大?文件越大,递归越深,导致stackoverflow。
尝试使用以下命令行参数执行java。
-Xms512m -Xmx512m
另外,如果从文件读取的新行与现有节点值相同怎么办?
以下代码将忽略该...(可能是一项要求,我只是说)。
if((value.compareTo(node.getValue())) < 0)
{
if (node.left != null)
{
insert(node.left, value);
}
else
node.left = new Node(value);
}
else if((value.compareTo(node.getValue())) > 0)
{
if(node.right !=null)
{
insert(node.right, value);
}
else
node.right= new Node(value);
}
答案 3 :(得分:0)
如果您的文件长度为3260953行并且已经排序,那肯定会解释您的问题。如果元素按升序排序,则每次insert
插入新元素时,每次都会将其放置在每个节点的右侧分支上。您最终得到的是一串3260953个线性链接节点,您的代码通过尽可能多的递归调用来访问这些节点。这将溢出堆栈。尝试在较小的文件上运行并按字母顺序排列。
红黑树通过重新分配元素自动平衡树来避免此问题。然而,对这样的数据结构进行编码并不是那么简单。