java thread not interrupt при запуске большого количества потоков

Question

В моем проекте на java swing есть 2 кнопки (начало/остановка) для подсчета.

Когда я нажимаю кнопку «Пуск». Существует один поток (Thread-0), затем нажмите кнопку остановки «Thread-0», но при многократном нажатии кнопки запуска. существует много потоков, таких как Thread-5, Thread-6, .. Thread-10 работает.

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

кнопка запуска

private void btnStartActionPerformed(java.awt.event.ActionEvent evt) {   
    start(); 
    btnStart.setEnabled(false); 
    btnStop.setEnabled(true); 
} 

кнопка остановки

private void btnStopActionPerformed(java.awt.event.ActionEvent evt) {    
    isEnable = false; 
    btnStop.setEnabled(false); 
    btnStart.setEnabled(true); 
}  

начало() метод:

isEnable = true; 
Thread refreshPlan = new Thread() { 
    @Override 
    public void run() { 
     while(isEnable) { 
      try { 
       sleep(CYCLE_TIME * 1000); 
       PLAN += 1; 
       planValue.setText(String.valueOf(PLAN)); 
      } catch (InterruptedException ex) { 
       //ignore 
      } 
     } 
    }; 
    }; 
    refreshPlan.start(); 

Может Я запускаю только один поток при многократном нажатии кнопки запуска? любое предложение? Благодарю.

извините за мой плохой английский.

Answer 1

Есть четыре важных проблем здесь:

• Если только isEnable объявлен как летучее переменной, там нет убедитесь, что запись из одной нити будет видна в другом потоке
• Аналогично ваш доступ к счетчику PLAN (который плохо назван - пожалуйста, следуйте Соглашения о присвоении имен Java) является небезопасным. Вы можете рассмотреть возможность использования AtomicInteger.
• Вы вносите изменения в пользовательский интерфейс из дополнительной темы. Вы не можете этого сделать - в Swing (и большинстве пользовательских интерфейсов) весь доступ к компонентам пользовательского интерфейса должен выполняться в потоке, отвечающем за этот интерфейс. См. Swing concurrency tutorial для более подробной информации.
• Потому что вы только проверяете isEnabled один раз в секунду, можно было бы остановить и запустить несколько потоков за это время ... приводя к тому, что сразу несколько потоков будут активны. Это может помешать вашему подсчету.

Возможно, вам будет полезно использовать javax.swing.Timer, который срабатывает один раз в секунду и просто проверяет, предназначено ли оно что-либо делать. Таким образом все может быть в потоке пользовательского интерфейса.

Answer 2

метод запуска создает экземпляр новой темы(), поэтому каждый раз, когда вы нажимаете на нее, она вносит новую жизнь в жизнь.

объявить Thread refreshPlan переменная класса, то в методе запуска поместить весь код в этой проверке

if(refreshPlan == null || !refreshPlan.isAlive()){ 
//ur existing code to instantiate new thread. 
}