メソッドの引数の所有権を 'リソースで試す'ブロックに渡します。

Question

Javaの「リソースを試そう」機能に関する質問があります。ほとんどの例では、リソースが次のようにtry文の中で宣言してインスタンス化されたモデルに従っているように見える：

try (BufferedReader in = 
     new BufferedReader(new InputStreamReader(socket.getInputStream()));) { 
    ... 
} 

私はメソッドに開閉可能なリソースを渡す必要があり、それが可能か、賢明だった場合、私は思っていましたメソッドの引数としてtryブロックに渡されたクローズ可能なリソースの所有権を渡します。例：

void handleConnection(Socket clientSocket) { 
    try (Socket socket = clientSocket; 
     BufferedReader in = 
      new BufferedReader(new InputStreamReader(socket.getInputStream()));) { 
     ... 
    } 

これで適切にclientSocketインスタンスがクリーンアップされますか？私は明示的に私のコードでclientSocketインスタンスを閉じることを避けることを望んでいます。

Answer 1

これは技術的には正しいものですが、危険です。 AutoCloseable（Socketを含む）の多くは、閉鎖されてから再開できません。だから、あなたのhandleConnectionメソッドのユーザーは、あなたのメソッドに渡されたオブジェクトはもはや使用できないことに注意しなければなりません。おそらく、オブジェクトの所有者にライフサイクルを処理させることをお勧めします。たとえば、以下のように、それ自体がAutoCloseableを実装するラッパークラスを作成することによって実現できます。

public class SocketHandler implements AutoCloseable { 
    private final Socket clientSocket; 

    public SocketHandler(Socket clientSocket) { 
     if (clientSocket == null) { 
      throw new IllegalArgumentException("Socket cannot be null"); 
     } 
     this.clientSocket = clientSocket; 
    } 

    public void handle() { 

    } 

    @Override 
    public void close() throws Exception { 
     clientSocket.close(); 
    } 

}