Тема Прерывание иногда работает, а иногда нет

Question

Пожалуйста, помогите мне с моей проблемой. У меня есть 2 JMenuItems, если я нажму на Start, он начнется и начнет работать. Если я нажму «Стоп», он остановится :) После того, как я нажал кнопку «Старт», я нажимаю «Остановить», а иногда останавливается, а иногда нет. Но я хочу, чтобы он всегда останавливался.

Что я сделал не так? :/

class DiashowListener implements ActionListener { 
Thread td; 
boolean ok = false; 

     public void actionPerformed(ActionEvent e) { 

      if (e.getActionCommand().equals("Start")) { 

       td = new Thread(new Runnable() { 
        public void run() { 
         if (bimg != null) { 
           while (!ok) { 
            try { 
             ... 


            } catch (Exception e2) { 
            } 
            frame.repaint(); 
           }      
         } 
        } 
       }); 

       td.start(); 

      } else if (e.getActionCommand().equals("Stop")) { 
       if (td != null){ 
        ok = true; 
       } 
      } 
     } 
} 

EDIT: хорошо я что-то изменилось, его работы сейчас, но: Если я нажимаю на Stop он должен немедленно прекратить.

Answer 1

Прежде всего, вы прерываете неправильную нить (должен быть td). Во-вторых, содержание оговорки try, которую вы опустили, действительно важно (некоторые операции являются бесперебойными). Наконец, Thread.isInterrupted скорее всего не то, что вы хотите использовать, так как флаг может быть очищен некоторым несвязанным кодом. В зависимости от того, что именно вы прерываете, он может (или не может) быть хорошей идеей просто поймать InterruptedException и прекратить работу в случае его броска. Лучше всего добавить свой собственный флаг, который будет проверяться потоком вместо isInterrupted, а обработчик события будет вместо (или в дополнение к) прерывать рабочий поток.