BufferedReader不一致地挂在我的socket的输入流上

时间:2013-08-22 01:46:31

标签: java http inputstream bufferedreader

我正在编写Java HTTP服务器。我认为整个服务器正在运行,它正在使用线程。但是,我意识到将请求读入BufferedReader的代码片段不能正常工作。

以下是读取传入请求的代码:

private String receive(WebSocket webSocket) throws IOException {
  int chr;
  System.out.println("Receiving!");
  StringBuffer buffer = new StringBuffer();

  while ( (chr = webSocket.in().read() ) != -1) {
    buffer.append((char) chr);
    if ( !webSocket.in().ready())
      break;
  }
  return buffer.toString();
}

我的Websocket类只包装Socket并提供输入和输出。我做了这个,以便我可以模拟套接字并测试我的服务器。

Websocket类看起来像这样:

package http.server.socket;

import java.io.*;
import java.net.Socket;

public class SystemSocket implements WebSocket {
  private Socket theConnection;
  private BufferedReader in;
  private OutputStream out;

  public SystemSocket(Socket theConnection) throws IOException {
    this.theConnection = theConnection;
    in = new BufferedReader(new InputStreamReader(theConnection.getInputStream()));
    out = new BufferedOutputStream(theConnection.getOutputStream());
  }

  public BufferedReader in() throws IOException {
    return in;
  }

  public OutputStream out() throws IOException {
    return out;
  }

  public void close() throws IOException {
    in.close();
    out.close();
    theConnection.close();
  }
}

问题是,对于用户在浏览器中输入的每个URL,会发出两个请求 - 一个用于请求的页面,另一个用于favicon。有时 - 似乎 - 图标请求没有进入并且线程挂起。

以下是我在控制台上打印的一些调试信息当事情正确时

Receiving!
Receiving!
REQUEST STRING = GET /color_picker.html HT
[20130821 20:29:23] REQUEST: http://localhost:5000/color_picker.html
[20130821 20:29:23] PAGE RENDERED
REQUEST STRING = GET /favicon.ico HTTP/1.1
[20130821 20:29:23] REQUEST: http://localhost:5000/favicon.ico
[20130821 20:29:23] PAGE RENDERED

只要读取请求,就会打印“正在接收”消息。因此,在这种情况下,“接收”消息被打印两次,两个请求进入并呈现两件事。但是,相同的页面(但在不同的时间)将执行此操作(大约10秒后)

Receiving!
Receiving!
REQUEST STRING = GET /color_picker.html HTTP/1.1
[20130821 20:41:25] REQUEST: http://localhost:5000/color_picker.html
[20130821 20:41:25] PAGE RENDERED
REQUEST STRING = 
Exception in thread "ServerThread" java.lang.ArrayIndexOutOfBoundsException: 1
  at http.request.Parser.setRequestLineData(Parser.java:42)
  at http.request.Parser.setRequestHash(Parser.java:27)
  at http.request.Parser.parse(Parser.java:13)
  at http.request.Request.get(Request.java:18)
  at http.server.ServerThread.run(ServerThread.java:39)

所有后续错误都是因为请求字符串为null。但我无法弄清楚为什么Request字符串为null。我甚至无法弄清楚如何调试。

任何人都可以帮忙吗?

同样重要的是要注意,如果第二个请求字符串没有立即进入,用户可以请求一个新的URL,它将导致第二个挂起进程完成(这样第四个请求网址就会挂起) 。因此,只有当用户停止请求时,在大约10秒后的最后一个请求中,我才会收到错误。有时我可以请求20个不同的页面,只有在我停止请求页面并等待几秒钟之后,我才会看到错误。我想这就是发生了什么?

更新

根据请求,这里是setRequestLineData()方法:

private void setRequestLineData() {
  requestHash = new HashMap<String, String>();

  if (requestLineParts.length == 3) {
    requestHash.put("httpMethod", requestLineParts[0]);
    requestHash.put("url", requestLineParts[1]);  //line 42
    requestHash.put("httpProtocol", requestLineParts[2]);
  }
  else {
    requestHash.put("httpMethod", requestLineParts[0]);
    requestHash.put("url", requestLineParts[1]);
    requestHash.put("queryString", requestLineParts[2]);
    requestHash.put("httpProtocol", requestLineParts[3]);
  }
}

更新

我想我在导师的帮助下想出了更多关于这里发生了什么的事情。他的想法是,一旦收到请求,浏览器就会立即启动另一个请求,以减少下一个请求的加载时间。这个声音对我来说是合理的,因为我可以一页又一页地加载页面,但是在请求最后一页后我才收到错误只有大约10秒钟。目前,我正在使用自定义异常处理此问题,但我正在研究更好的解决方案。谢谢你所有的帮助!

1 个答案:

答案 0 :(得分:3)

ready()不是消息结束的有效测试。它只会告诉您是否有可以在不阻塞的情况下读取数据。 TCP不是面向消息的协议,它是一种字节流协议。如果你想要消息,你必须自己实现它们,例如作为行,长度值元组,类型长度值元组,序列化对象,XML文档,......

ready()(或available())的正确用法很少,而且不是其中之一。