2

我得到了一组称为“字典”的字符串,它存储为一个字段,代表一个单词字典。

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

这是我的代码:

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. 检查字典中在这个线程中传递的可能性,由变量“chosen”表示。

  1. 如果字典确实包含这种可能性,请将其添加到调用结束时返回的名为“anagrams”的集合中。然后再次传递可能性以尝试从中做出其他组合。

3.. 如果字典中不包含该可能性,则修改字符串尝试其他可能性,然后递归测试。

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

4

1 回答 1

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下次条件返回的事情。

于 2011-02-22T00:58:22.520 回答