Java StackOverFlowError - 错误的递归调用?

时间:2011-02-22 00:54:52

标签: java recursion backtracking

我给了一组名为“dictionary”的字符串,存储为字段,表示单词字典。

我要编写一个接受String参数(“phrase”)的方法,并返回一个包含字典集中所有单词的集合,可以通过重新排列给定短语中的字符来实现。基本上我在字典中搜索字谜。

这是我的代码:

public Set<String> getWords(String phrase) {
    Set<String> anagrams = new TreeSet<String>();
    String chosen = "";
    anagrams.addAll(getWords(phrase, chosen));
    return anagrams;
}

public Set<String> getWords(String phrase, String chosen) {
    if (phrase == null) {
        throw new IllegalArgumentException();
    } 
    Set<String> anagrams = new TreeSet<String>();
    if (dictionary.contains(chosen)) {
        anagrams.add(chosen);
        anagrams.addAll(getWords(phrase, chosen));
    } else {
        for (int i = 0; i < phrase.length(); i++) {
            String ch = phrase.substring(i, i + 1);         
            String temp = phrase.substring(0, i) + phrase.substring(i + 1);
            anagrams.addAll(getWords(temp, chosen + ch));
        }
    }
    return anagrams;
}    

所以我的方法是: 1.检查字典中是否存在此线程中传递的可能性,由变量“selected”表示。

  1. 如果字典确实包含该可能性,请将其添加到呼叫结束时返回的名为“anagrams”的集合中。然后再次通过这种可能性,尝试用它来制作其他组合。
  2. 3 ..如果字典不包含这种可能性,请修改字符串以尝试其他可能性,然后以递归方式进行测试。

    上面的代码产生了“堆栈溢出错误”,我的研究表明这意味着我正在进行无限递归,或者无限地一遍又一遍地传递相同的字符串。不过,我无法看到我在做什么。你能吗?

1 个答案:

答案 0 :(得分:8)

public Set<String> getWords(String phrase, String chosen) {
    //...
    if (dictionary.contains(chosen)) {
        anagrams.add(chosen);
        anagrams.addAll(getWords(phrase, chosen)); //<--here we are

您使用完全相同的参数进行递归调用。并且您不会做任何会使条件下次返回false的事情。