Java - Невозможно остановить выполнение из основного класса

Question

Я стучу головой о стол, здесь друзья. Это то, что я пытаюсь выполнить. У меня есть класс Main, который создаст новый поток. Прямо сейчас, когда поток будет работать, он выполнит незначительную очистку, и поток остановится. Тем не менее, у меня есть кнопка «Стоп» в пользовательском интерфейсе, которая при нажатии должна «сделать некоторую очистку, а затем убить поток». Проблема в том, что я не могу обновить что-либо, что находится внутри этого нового потока, который я создал .

Вот некоторые фрагменты:.

Main Class

runner = new Thread(new Controller1(options)); 
runner.start(); 

в приведенном выше коде, я звоню класс Controller1 и установить некоторые "варианты" в конструкторе Это работает хорошо до сих пор ...

Теперь, в классе Controller1, это значение t I have ...

public volatile static boolean stopped; 

public void run() { 
    while(!stopped){ 
     System.out.println("In run"); 
     startProxy(proxyPort); 
     startWebDriver(); 
     driver.get(http); 
     UIMainJSwing1.updateStartButton(); 
     stopped=true; 
    } 

    //Run killAll to stop webdriver and the proxy 
    killAll(); 
    System.out.println("Thread complete"); 
} 

Проблема в том, что из класса Main я не могу позвонить или установить «остановлен» на true. Я могу вызвать runner.interrupt(), но проблема в том, что, поскольку поток просто умирает, моя функция killAll() никогда не запускается, и я остаюсь с запущенным WebDriver и прокси.

Answer 1

Я бы назвал thread.interupt() в основном потоке и использовать в try/finally block вокруг петли в вашем Runnable, чтобы убедиться, что ваш метод killAll() называется.

public void run() { 
    try { 
     while (!stopped) { 
      ... 
     } 
    } finally { 
     //Run killAll to stop webdriver and the proxy 
     killAll(); 
     System.out.println("Thread complete"); 
    } 
} 

Окончательно блок будет всегда называться. Даже если поток прерывается или генерирует исключение.

Пара Другие комментарии:

И вы упоминаете volatile boolean stopped флаг, но вы устанавливаете, что немедленно быть верным в конце цикла, так что я понимаю, почему у вас есть цикл вообще.

Предполагая, что нить не проходит и сразу вызывает killAll(), тогда она висит где-то в одном из методов. Когда вы вызываете interrupt() из основного потока, все, что ждет, будет кидать InterruptedException. Но так как я не вижу никаких попыток, возможно, методы перебрасывают его как RuntimeException? Они должны, по крайней мере, делать что-то вроде следующего, но это по-прежнему блокирует исключение:

try { 
    something.wait(); 
} catch (InterruptedException e) { 
    // restore the interrupted condition 
    Thread.currentThread().interrupt(); 
} 

Независимо попытка/наконец, это правильный путь, чтобы обеспечить killAll называется.

Answer 2

Во-первых, вы должны, вероятно, поймать InterruptedException, чтобы ваш метод killAll() запускался. Просто попробуйте/наконец

Во-вторых, почему вы не можете установить stop = true из вашего основного метода?

Answer 3

Если я понимаю, что правильно, вы злоупотребляете идиомой (!). В вашем коде вы проверяете, не остановился ли он один раз, а затем выполните все до конца, установите stop = true и выйдите. Вы просто не можете прекратить выполнение после его запуска. Когда он запустит, остановленное поле никогда не будет перенаправлено, пока оно не закончится.

Answer 4

Во-первых, потеряйте while(!stopped) и stopped=true частей. Также вам нужно оценить stopped на каждом отдельном этапе в потоке (и не забудьте запустить killAll() и снова включить кнопку запуска).