Запуск thread столько раз сколько нужно

@pro100kryto · Регистрация: 13.07.2017

Author24 — интернет-сервис помощи студентам

Здравствуйте дорогие форумчане!

Нужно мне реализовать один thread внутри того же класса откуда оно и будет вызываться. При чем чтобы thread выполнял run только тогда когда я ему скажу. Т.е. у меня код примерно таков:

Java

public class PlayState extends State {
  //тут переменные
  float timer1=0, timer11;
  Thread thr1;
  //...
 
  public PlayState(GameStateManager gsm){
 
   thr1 = new Thread(new Runnable() {
      //private volatil переменные
      public void run() {
        //...
        thr1IsFinished=true;
      }
    }
  }
 
  @Override
  public void update(float dt) {
    //...
    //вот тут какрас штука такая что timer11 не статичен
    if((timer1+=dt)>timer11){
      if (thr1IsFinished){
          thr1IsFinished=false;
          thr1.stop();
          thr1.start();
     }
     timer1%=timer11;
     }
    //...
  }
 
}

При таком коде выходит ошибка у строки start():
Exception in thread "LWJGL Application" java.lang.IllegalThreadStateException

Подскажите плиз что я за ерунду наплел, где ошибка?)) Это первый раз как на процессорную тему напал и может чего не понимать. Видел что в туториалах создают отдельные классы для своего thread, а мне так не канает потому что мой thr1 должен брать значения переменных из PlayState для своих рассчетов.

@korvin_ · 19.07.2017, 16:55

Сообщение от pro100kryto

Подскажите плиз что я за ерунду наплел, где ошибка?

Чтобы не плести ерунду, нужно документацию читать.

public void start()
...
It is never legal to start a thread more than once. In particular, a thread may not be restarted once it has completed execution.

@Deprecated
public final void stop()

Deprecated. This method is inherently unsafe. Stopping a thread with Thread.stop causes it to unlock all of the monitors that it has locked (as a natural consequence of the unchecked ThreadDeath exception propagating up the stack). If any of the objects previously protected by these monitors were in an inconsistent state, the damaged objects become visible to other threads, potentially resulting in arbitrary behavior. Many uses of stop should be replaced by code that simply modifies some variable to indicate that the target thread should stop running. The target thread should check this variable regularly, and return from its run method in an orderly fashion if the variable indicates that it is to stop running. If the target thread waits for long periods (on a condition variable, for example), the interrupt method should be used to interrupt the wait. For more information, see Why are Thread.stop, Thread.suspend and Thread.resume Deprecated?.

--- https://docs.oracle.com/javase... hread.html

@pro100kryto · 19.07.2017, 19:42 **[ТС]**

Хорошо, stop совсем убивает а start пишется только один раз.
Дак что происходит с процессом когда он проходит весь код run? Как запустить run еще разок в thread тогда когда тебе это потребуется?
Или тогда нужно по любому в цикл ставить чтобы он вечно проверял таймер типо так?

Java

  thr1 = new Thread(new Runnable() {
      //private volatil переменные
      public void run() {
        do{
            if(timer){
               //...
            }
        }while(true)
      }
    }

Просто хотелось чтобы thread просто так не бегал

.

Добавлено через 7 минут
Т.е. фактически нужно превратить private void xxx() в thread, т.е. чтоб xxx() запускался как побочный поток.

Добавлено через 1 час 35 минут
Ладно, вроде разобрался. Сделал цикл do-while(!Thread.interrupted()) в run(), проверка таймера

@korvin_ · 19.07.2017, 21:08

Сообщение от pro100kryto

Ладно, вроде разобрался. Сделал цикл do-while(!Thread.interrupted()) в run(), проверка таймера

М-да… Посмотри лучше в сторону ExecutorService.

@korvin_ 4539 / 2732 / 486 Регистрация: 28.04.2012 Сообщений: 8,628
	19.07.2017, 16:55	2
	Сообщение от pro100kryto Подскажите плиз что я за ерунду наплел, где ошибка? Чтобы не плести ерунду, нужно документацию читать. public void start() ... It is never legal to start a thread more than once. In particular, a thread may not be restarted once it has completed execution. @Deprecated public final void stop() Deprecated. This method is inherently unsafe. Stopping a thread with Thread.stop causes it to unlock all of the monitors that it has locked (as a natural consequence of the unchecked ThreadDeath exception propagating up the stack). If any of the objects previously protected by these monitors were in an inconsistent state, the damaged objects become visible to other threads, potentially resulting in arbitrary behavior. Many uses of stop should be replaced by code that simply modifies some variable to indicate that the target thread should stop running. The target thread should check this variable regularly, and return from its run method in an orderly fashion if the variable indicates that it is to stop running. If the target thread waits for long periods (on a condition variable, for example), the interrupt method should be used to interrupt the wait. For more information, see Why are Thread.stop, Thread.suspend and Thread.resume Deprecated?. --- https://docs.oracle.com/javase... hread.html 1

@korvin_ 4539 / 2732 / 486 Регистрация: 28.04.2012 Сообщений: 8,628
	19.07.2017, 21:08	4
	Сообщение было отмечено pro100kryto как решение Решение Сообщение от pro100kryto Ладно, вроде разобрался. Сделал цикл do-while(!Thread.interrupted()) в run(), проверка таймера М-да… Посмотри лучше в сторону ExecutorService. 1

Запуск thread столько раз сколько нужно

Решение