0

Java ソケットを使用して TCP/IP 経由でクライアントと通信するための大規模な Java アプリケーションのサーバー部分を作成しています。クライアント (PHP で記述) がサーバーに接続し、XML 形式でクエリを送信すると、サーバーが応答を返します。クエリ応答は、1 つの接続で複数回繰り返すことができます。

サーバー側は非常に単純です。複数のクライアント接続を許可することになっているため、受け入れられた接続ごとにセッションをリッスンして生成するスレッドがあります。セッションは、送受信用の 2 つの LinkedBlockingQueues、それらのキューを使用してメッセージを送受信するための 2 つのスレッド、および処理スレッドを含むオブジェクトで構成されます。

問題は、ソケットが閉じられた後にのみメッセージが実際に送信されることです。応答メッセージは問題なくメッセージ キューと PrintStream.println() メソッドに入りますが、wireshark は、クライアントが側で接続を閉じた場合にのみ送信を報告します。自動フラッシュを有効にしたり、flush() を使用して PrintStream を作成したりしても機能しません。サーバー側でソケットを閉じても機能しません。サーバーは引き続き機能し、メッセージを受信します。

また、サーバー側でクエリを受信するクライアントの現在の実装では正常に動作しますecho -e "test" | socat - TCP4:192.168.37.1:1337。ローカルの Linux 仮想マシンからも同じことが言えますが、サーバーに 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;
    }
}
4

2 に答える 2

2

送信とはまったく関係ありません。ストリームの終わりまで読んでいる場合、ソケットを閉じてピアがストリームを終了するまで、ストリームの終わりを取得しないと言う方が正確です。

これはトートロジーです。

于 2014-07-29T08:28:05.650 に答える
1

ソケットが閉じられるまでサーバーでメッセージが受信されない理由を理解できます-それは設計どおりのようです。in.readLine() は、TCP ソケット ストリームの場合、ソケットが閉じられたときを意味する、ストリームの終わりに到達したときにのみ null を返します。その前に readLine() ループを返したい場合、ループ内のコードは、メッセージを定義するために TCP 上で使用しているプロトコルを使用して、メッセージの終わりを検出する必要があります。

于 2012-05-06T17:29:54.473 に答える