Java初心者：ファイル内の特定のテキストを検索する無限ループ

Question

文字列のいくつかの行で構成されるテキストファイル内の特定の行に文字列を配置する必要があります。しかし、テキストやファイルの終わりを見つけるための私のループは、永遠に検索しています。私は文字列がファイル内にあることを知っています。ここでは、テキストを見つけるために使用しているコードですが、シンプルなテキストファイルであってもシステム上で試してみると、永遠のループに入ります。

私がここで間違っていることを説明するためのヒントやポインターをありがとう。

private static void locateText(String locateText, BufferedReader locateBffer) { 
    boolean unfound = true; 
    try 
    { 
     String line = locateBffer.readLine(); 
     while (unfound) 
     { 
      line = locateBffer.readLine(); 
      if ((line.equals(locateText)) || (line == null)) 
      { 
       unfound = false; 
      } 
     } 
    } 
    catch(IOException e) 
    { 
     System.out.println("I/O error in locateText"); 
    } 
} 

更新：問題を発見 - それは、ファイルの最初の行に一致するものを見つけていませんでした。

Answer 1

テキストは最初の行にありますか？あなたはあなたのループの外でreadLine操作を行い、その後、最初の行は基本的に無視されます。

Answer 2

変更そのようなことに、このループ、それがすべての行を読み込みます：

while((line = locateBffer.readLine()) != null){ 
if(line.equals(locateText)){ 
    break; 
} 
} 

は多分それが役立ちます。

Answer 3

私はGaryFが正しいと思います（あなたのテキストはファイルの最初の行にあります）。

私はあなたのコード内の行を指すように望んでいた：あなたの代わりにこの書き込みをしなければならない

if ((line.equals(locateText)) || (line == null)) { 

：確かに

if ((line == null) || (line.equals(locateText)) { 

をラインがnullの場合、あなたのコードは、NullPointerExceptionがスローされます。そのため、前にlineがnullであるかどうかをテストする必要があります。それに加えて

、私はそれが正の条件の後に名前のブールに通常より読みやすいです