run 메서드에서 synchronized 메서드 호출

Question

나는 계정 모델을 가지고 있으며이를 동기식으로 철회하고 싶습니다. 그러나 메서드를 동기화하고 run 메서드에서 호출 한 후에도 출력 순서대로 문제가 여전히 남아 있습니다. 여기 내 코드가있다. 나는 정말로 그 문제가 뭔지 모른다.

public class Account { 
private double balance; 
public Account(double balance) { 
    this.balance = balance; 
} 
public double getBalance() { 
    return balance; 
} 
public void setBalance(double balance) { 
    this.balance = balance; 
} 
} 


public class WithdrawThread implements Runnable { 
private double amount; 
private Account account; 

public WithdrawThread(Account account, double amount) { 
    this.account = account; 
    this.amount = amount; 
} 

public synchronized void withdraw(double amount) { 

    double bal = account.getBalance(); 
    if (amount > account.getBalance()) 
     throw new IllegalArgumentException("wrong amount"); 
    bal -= amount; 
    account.setBalance(bal); 
    System.out.println(amount + " withdraw in thread number" +  Thread.currentThread().getId() + " balance is " + bal); 
} 

public void run() { 
    withdraw(amount); 
} 

} 소티 Delimanolis 의해 명시된 바와

public class MainThread { 
public static void main(String[] args) { 
    Account account = new Account(200); 
    new Thread(new WithdrawThread(account, 40)).start(); 
    new Thread(new WithdrawThread(account, 10)).start(); 
    new Thread(new WithdrawThread(account, 10)).start(); 
    new Thread(new WithdrawThread(account, 20)).start(); 
    new Thread(new WithdrawThread(account, 30)).start(); 
    new Thread(new WithdrawThread(account, 10)).start(); 
    new Thread(new WithdrawThread(account, 40)).start(); 
} 
} 

Answer 1

이 동기화 방법은 WithdrawThread의 인스턴스가에서 호출 대상으로부터 자물쇠를 사용한다. 계정을 잠 그려면 ...

계정 방법을 철회하거나 동기화 된 블록을 사용하십시오.

public void withdraw(double amount) { 
    double bal; 
    synchronized(account) { 
     bal = account.getBalance(); 
     if (amount > account.getBalance()) 
      throw new IllegalArgumentException("wrong amount"); 
     bal -= amount; 
     account.setBalance(bal); 
    } 
    System.out.println(amount + " withdraw in thread number" + Thread.currentThread().getId() + " balance is " + bal); 
} 

Answer 2

synchronize 때 특정 개체를 동기화합니다. 그런 다음 다른 사용자가 완료 할 때까지 해당 개체를 동기화하지 못하게합니다.

개체를 지정하지 않으면 this에서 동기화됩니다.

그래서 각 스레드가 동기화됩니다.

대신 메서드 내에서 동기화 된 블록을 사용하고 수정되는 Account과 같은 공통 개체에 동기화해야합니다.

public void withdraw(double amount) { 
    synchronized (account) { 
     double bal = account.getBalance(); 
     if (amount > account.getBalance()) 
      throw new IllegalArgumentException("wrong amount"); 
     bal -= amount; 
     account.setBalance(bal); 
     System.out.println(amount + " withdraw in thread number" +  Thread.currentThread().getId() + " balance is " + bal); 
    } 
}