just_vladimir,

волтаил не атомарен. Инструкция на jls написана сухим языком, ее специалисту трудно бывает понять, а вы даете человеку который вообще не понимает.

Если 1 поток он читает ваш волатил, потом делает dataProcess и потом делает hasDataToProcess=false, то будет лажа. Будет и с synchronized и c volatile. Самое простое в этом случае - это делать dataProcess внутри того же synchronized.

rema174,
в такой формулировке вопроса как у вас, volatile самое оно. Но скорей всего, вы неверно сформулировали задачу. На это как бы намекает название hasDataToProcess и ваш класс вообще. То есть, у вас есть данные, есть обработка. Обработка имеет начало и конец, логично предположить что вконце обработки ваши данные_для_процесса перестают наличествовать, то есть после обработки надо hasDataToProcess поставить в false, а это уже конфликтует с условием задачи "один из которых только читает, второй только может устанавливать".
Хотя, это зависит от конкретного алгоритма. Например, если он обрабатывает каждый раз при проходе - то можно. Но не всегда, а только когда сами данные правильно опубликовались в другой поток.

Если у вас несколько потоков - мало объявить доступ к каждой переменной в synchronized. Это вообще ничего не даст, т.к. между разными synchronized оно рассинхронизируется. В простом случае надо выполнять все внутри одного synchronized. Псевдокод:




Но это если вам нужна однократная обработка. Если вам многократную, тогда можно volatile, но возникает проблема безопасной публикации из одного потока в другой самих данных. Т.е, если вы сделаете так:




Тут сама по себе проверка - ок, но надо быть готовым к тому, что поток 2 будет внутри processData(), а поток 1 их начнет в это время менять. В потоке 2 внутри вашего processData() часть данных считаются в старых значениях - а часть в новых. Иногда это оно, а иногда это не то, что вы хотите. Все зависит от "обвеса". Так что с этой многопоточностью надо быть очень осторожным.

Есть еще 1 психологический момент - если ты попытался сделать что-то сложное многопоточное, то твой мозг привыкает, что внутри if (условие) это условие может быть false из за действий другого потока. То есть, ты перестаешь быть уверенным буквально во всем. И потом, переключившись на написание однопоточного кода, какое-то время сложно соображать с той же скоростью как ты делал это раньше, потому что твой мозг постоянно тебе подбрасывает тебе идеи в стиле "а вдруг кто-то с другого потока прийдет и нагадит", ну и вообще начинаешь во всем сомневаться.