필드 잠금 자바

Question

:

public class SomeClass extends Thread { 
    ... 
    private double a; 
    private double b; 

    private double increaseVal; 

    public void setIncreaseVal(double increaseVal) { 
     this.increaseVal = increaseVal; 
    } 

    private void updateAB() { 
     ... 
     // Computing the new a and b values 
     a = ... * increaseVal; 
     b = ... * increaseVal; 
     ... 
    }  

    @Override 
    public void run() { 
     try { 
      ...     
      updateAB(); 
      Thread.sleep(500); 
     } 
     ... 
    } 
} 

public class OtherClass { 
    ... 
    private HashMap<String, SomeClass> someClasses; 

    private void setIncreaseValue(String id, double value){ 
     ... 
     someClasses.get(id).setIncreaseValue(value); 
     ... 
    } 
} 

든 increaseValSomeClassupdateAB 방법 등이 처리 될 때 동일한 increaseVal 값이 이용되는 경우 모두 a 및 b 필드를 잠그는 것이 가능 업데이트 되었습니까?

setIncreaseValue이 OtherClass 인 경우 a이 업데이트 된 후 increaseVal의 값을 변경할 수 있습니다. b이 업데이트되면이 방식으로 increaseVal의 새 값이 사용됩니다. 나는 그것을 피하고 싶습니다.

a과 b 값을 업데이트하기 전에 synchronized(this)을 입력하면 제대로 작동합니까? 이처럼

:

synchronized(this) { 
    // Computing the new a and b values 
    a = ... * increaseVal; 
    b = ... * increaseVal; 
} 

감사합니다!

Answer 1

스레드 안전 코드를 올바르게 구현하려면 먼저 스레드 안전성을 유지해야하는 클래스의 상태를 식별해야합니다. 은 당신의 예에서 상태 (즉, 모두가 읽기 및 쓰기) 상태에 대한 액세스가 같은 잠금 개체를 사용하여 동기화 다음 단계는 모든 있는지 확인하는 것입니다 세 개의 필드

private double a; 
private double b; 
private double increaseVal; 

으로 표시됩니다. 당신의 예에서

이 쉽게 setIncreaseVal(...) 및 updateAB() 방법에 동기화 된 키워드를 추가 한 후 잠금으로 this를 사용하여 구현할 수 있습니다

public synchronized void setIncreaseVal(double increaseVal) { 
    ... 
} 

private synchronized void updateAB() { 
    ... 
} 

이것은 코드 스레드 안전을 만들 것입니다 : a 및 b 항상 것입니다 increaseVal을 사용하여 업데이트하십시오. 다른 스레드가 updateAB()를 실행하는 등 같은 increaseVal 업데이트로 클라이언트 코드가 a 및 b를 사용한다는 보장은 없지만 그러나 코드는 a 읽 및 b 여전히 a 및 b을 읽을 수 있습니다 : 이것은 당신이 필요로하는 경우 , a과 b에 대한 액세스를 동기화해야합니다 (코드는 예제에 표시되지 않으므로 이에 대해서는별로 말할 수 없습니다). updateAB()의 동기화에 관해서는

: 사용하지 않는 방법에있어서 많은 코드가있는 경우 a, b 및 increaseVal하고 실행하는 데 시간이 오래 걸릴 수 있습니다 (예를 들어,

private void updateAB() { 
    ... 
    synchronized(this) { 
     // Computing the new a and b values 
     a = ... * increaseVal; 
     b = ... * increaseVal; 
    } 
    ... 
} 

Answer 2

increaseVal을 우려 할 경우 대신 increaseVal을 잠글 수 있습니다. 즉

synchronized(increaseVal) { 
    // Computing the new a and b values 
    a = ... * increaseVal; 
    b = ... * increaseVal; 
} 

Answer 3

는 AtomicInteger로 increaseVal을 확인합니다. 원자화하면 문제를 해결할 수 있습니다.