0

所以我在 NetBeans IDE 上用 java 运行 Boggle Solver。当我运行它时，我必须在 10 分钟左右后退出，因为它最终需要大约 2 小时才能完全运行。我的代码是否有问题，或者一种方法会更快？

public void findWords(String word, int iLoc, int jLoc, ArrayList<JLabel> labelsUsed){

    if(iLoc < 0 || iLoc >= 4 || jLoc < 0 || jLoc >= 4){
        return;
    }

    if(labelsUsed.contains(jLabels[iLoc][jLoc])){
        return;
    }

    word += jLabels[iLoc][jLoc].getText();
    labelsUsed.add(jLabels[iLoc][jLoc]);

    if(word.length() >= 3 && wordsPossible.contains(word)){
        wordsMade.add(word);
    }

    findWords(word, iLoc-1, jLoc, labelsUsed);
    findWords(word, iLoc+1, jLoc, labelsUsed);
    findWords(word, iLoc, jLoc-1, labelsUsed);
    findWords(word, iLoc, jLoc+1, labelsUsed);
    findWords(word, iLoc-1, jLoc+1, labelsUsed);
    findWords(word, iLoc-1, jLoc-1, labelsUsed);
    findWords(word, iLoc+1, jLoc-1, labelsUsed);
    findWords(word, iLoc+1, jLoc+1, labelsUsed);

    labelsUsed.remove(jLabels[iLoc][jLoc]);
}

这是我从以下位置调用此方法的地方：

public void findWords(){
    ArrayList <JLabel> labelsUsed = new ArrayList<JLabel>();
    for(int i=0; i<jLabels.length; i++){
        for(int j=0; j<jLabels[i].length; j++){
            findWords(jLabels[i][j].getText(), i, j, labelsUsed);
            //System.out.println("Done");
        }
    }
}

编辑：顺便说一句，我使用的是 GUI，板上的字母是使用 JLabel 显示的。

4

2 回答 2

2

好吧，对于初学者来说，你运行ArrayList.contains()( labelsUsed.contains(..)) 很多次，每次都是O(n)- 你应该考虑使用更有效的数据结构 -Set如果可能的话（没有重复元素）。

于 2012-12-03T16:57:01.457 回答
0

您正在混合迭代和递归方法，因此该findWords方法被称为时间而不是时间 。n^nn

要么删除该方法的八findWords...行，要么删除你的.findWordsformain

于 2012-12-03T17:02:30.200 回答