Остановите таймер качания изнутри прослушивателя действий

Я пытаюсь остановить таймер внутри ActionListener. Ниже приведен код того, что я пытаюсь сделать. Я пытаюсь остановить таймер, который я создал, когда определенное условие встречается внутри метода actionPerformed. timer.stop() не работает, компилятор не позволяет мне это делать.

Любая помощь. предложение, рекомендации были бы действительно полезными.

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;
}

}

Спасибо в Advance Кришнан

Ответ 1

Также возможно:

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)
    }
});

Или, используйте объект события, чтобы получить источник (и бросить его, boo):

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

Или сохраните таймер в переменной экземпляра, и вы можете ссылаться на него из вашего обработчика или вызвать вызов обработчиком метода в вашем классе, который может остановить/запустить его.

Ответ 2

Забавная проблема.

Вы должны сделать Timer final доступ к нему в анонимном ActionListener, в котором хотите его остановить. Но анонимный ActionListener все еще не скомпилируется, потому что таймер еще не инициализирован.

Обратите внимание, что Таймер может и не быть вашим лучшим выбором. Но чтобы сделать эту работу так, как есть, я бы обернул таймер во внутреннем классе.

В вашем методе ToggleAnnotationsAction.actionPerformed() добавьте строку, например:

MyTimer timer = new MyTimer();
timer.start();

Затем можно использовать такой класс, заменив мой простой код ActionListener кодом вашего таймера ActionListener:

private class MyTimer implements ActionListener{
  private Timer timer;

  private MyTimer(){
     timer = new Timer(10, this);
  }

  public void start(){
     timer.start();
  }

  public void stop(){
     timer.stop();
  }

  public void actionPerformed(ActionEvent e){
     if(isTimeToStop()){
        stop();
     }
  }

  public boolean isTimeToStop(){
     return true;
  }
}