Что здесь может быть неправильным?

Question

Может ли кто-нибудь дать мне, что здесь может быть не так?

import java.io.BufferedInputStream; 
    import java.util.HashMap; 
    import java.util.Map; 
    import java.util.Arrays; 
    import java.io.IOException; 

    public class TopWordsFinder { 

     /** 
     * Map of <word length, <word content, count in text>> Use some different, or 
     * even multiple data structures if it makes more sense to you. 
     */ 
     Map<Integer, Map<String, Integer>> wordsByLength = new HashMap<>(); 

     public static void main(String[] args) { 
     new TopWordsFinder().findTopWords(); 
     } 

     private void findTopWords() { 
     readWords(); // Read words to data structure 
     printTopWords(); // Print words from data structure 
     } 

     // Reads words from file and stores in some data structure 
     // Make sure to check the definition of word in context of this puzzle in 
     // class javadoc 
    private void readWords() { 
     BufferedInputStream in = new BufferedInputStream(TopWordsFinder.class.getResourceAsStream("c:/book-text.txt")); 
     StringBuilder word = new StringBuilder(""); 
     try{ 
      while (in.available() > 0) { 
       char c = (char) in.read(); 
       if(Character.isAlphabetic(c)){ 
        if(Character.isUpperCase(c)){ 
         c = Character.toLowerCase(c); 
        } 
        word.append(c); 
       }else{ 
        saveWord(word.toString()); 
        word.delete(0, word.length()-1);// Reset the sequence 
       } 
      } 
     } catch (IOException e) { 
      System.out.println("Error: "+e); 
     } 
    } 
private void printTopWords() { 
     Integer[] lengths = wordsByLength.keySet().toArray(new Integer[wordsByLength.keySet().size()]); 
     Arrays.sort(lengths); 
     for(int i = lengths.length-1; i>=0; i--){ 
      for(String word : wordsByLength.get(lengths[i]).keySet()){ 
       if(wordsByLength.get(lengths[i]).get(word) >= 3){ 
        System.out.println("Sõna: "+word+" sõna pikkus: "+word.length()+" kordusi: "+wordsByLength.get(lengths[i]).get(word)); 
        return; 
       } 
      } 
     } 
    } 

private void saveWord(String word) { 
    // FILL IN HERE: Store word in data structure you chose 
     if(wordsByLength.get(word.length()) == null){ 
      wordsByLength.put(word.length(), new HashMap<String, Integer>()); 
     } 
     if(wordsByLength.get(word.length()) == null){ 
      wordsByLength.get(word.length()).put(word, 1); 
     }else{ 
      int n = wordsByLength.get(word.length()).get(word); 
      wordsByLength.get(word.length()).put(word, ++n); 
     } 
    } 
} 

Это моя ява работа, но я получаю Error: java.io.IOException: Stream closed.

Answer 1

Кажется, что вы не закрываете поток. Я бы ожидал этого в блоке finally ... Я не могу найти этот блок.

BufferedInputStream in = new BufferedInputStream(TopWordsFinder.class.getResourceAsStream("c:/book-text.txt")); 

Что-то вроде

finally 
{ 
    in.close(); 
} 

Проверьте здесь для получения более подробной информации: http://www.javapractices.com/topic/TopicAction.do?Id=8

Answer 2

Вот ваше исключение IO

}catch (IOException e) { 
    System.out.println("Error: "+e); 
} 

Ваш char c = (char) in.read(); может бросить его когда либо поток само повреждено или так произошла ошибка во время чтения данных, то есть исключений безопасности, разрешенных прав и т. д. и/или набора Исключений, которые получены из IOEXception.

Чтобы знать лучше, вы должны использовать System.out.println("Error: " +e.getMessage()) в вашем catch block

Answer 3

Моя догадка Проблема заключается в in.avaialble() метода.

от doc

IOException происходит в этом способе, если этот входной поток был закрыт , вызывая его метод Close(), или происходит ошибка ввода/вывода.

но вы не закрываете поток. поэтому я думаю, что проблема в линии

BufferedInputStream in = new BufferedInputStream(TopWordsFinder.class.getResourceAsStream("c:/book-text.txt")); 

проверить, если ваш путь к файлу в порядке или есть другая проблема в этой строке.

и всегда закрыть поток после завершения задания (prefereable в finally блоке)

Answer 4

Я думаю, что вы можете использовать сканер вместо этого, ваш код будет легче читать, как это

private void readWords() { 
    FileInputStream in = new FileInputStream("c:/book-text.txt"); 
    Scanner scanner = new Scanner(in); 
    try { 
    while (scanner.hasNext()) { 
     String word = scanner.next(); 
     if (word != null) { 
     saveWord(word.toLowerCase()); 
     } else { 
     break; 
     } 
    } 
    } catch (IOException e) { 
    System.out.println("Error: " + e.getMessage()); 
    e.printStackTrace(); 
    } finally { 
    scanner.close(); 
    } 
} 

Answer 5

Вы должны использовать: -

FileInputStream fin = new FileInputStream("c:/book-text.txt"); 
BufferedInputStream in = new BufferedInputStream(fin); 

Answer 6

Вы получаете сообщение об ошибке от блока следующей попытки прилова

}catch (IOException e) { 
    System.out.println("Error: "+e); 
} 

Причина: Вы не проверяли, правильно ли был открыт файл. Поместите код, чтобы открыть файл в примерочных поймать блок, и если файл не открывается правильно:

private void readWords() { 
     FileInputStream fis = null; 
     BufferedInputStream in = null; 

     try { 
      fis = new FileInputStream(new File("c:/book-text.txt")); 
      in = new BufferedInputStream(fis); 
     } catch (Exception e1) { 
      e1.printStackTrace(); 
      return; // unable to open file, so should not proceed further in 
        // readWords 
     } 

Кроме того, вам нужно исправить код в saveWord метода, поскольку эта строка нулевой проверка не является правильным:

if(wordsByLength.get(word.length()) == null){ 
} 
if(wordsByLength.get(word.length()) == null){ 
} else {} 

Оно должно быть:

private void saveWord(String word) { 
    // FILL IN HERE: Store word in data structure you chose 
    if (wordsByLength.get(word.length()) == null) { 
     wordsByLength.put(word.length(), new HashMap<String, Integer>()); 
     wordsByLength.get(word.length()).put(word, 1); 
    } else { 
     int n = wordsByLength.get(word.length()).get(word); 
     wordsByLength.get(word.length()).put(word, ++n); 
    } 
}