Question

我正在创建一个Java（版本控制）应用程序，其中客户端和服务器通过套接字流进行通信。我遇到的问题是客户端或服务器在执行readline（）时往往会阻塞，最终导致另一方也阻塞。奇怪的是，第一次运行时似乎没有出现错误。

我花了很多时间搞清楚问题的原因以及如何解决问题，但我只能将问题强烈地减少到下面提供的代码中。 MAKE_STUB参数允许使用管道流而不是套接字流。它通常会导致在客户端完成读取之前关闭线程时导致的错误，但有时会出现与使用套接字时相同的错误，这可能表明问题不是由于套接字的误用造成的，而是滥用流？

我还尝试了以下方法：更改输出刷新时，在println（）中添加+“\ n”，依次发送/接收（PING_PONG），防止流被垃圾回收（使用静态列表），发送一个字符串而不是JSON等。我发现很奇怪这些解决方案只会导致块发生得更早或更晚。

客户代码：

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.OutputStream;
import java.io.PipedInputStream;
import java.io.PipedOutputStream;
import java.io.PrintWriter;
import java.io.Writer;
import java.net.InetAddress;
import java.net.InetSocketAddress;
import java.net.Socket;
import java.net.UnknownHostException;


public class TestClientProt {

private static final boolean MAKE_STUB = false;
private static final boolean PING_PONG = false;


private static void send(String value, InputStream rawInput, OutputStream rawOutput) throws IOException {
    PrintWriter output = new PrintWriter(rawOutput);
    output.println(value);
    output.flush();

    if(PING_PONG) {
        BufferedReader input = new BufferedReader(new InputStreamReader(rawInput));
        if(!input.readLine().equals(value))
            System.err.println("Ping pong failed!");
    }
}


private static int receiveInt(InputStream rawInput, OutputStream rawOutput) throws NumberFormatException, IOException {
    return Integer.parseInt(receiveString(rawInput, rawOutput));
}

private static String receiveString(InputStream rawInput, OutputStream rawOutput) throws IOException {
    BufferedReader input = new BufferedReader(new InputStreamReader(rawInput));
    String value = input.readLine();

    if(PING_PONG) {
        PrintWriter output = new PrintWriter(rawOutput);
        output.println(value);
        output.flush();
    }

    return value;
}


public static void main(String[] args) throws IOException {
    Socket socket;

    InputStream rawInput;
    OutputStream rawOutput;

    if(MAKE_STUB) {

        rawInput = new PipedInputStream();
        final OutputStream rawServerOutput = new PipedOutputStream((PipedInputStream) rawInput);

        final PipedInputStream rawServerInput = new PipedInputStream();
        rawOutput = new PipedOutputStream(rawServerInput);

        new Thread() {

            public void run() {
                try {
                    TestServerProt.startSession(rawServerInput, rawServerOutput);
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }

        }.start();


    } else {

        InetAddress ip = InetAddress.getByName("127.0.0.1");
        int port = 6789;

        InetSocketAddress serverAddress = new InetSocketAddress(ip, port);      
        socket = new Socket();
        socket.connect(serverAddress);

        rawInput = socket.getInputStream();
        rawOutput = socket.getOutputStream();

    }


    System.out.println("start checkout");

    send("CHKT repname", rawInput, rawOutput);
    System.out.println(receiveInt(rawInput, rawOutput));

    send("RREV", rawInput, rawOutput);
    System.out.println(receiveInt(rawInput, rawOutput));
    System.out.println("receiving revision");
    System.out.println(receiveString(rawInput, rawOutput));
    System.out.println("revision received");

    System.out.println(receiveInt(rawInput, rawOutput));
    System.out.println("receiving files");

    send("RFIL", rawInput, rawOutput);
    System.out.println(receiveInt(rawInput, rawOutput));
    System.out.println("receiving file: ");
    System.out.println(receiveString(rawInput, rawOutput));

    System.out.println("success!");


    if(!MAKE_STUB)
        socket.close();
}

}

服务器代码：

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.OutputStream;
import java.io.PrintWriter;
import java.net.InetSocketAddress;
import java.net.ServerSocket;
import java.net.Socket;

import org.json.simple.JSONArray;
import org.json.simple.JSONObject;


public class TestServerProt {

private static final boolean PING_PONG = false;


public static void startSession(InputStream rawInput, OutputStream rawOutput) throws IOException {
    System.out.println("starting session");
    System.out.println(receiveString(rawInput, rawOutput));

    send(101, rawInput, rawOutput);

    System.out.println(receiveString(rawInput, rawOutput));
    System.out.println("sending revision");
    send(104, rawInput, rawOutput);
    send(createRepString(), rawInput, rawOutput);

    System.out.println("sending files");
    send(101, rawInput, rawOutput);
    System.out.println("sending files");

    System.out.println(receiveString(rawInput, rawOutput));
    send(103, rawInput, rawOutput);
    send("content file 1", rawInput, rawOutput);

    System.out.println("success!");

    //send(rCode);
}


private static void send(String value, InputStream rawInput, OutputStream rawOutput) throws IOException {
    PrintWriter output = new PrintWriter(rawOutput);
    output.println(value);
    output.flush();

    if(PING_PONG) {
        BufferedReader input = new BufferedReader(new InputStreamReader(rawInput));
        if(!input.readLine().equals(value))
            System.err.println("Ping pong failed!");
    }
}

private static void send(int value, InputStream rawInput, OutputStream rawOutput) throws IOException {
    send(String.valueOf(value), rawInput, rawOutput);
}


private static int receiveInt(InputStream rawInput, OutputStream rawOutput) throws NumberFormatException, IOException {
    return Integer.parseInt(receiveString(rawInput, rawOutput));
}

private static String receiveString(InputStream rawInput, OutputStream rawOutput) throws IOException {
    BufferedReader input = new BufferedReader(new InputStreamReader(rawInput));
    String value = input.readLine();

    if(PING_PONG) {
        PrintWriter output = new PrintWriter(rawOutput);
        output.println(value);
        output.flush();
    }

    return value;
}


public static void main(String[] args) throws IOException {
    int port = 6789;

    InetSocketAddress serverAddress = new InetSocketAddress(port);
    ServerSocket serverSocket = new ServerSocket();
    serverSocket.bind(serverAddress);

    System.out.println("waiting for client");

    Socket clientSocket = serverSocket.accept();

    System.out.println("client accepted");

    startSession(clientSocket.getInputStream(), clientSocket.getOutputStream());

    serverSocket.close();
    clientSocket.close();
}

private static String createRepString() {
    JSONObject json = new JSONObject();

    json.put("revision_nr", new Integer(5));

    JSONArray jsonArrayContent = new JSONArray();
    JSONObject jsonContent = new JSONObject();

    jsonContent.put("filename", "f1.txt");
    jsonContent.put("revision_nr", 3);
    jsonArrayContent.add(jsonContent);

    jsonContent.put("filename", "f2.txt");
    jsonContent.put("revision_nr", 1);
    jsonArrayContent.add(jsonContent);

    jsonContent.put("filename", "f3.txt");
    jsonContent.put("revision_nr", 5);
    jsonArrayContent.add(jsonContent);

    json.put("content", jsonArrayContent);

    JSONArray jsonArrayStatus = new JSONArray();
    JSONObject jsonStatus = new JSONObject();

    json.put("status", jsonArrayStatus);

    return json.toJSONString();
}

}

Answer 1

在您的情况下，最重要的问题是每次阅读时都会创建一个新的BufferedReader。问题是BufferedReader将用可用数据填充整个缓冲区 - 因为它应该这样做。因此它将从超出行尾的基础流中读取。

解决方案很简单：只创建BufferedReader一次，然后传递它，而不是InputStream。

在Java输入/输出流上使用BufferedReader时，readline（）会阻塞

1 个答案: