synchronized void takeBackSalary(int salary){
this.salary = bank.deal(salary);
}
Данный метод нужно синхронизировать, как я сделал, или процесс и без этого атомарный?
*Использованный внутри метод синхонизирован.
Stack Overflow на русском — это сайт вопросов и ответов для программистов. Присоединяйтесь! Регистрация займёт не больше минуты.
Присоединиться к сообществуsynchronized void takeBackSalary(int salary){
this.salary = bank.deal(salary);
}
Данный метод нужно синхронизировать, как я сделал, или процесс и без этого атомарный?
*Использованный внутри метод синхонизирован.
Атомарность присваивания зависит от типа переменной salary
.
Тем не менее, это не единственный аспект, на который нужно обратить внимание. Если значение переменной salary
читается из несколькоих потоков, то необходимо обеспечить видимость изменения этой переменой другим потокам. Ваш пример ее обеспечивает из-за synchronized
. Хотя аналогичного эффекта можно достичь с помощью модификатора volatile
или атомарных ссылок (AtomicRef
или другой класс, в зависиомсти от типа).
PS: Прочитайте уже книгу "Concurrency in practice" и многое встанет на свои места. Не дело это, изучать тонкости многопоточного программирования с помощью форумов.