0

簡単なクライアントとサーバーを用意して、ネットワークについて少し勉強するために作成しました。セットアップ方法は、ソケットの作成/破棄を処理するメインサーバークラスと、各接続を表す ConnectionThread クラス (それぞれに独自のスレッドが与えられている) を持っていることです。クライアントは非常にシンプルです。

問題は、ConnectionThread クラスで入出力ストリームを作成することにあります。問題が何であるか正確にはわかりませんが、単純なテストクライアントが接続しようとするとクラッシュし、次のようになります。

~~MMO Server Alpha .1~~
Constructed Server
Server Initialized, preparing to start...
Server preparing to check if it should be listening...
Server should be listening, continuing as planned.
ServerSocket passed to ConnectionThread: ServerSocket[addr=0.0.0.0/0.0.0.0,localport=6969]
Constructing ConnectionThread.
Socket[addr=/10.0.1.10,port=55332,localport=6969]
ConnectionThread constructed.
Exception in thread "main" java.lang.NullPointerException
    at ConnectionThread.init(ConnectionThread.java:65)
    at Server.listen(Server.java:98)
    at Server.start(Server.java:62)
    at Server.main(Server.java:122)
ConnectionThread added to queue.
Establishing in and out streams:
null

クラスは次のとおりです(簡潔にするために修正されています):

public class Server {
    int PORT;
    boolean shouldListen;
    ArrayList<ConnectionThread> connections = new ArrayList<ConnectionThread>();
    ServerSocket serverSocket;



    public Server() {

        try {
            PORT = 6969;
            shouldListen = true;
            serverSocket = new ServerSocket(PORT);
        }
        catch (IOException e) {

            System.out.println("Error in server constructor.");
            System.exit(1);
        }
    }



    public void start() {
        System.out.println("Server preparing to check if it should be listening...");
        listen();
        System.out.println("Server finished listening.");
    }


    public void listen() {
        while (shouldListen) {
            ConnectionThread conn = null;
            System.out.println("Server should be listening, continuing as planned.");
            try {
            conn = new ConnectionThread(serverSocket);
            }
            catch (Exception e) {
                System.out.println("____Error constructing ConnectionThread. Could there be another instance of the server running?");
                System.exit(1);
            }
            System.out.println("ConnectionThread constructed.");

            connections.add(conn);
            System.out.println("ConnectionThread added to queue.");

            conn.init();
            System.out.println("Finished ConnectionThread initialization, verifying...");

            if (conn.isInitialized) {
                System.out.println("ConnectionThread Initialized, preparing to start new thread.");
                (new Thread(conn)).start();
            }
        }
    }


    public static void main(String[] args) {
        System.out.println("~~MMO Server Alpha .1~~");
        Server server = new Server();
        System.out.println("Constructed Server");
        server.init();
        System.out.println("Server Initialized, preparing to start...");
        server.start();

    }

}

ConnectionThread クラスは次のとおりです。

public class ConnectionThread implements Runnable {
    boolean shouldBeListening = true;
    boolean isThereAnUnsentOutgoingMessage = false;
    String outgoingMessage = "OUTGOING UNINITIALIZED";
    boolean IsThereAnUnsentIncomingMessage = false;
    String incomingMessage = "INCOMING UNITIALIZED";
    boolean isInitialized = false;
    PrintWriter out;
    BufferedReader in;

    String currentInputMessage = "Test Input Message from the Server ConnectionThread";
    String previousInputMessage = null;


    Socket socket;





    public ConnectionThread(ServerSocket s) {
        System.out.println("ServerSocket passed to ConnectionThread: " + s);
        /*
         * The purpose of the constructor is to establish a socket
         * as soon as possible. All transmissions/logic/anything else
         * should happen in init() and/or run().
         */
        System.out.println("Constructing ConnectionThread.");
        try {
        Socket socket = s.accept();
        System.out.println(socket);
        }
        catch (IOException e) {
            System.out.println("Error in ConnectionThread constructor");
            System.exit(1);
        }
    }




    public void init() {
        /*
         * Everything should be set up here before run is called.
         * Once init is finished, run() should be set to begin work.
         * This is to ensure each packet is efficiently processed.
         */
        try {
            System.out.println("Establishing in and out streams:");
            System.out.println(socket);
            out = new PrintWriter(socket.getOutputStream(), true);
            System.out.println("ConnectionThread: Output Stream (PrintWriter) Established");

            in = new BufferedReader(new InputStreamReader(socket.getInputStream()));
            System.out.println("ConnectionThread: InputStream (BufferedReader) Established");
        }
        catch (IOException e) {
            System.out.println("Error in ConnectionThread method Init.");
            System.exit(1);
        }



        isInitialized = true;
    }

オプションで、テスト クライアントを次に示します。

public class TestClient {
    static PrintWriter out;
    BufferedReader in;
    public final int PORT = 6969;
    Socket socket = null;
    InetAddress host = null;

    public TestClient() {
        out = null;
        in = null;
        socket = null;
        host = null;



    }



    public void connectToServer() {
        System.out.println("Connecting to server...");
        try {
            host = InetAddress.getLocalHost();
            socket = new Socket(host.getHostName(), PORT);
        }
        catch (Exception e) {
            System.out.println("Error establishing host/socket");
            System.exit(1);
        }




        try {
            System.out.println("Establishing I/O Streams");
            out = new PrintWriter(socket.getOutputStream(), true);
            in = new BufferedReader(new InputStreamReader(socket.getInputStream()));
        }
        catch (Exception e) {
            System.out.println("Error establishing in/out streams");
            System.exit(1);
        }
    }


    public static void main(String[] args) {
        System.out.println("~~TestClient Alpha .1~~");
        TestClient c = new TestClient();
        c.connectToServer();
        System.out.println("Should be connected to server. Sending test message...");
        while (true) {
            System.out.println("here");
            out.println("Hello there");
        }

    }

}
4

3 に答える 3

1

EJPの回答に加えて、あなたは ConnectionThread.run() メソッドを提供しませんでしたが、メソッドで fields を使用するとin仮定outsocketますrun()。これらのフィールドはvolatileまたはとしてマークされていないためfinal、運とコンピューターのコア数によっては、run() メソッドで NullPointerException が発生する場合もあります。

これは、新しい変数値がキャッシュ間で伝播されない可能性があり、新しいスレッドが変更された値を認識しないためです。

この考えられる問題の説明はこちら - 「volatile」宣言を証明できるコード例を使用する必要があります

于 2013-10-18T22:40:09.610 に答える