BufferedInputStream and Blocking

Question

I am using a BufferedInputStream to read from a socket. The BufferedInputStream reads as follows:

socketInput.read(replyBuffer, 0, 7);

It is instantiated by

socketInput = new BufferedInputStream(mySocket.getInputStream());

mySocket is defined as private Socket mySocket;

mySocket is instantiated by mySocket = new Socket(ipAddress, port);

I have verified that mySocket is connected to my device. I can send data to my device; however, I am not receiving from my device for unknown reasons but that is not the problem.

I want my BufferedInputStream to return after say 100ms if it does not read any data. Can the BufferedInputStream be setup to do this? Right now, it blocks indefinitely.

If you want any kind of non-blocking or timeout I/O, you should use the NIO (`java.nio`) classes, not stuff in `java.io`. — C. K. Young, Aug 25 '11 at 14:35

score 5 · Answer 1 · edited Jun 20 '20 at 09:12

5

Specify 100ms timeout for your socket. setSoTimeout

edited Jun 20 '20 at 09:12

Community

1
1

answered Aug 25 '11 at 14:37

umbr

440
2
4

is this still valid if I am using a BufferedInputStream? – Aug 25 '11 at 14:49
1

What will happen is you'll get a `SocketTimeoutException` from your `read()` call, even though there were actually enough bytes to satisfy your request. That's not good. – Ernest Friedman-Hill Aug 25 '11 at 14:55
@Code Monkey it's still valid for any wrapping InputStream. – umbr Aug 25 '11 at 19:16
1

@Ernest Friedman-Hill SocketTimeoutException throws if no data available. – umbr Aug 25 '11 at 19:21

score 4 · Accepted Answer · answered Aug 25 '11 at 14:37

4

It's generally a bad idea to use a buffered stream for reading from a socket, for exactly this reason: it will wait forever if it doesn't see enough data to fill its internal buffer (which is going to be larger than 7 characters!) There's no way to make it time out. Just use the SocketInputStream directly, and the problem goes away.

answered Aug 25 '11 at 14:37

Ernest Friedman-Hill

80,601
10
150
186

so you are recommending instead socket.getInputStream.read(..)? – Aug 25 '11 at 15:06
4

Yes, that's it. Then you can use `setSoTimeout()`, as another poster suggested, and you'll get the behavior you want. – Ernest Friedman-Hill Aug 25 '11 at 15:38

BufferedInputStream and Blocking

2 Answers2