Javaで非最終オブジェクトで同期する正しい方法

Question

ロックでオブジェクトを保護しようとしています。

"try..catch"という醜い構文のため、私はmutexを選択しませんでした。私は読みやすさを向上させることができますどのように...

class MyClass { 
    private final Object lock = new Object(); 
    private Channel channel = null; 

    public void setChannel() { 
     synchronized (lock) { 
      channel = new Channel(); 
      synchronized (channel) { 
       // setup channel 
      } 
     } 
    } 

    public void unsetChannel() { 
     synchronized (lock) { 
      synchronized (channel) { 
       channel.close(); 
      } 
     channel = null; 
     } 
    } 

    public boolean isSet() { 
     synchronized (lock) { 
      if (channel == null) 
       return false; 
      synchronized (channel) { 
       return channel.isActive(); 
      } 
     } 
    } 
} 

しかし、それは醜いと読みにくいようだ：

閲覧StackOverflowのは、私は、これが適切に私の目標を達成する方法であるという結論に達しました私の解決策？

Answer 1

あなたのロック方法を簡素化することができます実際には

class MyClass { 
    private final Object lock = new Object(); 
    private Channel channel = null; 

    public void setChannel() { 
     // other code can go here 

     synchronized (lock) { 
      channel = new Channel(); 
      // setup channel 
     } 

     // other code can go here 
    } 

    public void unsetChannel() { 

     // other code can go here 

     synchronized (lock) { 
      channel.close(); 
      channel = null; 
     } 

     // other code can go here 
    } 

    public boolean isSet() { 
     synchronized (lock) { 
      if (channel == null) { 
       return false; 
      } 
      return channel.isActive(); 
     } 
    } 
} 

は、lockオブジェクトは、チャネル変数へのアクセスを保護します。

channelと相互作用しない他のコードがロック外に存在する可能性がある場所を示します。

Answer 2

外側ロックlockはすべてを保護します。 2番目のロックは必要ありません。外側のロックが保持されている間は、2番目のスレッドには到達できません。