Финальный цикл не повторяется, хотя отладчик показывает, что тест верен
Последний цикл for в этом классе - виновник. Где я пишу слова режима во вновь созданный массив. Цикл for не будет повторяться в последний раз, даже если отладчик eclipse показывает значение i как меньшее, чем (tokens.length-2). Возможно, это проблема забора, но я попробовал цикл do while и кучу всего. Далее я разместил код клиента и текстовый файл, который я использую.
// This class creates an object wherein a text file is segmented and stored
// word for word in an array, facilitating a word count, the ability to check
// for the occurrence of a word and also the functionality of returning the
// most frequently occurring words.
import java.io.*;
import java.util.Arrays;
import java.util.Scanner;
public class TextAnalysis14 {
private String[] tokens;
int maxNoOfWords;
// Constructor that loads a file and assigns each word to an array index
public TextAnalysis14 (String sourceFileName, int maxNoOfWords) throws FileNotFoundException{
this.maxNoOfWords = maxNoOfWords;
Scanner in = new Scanner(new FileReader(sourceFileName));
String file = in.useDelimiter("\\Z").next();
this.tokens = file.split ("[^a-zA-Z]+");
in.close();
}
// Returns the number of words in the file.
public int wordCount(){
return tokens.length;
}
// Checks whether " word " is a word in the text.
public boolean contains(String word){
for(int i=0; i<tokens.length; i++){
if(tokens[i].equalsIgnoreCase(word)){return true;}
}
return false;
}
// Returns the most frequent word(s) in lexicographical order.
public String [] mostFrequentWords(){
Arrays.sort(tokens);
//Finds the mode word occurrence
int wordValue=1;
int maxValue=1;
for(int i=0; i<tokens.length-2; i++){
while(tokens[i].equalsIgnoreCase(tokens[i+1])){
wordValue++;
i++;
}
if(wordValue>maxValue){
maxValue=wordValue;
}
wordValue=1;
}
//Determines length of return array
int numberOfModes=1;
for(int i=0; i<tokens.length-2; i++){
while(tokens[i].equalsIgnoreCase(tokens[i+1])){
wordValue++;
i++;
}
if(wordValue==maxValue){
numberOfModes++;
}
wordValue=1;
}
//writes mode words to array
int cursor =0;
String[] modeWords = new String[numberOfModes];
for(int i=0; i<tokens.length-2; i++){
while(tokens[i].equalsIgnoreCase(tokens[i+1])){
wordValue++;
i++;
}
if(wordValue==maxValue){
modeWords[cursor]=tokens[i];
cursor++;
}
wordValue=1;
}
return modeWords;
}
}
Вот мой код клиента:
import java.io.FileNotFoundException;
import java.util.Arrays;
public class TextAnalysis_test01 {
public static void main(String[] args) throws FileNotFoundException {
TextAnalysis14 ta14 = new TextAnalysis14("testtext01.txt", 100);
System.out.println(ta14.wordCount());
System.out.println(ta14.contains("Bla"));
System.out.println(ta14.contains("hello"));
System.out.println(Arrays.toString(ta14.mostFrequentWords()));
}
}
Ниже приводится содержание моего текстового файла:
bla bla
dim dim
dum dum
И я получаю вывод:
6
true
false
[bla, dim, null]
Насколько я могу судить, я ничего не записываю в последний индекс возвращаемого строкового массива, потому что последний цикл for не выполняет итерацию в последний раз. Часть моего класса с комментариями: // записывает слова режима в массив.
Любая помощь или совет будут находкой. Ура!
1 ответ
Я бы действительно посоветовал вам провести рефакторинг вашего кода в попытке улучшить его, но если вы только после исправления, следующее изменение должно дать ожидаемый результат:
int cursor =0;
String[] modeWords = new String[numberOfModes];
for(int i=0; i<=tokens.length-2; i++){
while(i+1 <= tokens.length-1 && tokens[i].equalsIgnoreCase(tokens[i+1])){
wordValue++;
i++;
}
if(wordValue==maxValue){
modeWords[cursor]=tokens[i];
cursor++;
}
wordValue=1;
}
return modeWords;
}