Java （java.lang.OutOfMemoryErrorの原因）

Question

前提・実現したいこと

ローマ字（仏語）の文章を単語に区切るプログラムを作成中、以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
	at java.base/java.lang.StringLatin1.newString(StringLatin1.java:764)
	at java.base/java.lang.String.substring(String.java:1908)
	at structuredonnées/langueIdentifier.Test.main(Test.java:33)

該当のソースコード

package langueIdentifier;

import java.io.BufferedReader;
import java.io.IOException;
import java.nio.charset.Charset;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.text.BreakIterator;
import java.util.ArrayList;

public class Test {
	
private String textFile;
	
public Test (String textFile) {

this.textFile = textFile;
	}
	
public static void main(String[] args) {
Path p1 = Paths.get("path");
Charset charset = Charset.forName("UTF-8");
ArrayList<String> mots = new ArrayList<String>();

try (BufferedReader reader = Files.newBufferedReader(p1, charset)){
String line = reader.readLine();
BreakIterator wb = BreakIterator.getWordInstance();

while (line != null) {
wb.setText(line);
int lastIndex = wb.first();
while (BreakIterator.DONE != lastIndex) {
int firstIndex = lastIndex;
lastIndex = wb.next();
if (lastIndex != wb.DONE && Character.isLetterOrDigit(line.charAt(firstIndex))) {
String mot = line.substring(firstIndex, lastIndex).toLowerCase();
mots.add(mot);
}

}

}
} catch (IOException ioe) {

System.out.println(ioe.toString());
}
for (String mot : mots) {
			
System.out.println(mot);
}
}
}
	

試したこと

ネットでも調べてみましたがよくわかりませんでした。問題のテキストファイルは大したサイズのファイルでもなく、もう少し重いテキストファイルを読み込む他のプログラムは正常に作動しているので、不可解です。

よろしくお願いします。

Comments

[m.ts10806]

無限ループでは。デバッグしてみましたか？

Answer 1

while (line != null) {
  wb.setText(line);
  int lastIndex = wb.first();
  while (BreakIterator.DONE != lastIndex) {
    int firstIndex = lastIndex;
    lastIndex = wb.next();
    if (lastIndex != wb.DONE && Character.isLetterOrDigit(line.charAt(firstIndex))) {
      String mot = line.substring(firstIndex, lastIndex).toLowerCase();
      mots.add(mot);
    }

  }

}

このループ内で、lineを書き換える場所は1つもありません（そして、ループ内に入出力処理はないので、IOExceptionも起きません）。結果、一度このループに入れば抜け出すルートのない無限ループとなります。

Comments

[hankechi78]

早速のご回答ありがとうございます！
line = reader.readLine()を忘れていたので追加したところ正常に作動しました。