JavaのサーバープログラムでSocketオブジェクトへの参照を渡す

Question

Javaでマルチスレッドプログラムを作成しています。ここでは、新しいクライアント接続を処理するために別のスレッドを作成しています。私持っている：私は持っているClientHandlerのスレッドで

Socket s; 


while(true) 
{ 
    s = server.accept(); 
    ClientHandler ch = new ClientHandler(s); 
    Thread servingThread = new Thread(ch); 
    servingThread.start(); 
} 

：

public class ClientHandler implements Runnable 
{ 
    private Socket s; 

public ClientHandler(Socket _Socket, boolean _accepted, BaseStation _bs) 
{ 
    this.s = _Socket; 
} 

をJavaで私は、オブジェクトが、それへの参照のみを渡すことができない場合は、それが問題を引き起こすことが起こっていませんserverが新しい接続を受け入れるたびに、ClientHandler内のsインスタンスに接続しますか？ ClientHandlerの内部でも変更されず、破損しますか？もしそうなら、正しい方法は何ですか？

Answer 1

Socketオブジェクト（Javaの他のオブジェクトと同様）は参照によって渡されるため、異なるモジュールまたは異なるスレッドによって参照される可能性があり、予測できない動作を引き起こす可能性があります。

ただし、プログラムの場合は問題ありません。理由は：サーバーが新しい接続を受け入れるたびに、新しいSocketというオブジェクトが作成され、の新しいのインスタンスClientHandlerに渡されます。したがって、クライアントからの新しい接続はすべて、独立したClientHandlerインスタンスによって提供されるため、Socketオブジェクトが心配する競合条件は存在しません。あなたは今のところ安全です。反例として

あなたはこのような同じSocketオブジェクトから読み取るための2つのClientHandlerスレッドを作成することにした場合...

s = server.accept(); 
ClientHandler ch = new ClientHandler(s); 
ClientHandler ch2 = new ClientHandler(s); 
new Thread(ch).start(); 
new Thread(ch2).start(); 

... 2つのClientHandler sが同じSocketオブジェクトへ同じの参照を取得し、両方の試みは、同じソケットから読み取る場合、彼らはそれぞれが半分だけを取得しますので、あなたはトラブルになる可能性がありますデータの例えば。文字列"hello"を受け取った場合は、chと表示され、"hel"とch2と表示され、"lo"と表示されます。

Answer 2

少し混乱しますが、コードは表示されているとおりに動作するはずです。 Javaは最初に値渡しされ、次に参照渡しされます。 I.E.では、 "Socket s"へのポインタを渡すのではなく、実際の値 "s"を渡しています。

混乱を避けるため、「Socket s;」という宣言を削除し、ループの最初の行を「Socket s = server.accept();」にします。

Answer 3

説明するにはIs Java "pass-by-reference" or "pass-by-value"?を参照してください。 ClientHandler内Socketインスタンスのコピーに影響を与えていないローカル変数sの値に変更 - 各ClientHandlerは独自のSocketを持っているため、新しいClientHandlerインスタンスに新しいSocketインスタンスを渡す

は大丈夫です。同じ基本Socketオブジェクトインスタンスに

ClientHandler ch = new ClientHandler(s); 
Thread servingThread = new Thread(ch); 
servingThread.start(); 

s.someMethod(); // close, read etc. 

ch.sとsので、両方のポイント：このよう

何かが問題を引き起こします。

ところで、新しいスレッド全体をwhileループで回転させるのは、それらのスレッドが終了することが確実でない限り、おそらく悪いです。おそらくThreadPoolを使うほうが良いでしょう。