我有一个包含 50 行的文件,每行仅包含一个单词,并且我有以下类从文件中读取随机行:
public class WordGenerator{
static Random generator;
static int numberGenerator(){
generator = new Random();
return generator.nextInt(50)+1;
}
static String wordSelector(){
String word = null;
FileInputStream fis = null;
BufferedReader br = null;
String temp = String.format("..%sutil%s", File.separator, File.separator);
try{
fis= new FileInputStream(temp + "Words.txt");
br = new BufferedReader(new InputStreamReader(fis));
}catch(FileNotFoundException ex){
System.out.println("File not found: " + ex);
}
try{
for(int i = 0; i < numberGenerator(); ++i){
word = br.readLine();
}
}catch(IOException e){
System.out.println("Error in line reading: " + e);
}
System.out.println(word);
return word;
}
}
问题是,由于某种原因,该类永远不会返回前 5-10 行之后的任何内容,而只是不断返回前几行。
这可能是什么原因造成的?
您的循环在每次迭代时都会调用
numberGenerator()
。这意味着您不是从 1 到 50 中选择一个随机数,而是每次通过循环绘制一个新数字。
当您完成 5 或 10 次迭代时,您可能已经抽取了一个小于 i 的随机数。
修复该问题时,将
new Random()
实例化移到该方法之外。你只需要一个随机数生成器,并且你可以继续调用 nextInt()
。您每次都不必要地创建一个新对象。