2017-02-21 19 views
11

私はcountDownTimerを持っており、ユーザーが12秒以内にgameButtonをヒットしなかった場合、gameOverメソッドが呼び出されます。問題私は、countDownTimerが12のときにinstamtlyと呼ばれるゲームのfuntionを得るか、タイマーはちょうどカウントダウンを続けます。だから私はpostDelayed()メソッドを使用して、ユーザーにボタンを押してcountDownTimerを続行させる完全な秒を与えようとしていますが、私のコードは今のところゲームは12に関係なく停止します。android studioでpostDelayed()を正しく使う方法は?

import android.app.Activity; 
import android.os.CountDownTimer; 
import android.os.Handler; 
import android.support.v7.app.AppCompatActivity; 
import android.os.Bundle; 
import android.view.View; 
import android.widget.Button; 
import android.widget.TextView; 
import android.widget.Toast; 



public class GameScreen extends Activity { 

    private TextView time; 
    private Button start; 
    private Button cancel; 
    private Button gameButton; 
    private CountDownTimer countDownTimer; 
    public static int count = 0; 
    public static int countFail = 0; 

    final Handler handler = new Handler(); 
    final Runnable r = new Runnable() { 
     public void run() { 
      handler.postDelayed(this, 1000); 
      gameOver(); 
     } 
    }; 


    private View.OnClickListener btnClickListener = new View.OnClickListener(){ 

     @Override 
     public void onClick(View v) { 

      switch(v.getId()){ 
       case R.id.start_ID : 
        start(); 
        break; 
       case R.id.cancel : 
        cancel(); 
        break; 
       case R.id.gameButton_ID : 
        gameButton(); 
        break; 
      } 

     } 


    }; 


    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_game_screen); 


     start = (Button) findViewById(R.id.start_ID); 
     start.setOnClickListener(btnClickListener); 
     cancel = (Button) findViewById(R.id.cancel); 
     cancel.setOnClickListener(btnClickListener); 
     time = (TextView) findViewById(R.id.time); 
     gameButton = (Button) findViewById(R.id.gameButton_ID); 
     gameButton.setOnClickListener(btnClickListener); 


    } 

    public void start(){ 

     time.setText("16"); 
     //this doesnt work and makes app crash when you hit start button 

     countDownTimer = new CountDownTimer(16 * 1000, 1000) { 
      @Override 
      public void onTick(long millsUntilFinished){ 
       time.setText("" + millsUntilFinished/1000); 

       //turns textview string to int 
       int foo = Integer.parseInt(time.getText().toString()); 

       if(time.getText().equals("12")){ 

        r.run(); 

       } 

      } 

      public void onFinish() { 
       time.setText("Done !"); 
      } 
     }; 
     countDownTimer.start(); 
    } 

    private void cancel(){ 
     if(countDownTimer != null){ 
      countDownTimer.cancel(); 
      countDownTimer = null; 
     } 
    } 

    private void gameOver(){ 
     Toast.makeText(getApplicationContext(), "You scored " + count, Toast.LENGTH_SHORT).show(); 
     count = 0; 
     countFail = 0; 
     cancel(); 
    } 

    private void gameButton(){ 

     int foo = Integer.parseInt(time.getText().toString()); 

     if(foo % 2 == 0) { 
      Toast.makeText(getApplicationContext(), "PASS", Toast.LENGTH_SHORT).show(); 
      handler.removeCallbacks(r); 
      ++count; 
     } 

     else{ 
      gameOver(); 
     } 
    } 

} 

答えて

26

あなたはほぼpostDelayed(Runnable, long)を正しく使用していますが、あまりよくありません。あなたのRunnableを見てみましょう。我々はr.run();を呼び出すと

final Runnable r = new Runnable() { 
    public void run() { 
     handler.postDelayed(this, 1000); 
     gameOver(); 
    } 
}; 

はそれをどうするつもりだ最初のものは1000ミリ秒後にRunnableを非常に同じことを実行するために、あなたのhandlerを伝え、その後、gameOver()を呼び出すことです。これが実際に起こるのは、gameOver()メソッドが2回呼び出されることです。すぐに1回、ハンドラが1000ミリ秒待ってからもう一度呼び出されます。

代わりに、あなたはこれまであなたのRunnableを変更する必要があります。

final Runnable r = new Runnable() { 
    public void run() { 
     gameOver(); 
    } 
}; 

をそして、このようにそれを呼び出す:

handler.postDelayed(r, 1000); 

は、この情報がお役に立てば幸いです。

+0

ちょうど私が必要としたもの。簡単にそれを分解していただきありがとうございます。 –