개체 모니터 또는 내 경우에는 세마포어에 동기화 된?

Question

내 메서드가 스레드로부터 안전한지 또는 그 반대인지 의심 스러워요. 스레드로부터 안전하지는 않지만 확실히 해결책을 찾지 못했습니다.

class TestLockSingleton 
    { 
     private static TestLockSingleton ourInstance = new TestLockSingleton(); 
    public static TestLockSingleton getInstance() { 
     return ourInstance; 
    } 

    private TestLockSingleton() { 
    } 

    ... 
    private Object LockMonitor= new Object(); 

      interface Listener 
     { 
      void isEnable(boolean result); 
     } 

public void setStatus(int status){ 
    synchronized(LockMonitor){ 
     this.status = status; 
     this.setted = true; 
    } 
} 

     public void isEnable(final Listener listener){ 
      synchronized(LockMonitor){ 
       if(!setted){ 
        LocalBroadcastManager.getInstance(context).registerReceiver(new BroadcastReceiver() { 
        @Override 
        public void onReceive(Context context, Intent intent) { 
         synchronized(LockMonitor){ 
          Bundle bundle = intent.getExtras(); 
          setStatus(bundle.getInt("Status", OFFVALUE)); 
          Listener.isEnable((status==ACTIVEVALUE)?true:false); 
          LocalBroadcastManager.getInstance(context).unregisterReceiver(this); 
         } 
        } 
        },new IntentFilter(MYACTION)); 
       }else{ 
        Listener.isEnable((status==ACTIVEVALUE)?true:false); 
       } 
      } 

     } 
    .... 
    } 

지금 내 문제는 내가 TestLockSingleton 클래스 내부의 방법 isEnable (최종 리스너 리스너) 스레드 안전이 아니라고 생각한다는 것입니다 : 다음 내 코드입니다.

둘 이상의 수신기를 처리하고 싶지는 않지만 대신 마지막 메서드 인 isEnable (최종 리스너 수신기)을 호출 한 사람이 콜백을 수신하는지 확인해야합니다.

두 개의 다른 스레드가 isEnable (최종 리스너 수신기)를 호출하면 메소드를 호출 한 마지막 수신기는 두 개의 수신기 콜백을 수신합니까?

편집 : 나는 "상태"변수가 이미 isEnable에 선행 호출에서 잘 살고 있지 않은 경우에만 수신기 (최종 리스너 리스너를) 등록 할

• :이 그렇게하고 싶지는 무엇 코드를 추정 그렇게하기 위해서 나는 "setted"변수를 만들고 이것을 상태 변수를 설정하는 것과 동시에 true로 설정했습니다.

   TestLockSingleton.getInstance().isEnable(
        new Listener() 
      { 
         @Override 
       public void isEnable(boolean result) { 
  
       } 
      }) 
  

  :

  시간에서 1 호출을 스레드 :

• 내가 이것을 달성하려면 "걸렸고"및 "상태"멤버 변수

• 동시 액세스를 방지하기 위해 동기화 사용하십시오 스레드 B는 시간 2에 똑같은 작업을 수행합니다.

  나는 스레드 B가 리스너 resu 왜냐하면 나는 그 스레드 B가 두 개의 콜백을 동일한 결과로 수신하기 때문에, 하나 이상의 BroadcastReceiver를 등록하는 것을 막을 것이다.

Answer 1

아래와 같은 코드가 필요하다고 생각합니다.

주의해야합니다. Listener.isEnable은 복수 스레드에서 호출됩니다. TestLockingSingleton.isEnable을 호출하는 스레드와 주 스레드입니다.

public class TestLockSingleton { 
    // ... 

    public interface Listener { 
    void isEnable(boolean isActive); 
    } 

    private static TestLockSingleton instance = new TestLockSingleton(); 

    public static TestLockSingleton getInstance() { 
    return instance; 
    } 


    private final Object lock = new Object(); 
    private int status; 
    private boolean enabled; 

    private TestLockSingleton() { 
    } 

    public void setStatus(int status) { 
    synchronized (lock) { 
     this.status = status; 
     this.enabled = true; 
    } 
    } 

    public void isEnable(Context context, final Listener listener) { 
    boolean enabled; 
    boolean active; 
    synchronized (lock) { 
     enabled = this.enabled; 
     active = status == ACTIVEVALUE; 
    } 

    if (enabled) { 
     listener.isEnable(active); 
     return; 
    } 

    LocalBroadcastManager.getInstance(context).registerReceiver(
     new BroadcastReceiver() { 
     @Override 
     public void onReceive(Context context, Intent intent) { 
      int status = intent.getExtras().getInt("Status", OFFVALUE); 
      setStatus(status); 
      listener.isEnable(status == ACTIVEVALUE); 
      LocalBroadcastManager.getInstance(context).unregisterReceiver(this); 
     } 
     }, 
     new IntentFilter(MYACTION)); 
    } 

    // ... 
} 

편집 :는 Listener.isEnable를 호출하는 동안 잠금을 보유에서 TestLockSingleton.isEnable을 방지합니다. 이 경우에 이것이 바람직한 지 명확하지 않다.