자바 : 동기화 스레드에 실패

Question

나는 다음과 같은 코드가 있습니다

for (int iThreadCounter = 1; iThreadCounter <= CONNECTIONS_NUM; iThreadCounter++){ 
    WorkThread wt = new WorkThread(iThreadCounter); 
    new Thread(wt).start(); 
    m_arrWorkThreadsToCreate.add(wt); 
} 

그 스레드가 다음 코드 호출

int res = m_spLegJoin.call(m_workTread, m_workTread.getConfId()); 

그리고이 LegJoinSp 클래스 내부의 호출 방법 :

public class LegJoinSp extends ConnEventSp { 

    private static final int _LEG_JOIN_ACTION_CODE = 22; 
    private static int m_nLegId = Integer.valueOf(IniUtils.getIniValue("General", "LEG_ID_START")); 
    private final Lock m_lock = new ReentrantLock(); 

    public int call(WorkThread a_workThread, String a_sConfId) { 

     synchronized (this) { 

      //m_lock.lock(); 
      m_nLegId++; 

      boolean bPass = false; 

      Log4jWrapper.writeLog(LogLevelEnum.DEBUG, "LegJoinSp - call", "a_workThread = " + a_workThread.getThreadId() + " a_sConfId = " + a_sConfId); 

      if (super.call(a_workThread, a_sConfId, _LEG_JOIN_ACTION_CODE, "" + m_nLegId) == 0) { 

       bPass = true; 

      } else { 
       bPass = false; 
      } 

      //m_lock.unlock(); 

      if (bPass) { 

       Log4jWrapper.writeLog(LogLevelEnum.DEBUG, "LegJoinSp - call", "a_workThread = " + a_workThread.getThreadId() + " a_sConfId = " + a_sConfId + " returned leg id " + m_nLegId); 

       return m_nLegId; 
      } else { 

       return -1; 
      } 
     } 

    } 

    public Lock getLock() { 
     return m_lock; 
    } 

} 

을

이 call() 메소드를 호출하는 스레드가 두 개 있습니다. m_nLegId는 당신은 내가 함께 방법을 잠글 시도 볼 수 있듯이 100 으로 시작됩니다 모두

synchronized(this) 

및

m_lock.lock() and m_lock.unlock() 

문제입니다 처음 경우에 (bPass)를 얻을 때 내부 코드는 102를 m_nLegId 값으로 내 로그에 씁니다. 그러나 나는 10130이기를 기대한다. m_nLegId ++; 성명. 첫 번째 스레드 실행을 위해 동기화 블록이 끝나기 전에 두 번째 스레드가 코드 내부로 들어가는 것 같습니다.

어떻게 해결할 수 있습니까?

은 (당신이 이에 잠금을 적용되는) 것은 당신이 모든 스레드가 새로운 객체를 만드는 것입니다,하지만 당신은 잠금을 적용하는 방식은 동일 객체에 적용하면

Answer 1

나에게 당신의 버그는 m_nLegId이 static field이라는 사실과 관련이 있으며 클래스의 현재 인스턴스에 대한 액세스를 동기화하여 필드의 동시 수정을 올바르게 방지하지 못하게하려고합니다.

나는

synchronized (this) { 

오히려

synchronized (LegJoinSp.class) { 

NB 수 있어야 의미 : 당신은 단지 카운터를해야 할 경우, 당신의 필드 대신 지능에 대한 AtomicInteger을 사용하는 것이 좋습니다.

Answer 2

감사드립니다.
클래스 수준에서 잠금을 적용하려는 경우 정적 개체를 만들고 달성하려는 목적을 달성 할 수있는 해당 개체에 잠금을 적용 할 수 있습니다 (주석을 기반으로 올바르게 문제를 이해 한 경우)