Таймер ускоряется после каждого запуска

Question

Я делаю простой просмотрщик изображений для Android. Мне также нужно реализовать слайд-шоу. Я вроде как сделал, за исключением того, что у меня есть раздражающая ошибка. Когда я запускаю слайд-шоу (нажатием кнопки), он работает. Но когда я отменяю слайд-шоу и запускаю его снова (та же кнопка), слайд-шоу ускоряется.

EDIT: Добавлен фиксированный код, обратите внимание на комментарии в слайд-шоу части кода, чтобы увидеть, что нужно фиксируя

package csc2002.imageviewer; 
    //imports 

public class MainActivity extends Activity implements OnClickListener { 
    static Timer timer = new Timer(); 
    int arrayIndex = 0; 
    int checker=0; 
    int start,delay = 1800; 
    boolean toggle; 


    private static Integer[] imageIds = { //Hard coded array 
     R.raw.bulbasaur,R.raw.switch_brain,R.raw.quote,R.raw.victory,R.raw.penguins,R.raw.jellyfish,R.raw.koala}; 

    private static final int IMAGE_COUNT = imageIds.length; 

    @Override 
    public void onCreate(Bundle savedInstanceState) { // Called when the app is opened 

     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_main); 
     Button back = (Button)findViewById(R.id.backButton); 
     back.setOnClickListener(this);  
     Button next = (Button)findViewById(R.id.nextButton); 
     next.setOnClickListener(this); 
     Button slideshow = (Button)findViewById(R.id.slideButton); 
     slideshow.setOnClickListener(this); 
     displayImage(); 
    } 

    // responsible for displaying the image and the name of the image. 
    private void displayImage() { // Displays the image on the screen according the the current array index 

     ImageView imgView = (ImageView) findViewById(R.id.myimage);    
     imgView.setImageResource(imageIds[arrayIndex]); 
     TextView text = (TextView)findViewById(R.id.name); 
     text.setText(imageIds[arrayIndex]); 
    } 

    // This method allows the cycling of images 

    public void onClick(View v) { 
     if(v.getId()==(R.id.backButton)){ //Back button 
      arrayIndex--; 

      if(arrayIndex==-1){ 
       arrayIndex = IMAGE_COUNT-1; 
      } 
      displayImage(); 
     } 

     else if (v.getId()==(R.id.nextButton)){ //NextButton 
      arrayIndex++; 
     } 
     if(arrayIndex==IMAGE_COUNT){ 
      arrayIndex = 0; 
     } 
     displayImage(); 

     if (v.getId()==R.id.slideButton){ //Slideshow button 
      Button slideshow = (Button)findViewById(R.id.slideButton); 
      toggle^= true; 
      if(toggle==true){ 
       slideshow.setText("Stop Slideshow"); 
      } 
      else{slideshow.setText("Start Slideshow");} 
     } 
     // Slideshow functionality 

     if(toggle==true){ 
      //timer=new Timer(); //this was added in the correct solution(This was the main problem). 
      timer.scheduleAtFixedRate(new TimerTask() { 

       @Override 
       public void run() { 
        if(toggle==true){ 
         MainActivity.this.runOnUiThread(new Runnable() { 
          public void run() { 
           arrayIndex++; 
           if(arrayIndex==IMAGE_COUNT){ 
            arrayIndex=0; 
           } 

           displayImage(); 
          } 
         }); 
        } 
       } 

      },start,delay); 

     } 
     else if(toggle==false){ 
      //timer.cancel(); This was added to correct the solution 
     } 
    } 

    @Override 
    protected void onSaveInstanceState(Bundle outState) { //This saves data when the app is rotated 
     outState.putInt("KEY", arrayIndex); 
     super.onSaveInstanceState(outState); 

    } 
    @Override 
    protected void onRestoreInstanceState(Bundle save){ //Restores Data after interruption 
     super.onRestoreInstanceState(save); 
     arrayIndex = save.getInt("KEY"); 
     displayImage(); 
    } 


} 

Answer 1

я имел аналогичные проблемы, связанные с проектом, я сделал пару лет назад. Конечно, я использовал Timer объект, но вот несколько вещей, которые я сделал, чтобы решить эту проблему:

• Убедитесь, что вы явно остановить таймер.
• Повторно инициализируйте таймер, перезаписав оригинальный таймер.

Если эти предложения не работают, попробуйте опубликовать больше кода.

EDIT

public void onClick(View v) { 
    //... 
    if (v.getId()==R.id.slideButton){ //Slideshow button 
     toggle^= true; 
     if(toggle==true){ 
      timer = new Timer(); 
      timer.schedule(/*Whatever TimerTask you had here before*/);   
     } 
     else{ 
      timer.cancel(); 
     } 
    }//end if(slide button) 
}//end onClick() 

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