无法关闭它,无法通过Java套接字发送消息

时间:2012-05-06 17:09:02

标签: java sockets

我正在编写一个大型Java应用程序的服务器部分,用于使用Java套接字通过TCP / IP与客户端进行通信。客户端(用PHP编写)连接到服务器,以XML格式发送查询,然后服务器发回响应。查询响应可以在一个连接中重复多次。

服务器端非常简单。它应该允许多个客户端连接,因此有一个线程正在监听并为每个接受的连接生成一个会话。该会话包含一个对象,该对象包含两个用于发送和接收的LinkedBlockingQueues,两个用于使用这些队列和处理线程发送和接收消息的线程。

问题是任何消息实际上只有在套接字关闭后才会传输。响应消息在没有问题的情况下进入消息队列和PrintStream.println()方法,但仅当客户端关闭其侧面的连接时,wireshark才会报告传输。创建启用了自动刷新或使用flush()的PrintStream不起作用。关闭服务器端的套接字也不起作用,服务器仍然可以运行并接收消息。

此外,当前在服务器端接收查询的客户端的实现工作正常,同样来自本地Linux虚拟机的echo -e "test" | socat - TCP4:192.168.37.1:1337,但当我telnet到服务器并尝试发送内容然后服务器在关闭telnet客户端之前没有收到任何内容,与上述问题相同。

相关的服务器代码(整个应用程序太大了,无法粘贴所有内容,而且我正在使用很多其他人的代码):

package Logic.XMLInterfaceForClient;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.PrintStream;
import java.net.Socket;
import java.util.HashSet;
import java.util.concurrent.LinkedBlockingQueue;

import Data.Config;
import Logic.Log;

public class ClientSession {

    /**
     * @author McMonster
     * 
     */
    public class MessageTransmitter extends Thread {

        private final Socket socket;
        private final ClientSession parent;

        private PrintStream out;

        /**
         * @param socket
         * @param parent
         */
        public MessageTransmitter(Socket socket, ClientSession parent) {
            this.socket = socket;
            this.parent = parent;
        }

        /*
         * (non-Javadoc)
         * 
         * @see java.lang.Runnable#run()
         */
        @Override
        public void run() {
            try {
                out = new PrintStream(socket.getOutputStream(), true);

                while (!socket.isClosed()) {
                    try {
                        String msg = parent.transmit.take();
                        // System.out.println(msg);
                        out.println(msg);
                        out.flush();
                    }
                    catch(InterruptedException e) {
                        // INFO: purposefully left empty to suppress spurious
                        // wakeups
                    }
                }

            }
            catch(IOException e) {
                parent.fail(e);
            }

        }

    }

    /**
     * @author McMonster
     * 
     */
    public class MessageReceiver extends Thread {

        private final Socket socket;
        private final ClientSession parent;

        private BufferedReader in;

        /**
         * @param socket
         * @param parent
         */
        public MessageReceiver(Socket socket, ClientSession parent) {
            this.socket = socket;
            this.parent = parent;
        }

        /*
         * (non-Javadoc)
         * 
         * @see java.lang.Runnable#run()
         */
        @Override
        public void run() {
            try {
                in = new BufferedReader(new InputStreamReader(socket.getInputStream()));

                while (!socket.isClosed()) {
                    String message = "";
                    String line;

                    while ((line = in.readLine()) != null) {
                        message = message + line + "\n";
                    }

                    if(message != "") {
                        parent.receive.offer(message.toString());
                    }
                }
            }
            catch(IOException e) {
                parent.fail(e);
            }

        }
    }

    public final LinkedBlockingQueue<String> transmit = new LinkedBlockingQueue<>();
    public final LinkedBlockingQueue<String> receive = new LinkedBlockingQueue<>();

    private final XMLQueryHandler xqh;
    private final Socket socket;

    private String user = null;
    private HashSet<String> privileges = null;

    /**
     * @param socket
     * @param config
     * @throws IOException
     * @throws IllegalArgumentException
     */
    public ClientSession(Socket socket, Config config)
            throws IOException,
            IllegalArgumentException {
        // to avoid client session without the client
        if(socket == null) throw new IllegalArgumentException("Socket can't be null.");

        this.socket = socket;

        // we do not need to keep track of the two following threads since I/O
        // operations are currently blocking, closing the sockets will cause
        // them to shut down
        new MessageReceiver(socket, this).start();
        new MessageTransmitter(socket, this).start();

        xqh = new XMLQueryHandler(config, this);
        xqh.start();
    }

    public void triggerTopologyRefresh() {
        xqh.setRefresh(true);
    }

    public void closeSession() {
        try {
            xqh.setFinished(true);
            socket.close();
        }
        catch(IOException e) {
            e.printStackTrace();
            Log.write(e.getMessage());
        }
    }

    /**
     * Used for reporting failures in any of the session processing threads.
     * Handles logging of what happened and shuts down all session threads.
     * 
     * @param t
     *            cause of the failure
     */
    synchronized void fail(Throwable t) {
        t.printStackTrace();
        Log.write(t.getMessage());
        closeSession();
    }

    synchronized boolean userLogin(String login, HashSet<String> privileges) {
        boolean success = false;

        if(!privileges.isEmpty()) {
            user = login;
            this.privileges = privileges;
            success = true;
        }

        return success;
    }

    public synchronized boolean isLoggedIn() {
        return user != null;
    }

    /**
     * @return the privileges
     */
    public HashSet<String> getPrivileges() {
        return privileges;
    }
}

2 个答案:

答案 0 :(得分:1)

它与发送完全没有任何关系。更准确地说,如果你直到流结束,那么在对等体通过关闭套接字结束流之前你不会得到流的结束。

这是重言式。

答案 1 :(得分:0)

我可以理解为什么在套接字关闭之前服务器上没有收到任何消息 - 这似乎是设计的。 in.readLine()仅在到达流的末尾时返回null,使用TCP套接字流表示套接字关闭时。如果您希望readLine()循环在此之前返回,则循环中的代码必须使用您在TCP上使用的任何协议来检测消息的结尾以定义消息。