How to pipe input to Java program with bash

Question

My Java program is listening on standard input:

InputStreamReader isReader = new InputStreamReader(System.in);
BufferedReader bufReader = new BufferedReader(isReader);
while(true){
    try {
        String inputStr = null;
        if((inputStr=bufReader.readLine()) != null) {
            ...
        }
        else {
            System.out.println("inputStr is null");
        }
    }
    catch (Exception e) {
        ...
    }
}

Now, I want to pipe input to this program from bash. I tried the following:

echo "hi" | java -classpath ../src test.TestProgram

But it's just printing inputStr is null infinite times. What am I doing wrong?

Edit 1: Updated question to include more code / context.

Edit 2:

Looks like I'm experiencing the same issue as this OP: Command Line Pipe Input in Java

How can I fix the program so that I can pipe input in for testing, but running the program normally will allow users to enter input on standard input as well?

Your code worked perfectly for me. What else are you doing in your non-posted code? — Nishan
– Nishan, Commented Apr 20, 2011 at 1:46
Any chance that the test.TestProgram you're running is not the one you're compiling? — trashgod
– trashgod, Commented Apr 20, 2011 at 1:53
I use Scanner instead of BufferedReader - I doubt it would make a difference though. — corsiKa
– corsiKa, Commented Apr 20, 2011 at 1:56
I've updated the question to include more code. I guess simplification wasn't a good idea in this case. — Chetan
– Chetan, Commented Apr 20, 2011 at 2:03

slim · Accepted Answer · 2012-08-29 10:17:22Z

9

You have while(true), so infinite looping is what you're going to get.

Adding a break somewhere in the loop is one way to fix it. But it's not good style, because the reader has to hunt around the loop to find out if and when it exits.

It's much better to make your while statement show clearly what the exit condition is:

String inputStr = "";
while(inputStr != null) {
    inputStr=bufReader.readLine(); 
    if(inputStr != null) {
        ...
    } else {
        System.out.println("inputStr is null");
    }
}

answered Aug 29, 2012 at 10:17

slim

41.5k14 gold badges101 silver badges131 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Chetan · Accepted Answer · 2011-04-20 02:13:13Z

6

Fixed it. After the piping of input was completed, readLine() kept returning null, so the infinite loop kept looping.

The fix is to break from the infinite loop when readLine() returns null.

answered Apr 20, 2011 at 2:13

Chetan

48.4k32 gold badges110 silver badges146 bronze badges

Comments

Armand · Accepted Answer · 2014-04-24 16:05:20Z

I like slim's answer, only I tend handle it a bit differently. this is a basic template I use for reading a stream of text, line by line.

try {
    // Wrap the System.in inside BufferedReader
    // But do not close it in a finally block, as we 
    // did no open System.in; enforcing the rule that
    // he who opens it, closes it; leave the closing to the OS.
    BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
    String line;
    while ((line = in.readLine()) != null) {
        // TODO: Handle input line
    }

    // Null was received, so loop was aborted.

} catch (IOException e) {
    // TODO: Add error handler
}

If I am reading a file, I change it slightly, to close the file like this

try {
    File file = new File("some_file.txt");

    // Wrap the System.in inside BufferedReader
    // But do not close it in a finally block, as we
    // did no open System.in; enforcing the rule that
    // he who opens it, closes it; leaves the closing to the OS.
    BufferedReader in = new BufferedReader(new FileReader(file));
    try {
        String line;
        while ((line = in.readLine()) != null) {
            // TODO: Handle input line
        }

        // Null was received, so loop was aborted.

    } finally {
        try {
            in.close();
        } catch (IOException e) {
        }
    }

} catch (IOException e) {
    // TODO: Add error handler
}

Stephen C · Accepted Answer · 2011-04-20 01:58:10Z

What am I doing wrong?

I see no reason why the snippet would behave that way. I suspect that the problem is in something that you haven't shown us ...

For instance, which version of echo are you using? The shell builtin? The standard one in '/bin'? Some funky one on your search path?

You could try some simple experiments to determine if the problem is at the shell / command level or within the Java application; e.g.

$ echo hi > tmp
$ cat tmp
$ java -classpath ../src test.TestProgram < tmp
$ cat tmp | java -classpath ../src test.TestProgram

etcetera.

If none of those experiments yield any clues, post the real Java source code of a small program that demonstrates your problem.

(And as @trashgod rightly points out, you might have "fat fingered" the build step, and be running a version of the program that no longer matches your source code.)

Community · Accepted Answer · 2017-05-23 12:17:39Z

2

You may consider using named pipes (fifos) to allow both normal input via the controlling terminal /dev/tty (or /dev/stdin) and piped input via an input fifo.

See: Redirecting input of application (java) but still allowing stdin in BASH

edited May 23, 2017 at 12:17

CommunityBot

11 silver badge

answered Apr 20, 2011 at 14:43

jon

211 bronze badge

Collectives™ on Stack Overflow

How to pipe input to Java program with bash

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related