что с этим кодом?

Question

Я читаю содержимое из файлов каталога. Я должен отделить файлы в соответствии с их именами, а затем прочитать их содержимое. Когда я запускаю код просто, не читая содержимое, все файлы перечисляются в определенном имени файла, но когда я пытаюсь прочитать содержимое, он считывает содержимое из нескольких файлов, на самом деле всего 10 из них. Но каталог содержит около 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. Только использование a for, если вам действительно нужен индекс массива. Предпочтительно для каждого цикла (то есть for (filename : filenames) ...).

2. Объявлять переменные в самом узком объеме. В этом случае вы должны объявить переменные read и br, где я инициализирую их до null.

3. Никогда не открывайте файл, если вы его не используете. Здесь это означает его открытие внутри условный блок.

4. С момента открытия файла может возникнуть исключение, br может не получиться инициализированным, и в этом случае вы не сможете его получить. close. Сначала необходимо проверить на null.

Answer 2

Вы должны закрыть FileReader объект read, а также.

Если это не домашнее задание, я бы также предложил вам взглянуть на commons-io.

РЕДАКТИРОВАТЬ # 1: Я бы предложил выполнить обе операции закрытия в блоке finally.

EDIT # 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 ожидает регулярное выражение, а не * -globs. Вы использовали правильное выражение для [something]ham[something] - это то, что вы имели в виду?

Я не думаю, что вам нужно закрыть Filereader - я думаю, что BR's close распространяется вверх. Но это стоит проверить. EDIT Как уже упоминалось, вам нужно всегда закрывать файл, вне if.

Answer 4

1000+ файлов много файлов для чтения. Если он не может прочитать файл, он должен исключить исключение (особенно для IOException). Возможно, напечатайте сообщение об исключении в блоке catch и вставьте его здесь.

Я не знаю класс StringTokenizer, но код дает ошибки, когда вы просто печатаете строку без StringTokenizer?

Другой вариант - использовать потоки. У вас есть массив файлов, а затем вы запускаете некоторые потоки, которые читают файл (проблема производителя/потребителя).

Кстати, вы можете фильтровать файлы с помощью класса FileFilter.

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