Stop a swing Timer from inside the Action Listener

Question

I'm trying to stop a timer inside the the ActionListener. Below is the code of what i'm trying to do. I'm tring to stop the timer i created when a certain condition is met inside the actionPerformed method. timer.stop() does not work , the compiler does not let me do that.

Any help . suggestion , advice would be really helpful.

public class ToggleAnnotationsAction extends IdentifiedMultiAction {     //This status indicates if the Toggle action has been completed  /**  * Defines the toggling direction of a <code>ToggleAnnotationAction</code> instance.  */ public static enum Direction {FORWARD, BACKWARD}; private Direction myDir;  /**  * Create an action with the direction presets given by the provided <code>Enum</code>.  *   * @param dir An <code>Enum</code> defined in this class which maps to the correct direction of toggling  * @see behaviors.multiact.IdentifiedMultiAction#IdentifiedMultiAction(Enum)  */ public ToggleAnnotationsAction(Direction dir) {     super(dir);     this.myDir = dir; }  /**  * Performs the toggling, moving the audio position to the next/previous annotation.  *   * Afterward sends an update to all <code>UpdatingActions<code>.  *  * Since the waveform display autonomously decides when to paint itself, this action may not result in an instant visual change.  *   * <p>Prints warnings if an appropriate Annotation could not be found, despite the action being enabled.  *   * @param e The <code>ActionEvent</code> provided by the trigger  */ public void actionPerformed(ActionEvent e) {     //Reset Status to 0      status =0;       Annotation ann = findAnnotation(myDir, CurAudio.getMaster().framesToMillis(CurAudio.getAudioProgress()));     if(ann == null) {         System.err.println("It should not have been possible to call " + getClass().getName() + ". Could not find matching annotation");     }     else {         final long approxFrame = CurAudio.getMaster().millisToFrames(ann.getTime());         final long curFrame = CurAudio.getAudioProgress();         if(approxFrame < 0 || approxFrame > CurAudio.getMaster().durationInFrames() - 1) {             GiveMessage.errorMessage("The annotation I am toggling to isn't in range.\nPlease check annotation file for errors.");              return;         }               Timer timer = new Timer(10, new ActionListener() {                  private long panFrame = curFrame;                  private long endFrame = approxFrame;                  public void actionPerformed(ActionEvent evt) {                       if(myDir == Direction.FORWARD){                          if (panFrame >= endFrame) {                               //How do i Stop my timer here ?                               return;                          }                          CurAudio.setAudioProgressWithoutUpdatingActions(panFrame);                          panFrame += 4000;                     }                  else if(myDir == Direction.BACKWARD){                      if (panFrame <= endFrame) {                          // How do i Stop my timer here ?                           return;                      }                      CurAudio.setAudioProgressWithoutUpdatingActions(panFrame);                      panFrame -= 4000;                 }              }           }          );           timer.start();        }     MyFrame.getInstance().requestFocusInWindow(); }   /**  * A forward (backward) <code>ToggleAnnotationsAction</code> should be enabled only when audio is open, not playing, and when there is an annotation following (preceding) the current position.  */ @Override public void update() {     if(CurAudio.audioOpen()) {         if(CurAudio.getPlayer().getStatus() == PrecisionPlayer.Status.PLAYING) {             setEnabled(false);         }         else {             double curTimeMillis = CurAudio.getMaster().framesToMillis(CurAudio.getAudioProgress());             if(findAnnotation(myDir, curTimeMillis) != null) {                 setEnabled(true);             }             else {                 setEnabled(false);             }         }     }     else {         setEnabled(false);     } }  /**  * Finds the next/previous <code>Annotation</code> relative to a certain audio position in milliseconds.  *   * @param dir The direction of movement  * @param curTimeMillis The present time in milliseconds  *   * @return In principle, the <code>Annotation</code> after/before <code>curTimeMillis</code>  */ private Annotation findAnnotation(Direction dir, double curTimeMillis) {     Annotation[] anns = AnnotationDisplay.getAnnotationsInOrder();     if(myDir == Direction.FORWARD) {         for(int i = 0; i < anns.length; i++) {             if(anns[i].getTime() - curTimeMillis > 1) {                 return anns[i];             }         }     }     else {         for(int i = anns.length - 1; i >= 0; i--) {             if(curTimeMillis - anns[i].getTime() > 1) {                 return anns[i];             }         }     }     return null; } 

}

Thanks in Advance Krishnan

Answer 1

Also possible:

final Timer timer = new Timer(10, null); timer.addActionListener(new ActionListener() {     public void actionPerformed(ActionEvent evt) {         (as in the question, except that you can refer to timer here)     } }); 

Or, use the event object to get the source (and cast it, boo):

final Timer timer = new Timer(10, new ActionListener() {     public void actionPerformed(ActionEvent evt) {         ((Timer)evt.getSource()).stop();     } }); 

Or, keep the timer in an instance variable and you can reference it from your handler or have the handler call a method on your class which could stop/start it.