このコードで何が間違っていますか？

Question

ディレクトリのファイルから内容を読み取っています。私はファイルを名前に応じて分離し、その内容を読まなければなりません。コンテンツを読み込まずにコードを実行すると、すべてのファイルが特定のファイル名にリストされますが、コンテンツを読み込もうとすると、ほんのわずかなファイルからコンテンツが読み込まれます。しかし、ディレクトリには約1000の特定の名前のファイルがあります。私はここにコードを掲載しています。

for (i = 0; i <= filenames.length; i++) { 
    read = new FileReader("trainfiles/"+filenames[i]);   
    br = new BufferedReader(read); 

    if (filenames[i].matches(".*ham.*")) { 
     System.out.println("ham:" + filenames[i]); 
     while ((lines = br.readLine()) != null) { 
      st = new StringTokenizer(lines); 
      while (st.hasMoreTokens()) { 
       System.out.println(st.nextToken()); 
      } 
     } 
     br.close(); 
    } 
} 

誰かが私に間違っていると教えてもらえますか？
おかげ

EDIT＃1は、私がここに言われているいくつかの修正をしましたが、問題が解決しない、ここにコードがあります。

for(i=0;i<=filenames.length;i++){ 
      read = new FileReader("trainfiles/"+filenames[i]); 

      br = new BufferedReader(read); 

      if(filenames[i].matches(".*ham.*")){ 
       System.out.println("ham:"+filenames[i]); 

         while((lines = br.readLine())!= null){ 
          st = new StringTokenizer(lines); 
          while(st.hasMoreTokens()){ 
           System.out.println(st.nextToken()); 
          } 

         } 

      } 
      br.close(); 
      read.close(); 




         } 

EDIT＃2は今、コードは次のように見えますが、再び...そのは私が望む結果を与えていません。

for (i = 0; i < filenames.length; i++) { 
       try { 


       if (filenames[i].matches(".*ham.*")) { 
        read = new FileReader("trainfiles/"+filenames[i]);   
         br = new BufferedReader(read); 
        System.out.println("ham:" + filenames[i]); 
        while ((lines = br.readLine()) != null) { 
         st = new StringTokenizer(lines); 
         while (st.hasMoreTokens()) { 
          System.out.println(st.nextToken()); 
         } 
        } 
       } 
       } finally { 

       read.close(); 
       br.close(); 
       } 
      } 

Answer 1

私はこのような再書き込みあなたのコードだろう、とあなたは何を得るの出力を参照してください。

for (filename : filenames) { 
    if (filename.matches(".*ham.*")) { 
     System.out.println("ham:" + filename); 

     // reset these to null (where are they declared?) 
     read = null; 
     br = null; 
     try { 
     read = new FileReader("trainfiles/"+filename);   
     br = new BufferedReader(read); 

     while ((lines = br.readLine()) != null) { 
      System.out.println(lines); 
      // st = new StringTokenizer(lines); 
      // while (st.hasMoreTokens()) { 
      // System.out.println(st.nextToken()); 
      // } 
     } 
     } catch (Exception e) { 
     e.printStackTrace(); 
     } finally { 
     if (br != null) br.close(); 
     if (read != null) read.close(); 
     } 
    } 
} 

あなたの元のコードのいくつかの一般的なコメント：

1. のみ使用実際に配列のインデックスが必要な場合はループします。for for-eachループを好む（つまり、for (filename : filenames) ...）。

2. 可能な限り最も狭い範囲の変数を宣言します。この場合、私はnullに初期化するあなたのreadとbr変数を宣言する必要があります。

3. ファイルを使用する場合を除き、決してファイルを開かないでください。ここでは、の中にという条件ブロックを開きます。

4. ファイルを開くと例外がスローされる可能性があるため、brが初期化されない可能性があります。その場合はcloseにはなりません。最初にnullを確認する必要があります。

Answer 2

あなたにも、あなたのFileReaderオブジェクトreadを閉じる必要があります。

これが宿題でない限り、commons-ioをご覧ください。

EDIT＃1： finallyブロックで両方のクローズ操作を行うことをお勧めします。

編集＃2：これを試しましたか？

for (i = 0; i <= filenames.length; i++) { 
    try { 
    read = new FileReader("trainfiles/"+filenames[i]);   
    br = new BufferedReader(read); 

    if (filenames[i].matches(".*ham.*")) { 
     System.out.println("ham:" + filenames[i]); 
     while ((lines = br.readLine()) != null) { 
      st = new StringTokenizer(lines); 
      while (st.hasMoreTokens()) { 
       System.out.println(st.nextToken()); 
      } 
     } 
    } 
    } finally { 
    br.close(); 
    read.close(); 
    } 
} 

Answer 3

まず、i<filenames.lengthを使用してください。次に、matchesは、正規表現ではなく、*のグロブを必要とします。使用した表現は[something]ham[something]の有効な正規表現です - これは意味ですか？

私はあなたがFilereaderを閉じる必要はないと思います - 私はBRのcloseが伝播すると思います。しかしそれはチェックの価値がある。 EDITこのように、ifの外側でファイルを閉じる必要があります。

Answer 4

1000+ファイルは、読み込むファイルがたくさんあります。ファイルを読み取れない場合は、例外（IOExceptionを特定）をスローする必要があります。おそらく例外メッセージをキャッチブロックに印刷して、ここに貼り付けてください。

StringTokenizerクラスはわかりませんが、StringTokenizerを使用しないで行を印刷するとエラーが発生しますか？

スレッドを使用することもできます。あなたはファイルの配列を持っていて、次にファイルを読み込むスレッドをいくつか起動します（プロデューサ/コンシューマの問題）。

ところで、FileFilterクラスでファイルをフィルタリングすることができます。

http://download.oracle.com/javase/1.4.2/docs/api/java/io/File.html#listFiles%28java.io.FileFilter%29