Java: Является ли этот подход потокобезопасным?

Question

Java: Является ли этот подход потокобезопасным?

1

У меня есть Thread, и мне нужно установить, когда он прослушивает или находится в режиме ожидания, поскольку я определил

public static enum ListenerState { STAND_BY, LISTENING };

и метод

public void setState(ListenerState state){
    this.state = state;
}

теперь, в основном цикле, я проверяю состояние таким образом

@Override
public void run() {
    while (!Thread.interrupted()) {
        try {
            if (state==ListenerState.LISTENING){
               // do my job
            }
            else{
                Thread.sleep(300);
            }
        }
    }
}

Является ли этот подход безопасным для потоков?

jmhostalet 28 нояб. 2014, в 15:30

Источник

0

Связанный: Этот пример в Спецификации языка Java.
VGR 28 нояб. 2014, в 14:38

Теги:

java

multithreading

3 ответа

2

Вы можете найти свой ответ здесь: нужно ли добавить некоторые блокировки или синхронизацию, если есть только одна запись потока и чтение нескольких потоков?

одним словом: лучше добавить ключевое слово volatile в состояние.

DontRelaX 28 нояб. 2014, в 12:19

1

если состояние может быть изменено или прочитано другим потоком, тогда вам нужно синхронизировать блок для чтения и записи. или как лучший способ, используйте AtomicBoolean. это идеальный объект, чтобы избавиться от блока syncronize и сделать его потокобезопасным

https://docs.oracle.com/javase/7/docs/api/java/util/concurrent/atomic/AtomicBoolean.html

Adem 28 нояб. 2014, в 12:42

Ещё вопросы

Связанный: Этот пример в Спецификации языка Java.

win_wave · Accepted Answer · 2014-11-28T11-23-00.000Z

Нет, сделайте так:

class MyThread implements Runnable {

    private volatile ListenerState state;

    public synchronized void setState(ListenerState state){
       this.state = state;
    }

    @Override
    public void run() {
       while (true) {
         try {
            if (state==ListenerState.LISTENING){
              // do my job
            } else{
              Thread.sleep(300);
            }
          } catch (IterruptedException ex){
            return; 
          }
        }
    }
}