来自System.in的BufferedReader输入尝试抛出异常

时间:2013-06-18 17:01:55

标签: java input bufferedreader inputstreamreader

我正在尝试使用BufferedReader从输入中读取a。它第一次工作,但第二次运行我得到一个例外。

john@fekete:~/devel/java/pricecalc$ java frontend.CUI 
> gsdfgd
Invalid command!
> I/O Error getting string:  java.io.IOException: Stream closed
I/O Error: java.io.IOException: java.io.IOException: Stream closed
> I/O Error getting string:  java.io.IOException: Stream closed
I/O Error: java.io.IOException: java.io.IOException: Stream closed
> I/O Error getting string:  java.io.IOException: Stream closed

它只是在循环中继续运行。我一定错过了什么。

public static void main(String args[]) {
            if (args.length == 0) {
                    while (!exit) {
                            try {
                                    exit = processLine(commandLine());
                            } catch (IOException e) {
                                    System.out.println("I/O Error: " + e);
                            }
                    }
                    System.out.println("Bye!");
            } else if (args.length == 1) {
                    String line = new String(args[0]);
                    processLine(line);
            } else {
                    String line = new String(args[0]);
                    for (String np : args) {
                            line = new String(line + " " + np); 
                    }
                    processLine(line);
            }
    }

    static private String commandLine() throws IOException {
            String str = new String();
            try (BufferedReader br = new BufferedReader(new InputStreamReader(System.in))) {
                    System.out.print("> ");
                    str = new String(br.readLine());
                    str = str.trim();
            } catch (IOException e) {
                    System.out.println("I/O Error getting string: "+ str + " " + e);
                    throw new IOException(e);
            }

            return str;
    }

似乎所有关于commandLine()的工作都没有工作,所以我只是把它包括在那里。

1 个答案:

答案 0 :(得分:7)

是的,你在这里关闭了流:

try (BufferedReader br = new BufferedReader(new InputStreamReader(System.in)))

该try-with-resources语句将关闭块末尾的BufferedReader,这将关闭InputStreamReader,这将关闭System.in

在这种情况下你不想这样做,所以只需使用:

// Deliberately not closing System.in!
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));   
try {
    ...
}

这仍然可能不会像你想要的那样,因为BufferedReader可能会消耗(和缓冲)更多的数据。你最好在创建BufferedReader 一次(在调用代码中)并将其传递给方法。

哦,我建议你摆脱这个:

String str = new String();

根本不需要它。这会更好:

String str = "";

但即便如此,这是一项毫无意义的任务。同样,您不需要从readLine()返回的字符串创建新的字符串。只需使用:

return br.readLine().trim();

...在try区块内。此外,在str块中记录catch是没有意义的,因为它将是空的 - 只有在读取该行时才会抛出IOException ...