Убедитесь, что переменная установлена ​​только один раз

Question

Мне нужно убедиться, что изделие продается только один раз. Это должно быть обеспечено также для нескольких потоков.

Достаточно ли проверить, нет ли переменной buyer? Так, на мой взгляд, второй вызывающий абонент получит AlreadyBoughtException?

public synchronized void buy(Buyer buyer) throws AlreadyBoughtException { 
    if (this.buyer != null) { 
     throw new AlreadyBoughtException(); 
    } 

    System.out.println(buyer + " bought article " + identifier); 

    this.buyer = buyer; 
    this.sold = true; 
} 

Это поточно-и я могу предположить, что нет никаких шансов, что эта статья не может быть куплен, когда метод buy вызывается в то же время (точно то же время)?

Answer 1

Тот факт, что ваш метод синхронизирован, делает его безопасным - пока это единственный способ, которым вы когда-либо обновляете покупателя.

Answer 2

Пока buyer не может быть установлен с помощью несинхронизированного метода, ваш код является потокобезопасным.

JVM гарантирует, что для синхронизированного метода никогда не произойдет «точно такое же время» - один из вызывающих абонентов будет считаться первым, а другой должен будет дождаться завершения первого вызова перед продолжением.

Answer 3

Да, это нормально (если вы, конечно, не обращаетесь к полю покупателя от другого метода).

Два потока, пытающихся выполнить этот метод в то же время, не смогут этого сделать: только один сможет получить блокировку и инициализирует покупателя. Затем другой поток выполнит его и увидит инициализированного покупателя, поскольку метод синхронизирован.

Answer 4

Вы можете

this.buyer 

летучий переменная