如何阻止Javax。摆动计时器？

Question

我是一个java新手，我正在使用netbean创建一个调度程序。我的软件将从用户那里获取时间和操作，并将这些数据输入到单独的数组列表中。然后我使用javax.swing.timer启动倒计时计时器。当用户完成输入数据并点击“运行”按钮时，计时器将从数组列表中的第一个元素中获取运行时间并开始倒计时。当倒计时时间达到0时，计时器将从数组列表中的下一个元素中获取倒计时时间，依此类推。

我的问题是，计时器运行得很好，但是当数组列表中没有更多元素时，它仍然不会停止。请帮我。这是我代码中遇到麻烦的部分：

private void btnstartRunActionPerformed(java.awt.event.ActionEvent evt) {                                            
  countdown = ActionTime.get(0);
  Action.setText(ActionName.get(c));
  timer = new Timer(1000,new ActionListener(){
    @Override
    public void actionPerformed(ActionEvent event){

      if(countdown<=0){
          try{
            c++;
            Action.setText(ActionName.get(c));
            //this nested if statement is to check whether the array has any more elements.
            //(This is the part that doesn't work.)
            if(c >= ActionTime.size()){
              timer.stop();
              JOptionPane.showMessageDialog(null, "Workout Completed!");
            }  
            countdown=ActionTime.get(c);
            timer.restart();

          } catch (Exception error2){
          }
       }
       Time.setText(Integer.toString(countdown));
       countdown--;
     }
  });
  timer.start();
}                                           

Answer 1

下面是一个例子：

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JFrame;
import javax.swing.Timer;

public class Main extends JFrame {
    Timer timer;

    int counter;

    Main(String title) {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        ActionListener a = new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                System.out.println("Counter = " + counter);

                if (++counter > 10) {
                    timer.stop();
                    System.exit(0);
                }
            }
        };

        timer = new Timer(300, a);
        timer.start();

        pack();
        setVisible(true);
    }

    public static void main(String[] args) {
        new Main("Timer Demo1");
    }
}

Answer 2

你应该改变：

            c++;
            Action.setText(ActionName.get(c)); //move this down
            if(c>=ActionTime.size()){
                timer.stop();
                JOptionPane.showMessageDialog(null, "Workout Completed!");
            }

致：

            c++;
            if(c>=ActionTime.size()){
                timer.stop();
                JOptionPane.showMessageDialog(null, "Workout Completed!");
                return; //add this too as the program should not continue with below
            }
            Action.setText(ActionName.get(c)); //to here

因为当你尝试做ActionName时。get（c）当c的值超过最大可能值时，get方法将抛出一个异常。这就引出了你犯的第二个错误：

           catch(Exception error2){

           }

这样做会使程序忽略异常，并像什么都没发生一样继续。这是一个容易犯的错误，它欺骗了你，让你认为其他事情是错的，但现在你知道了！将其更改为：

           catch(Exception error2){
                error2.printStackTrace();
                System.exit(-1); //or some error handling
           }

或者删除try/catch块，以便异常将结束程序或以其他方式处理。