2016-04-25 9 views
0

こんにちは私はクイズゲームを作っています。私は問題を抱えています。私は下の簡単な方法でディスパッチしようとします。質問と4つのボタンが答えを表します。私は答えをクリックする私は2秒間、オレンジ色に色を変えて、そして答えが正しければ緑色に変わり、答えが間違っていて正しい答えが緑色に変わって、もう2秒前にこの状態にとどまる新しい質問を持ち、すべてのボタンは以前の(デフオート)色になります。Javaで一定期間実行を停止する

私はTimeUnit.SECONDS.sleep(2);,Thread.sleep(2000);などの多くの方法を試しましたが、その期間が経過した後に変更が表示されます。今度は2秒間表示する必要があります。

私が試した最後の方法は、このようなタイマーです:

 long startTime = System.currentTimeMillis(); 
    long endTime = startTime + 2*1000; // 2 seconds * 1000 ms/sec 
    while (System.currentTimeMillis() < endTime) 
    { 

    } 

2秒間whileループに実行を停止します。 ボタンをクリックすると、ボタンの色が2秒間赤に変わってから前の色に戻っていきたいと思っています。ここ は、Javaコードである:ここで

package com.example.sakis.fillbutton; 

import android.os.Handler; 
import android.support.v7.app.AppCompatActivity; 
import android.os.Bundle; 
import android.view.View; 
import android.widget.Button; 

import java.util.Timer; 
import java.util.TimerTask; 
import java.util.concurrent.TimeUnit; 

public class MainActivity extends AppCompatActivity { 

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

    but=(Button)findViewById(R.id.button); 
    but.setBackgroundResource(R.drawable.colorbut); 


} 




public void ClickButton(View view) { 




    but.setBackgroundResource(R.drawable.initcolor); 

    long startTime = System.currentTimeMillis(); 
    long endTime = startTime + 2*1000; // 2 seconds * 1000 ms/sec 
    while (System.currentTimeMillis() < endTime) 
    { 

    } 


    but.setBackgroundResource(R.drawable.colorbut); 

} 

}

XMLコードです:

<?xml version="1.0" encoding="utf-8"?> 
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" 
xmlns:tools="http://schemas.android.com/tools" 
android:layout_width="match_parent" 
android:layout_height="match_parent" 
android:paddingBottom="@dimen/activity_vertical_margin" 
android:paddingLeft="@dimen/activity_horizontal_margin" 
android:paddingRight="@dimen/activity_horizontal_margin" 
android:paddingTop="@dimen/activity_vertical_margin" 
tools:context="com.example.sakis.fillbutton.MainActivity"> 

<TextView 
    android:layout_width="wrap_content" 
    android:layout_height="wrap_content" 
    android:text="Hello World!" 
    android:id="@+id/textView" /> 

<Button 
    android:layout_width="wrap_content" 
    android:layout_height="wrap_content" 
    android:text="New Button" 
    android:id="@+id/button" 
    android:layout_below="@+id/textView" 
    android:layout_centerHorizontal="true" 
    android:layout_marginTop="88dp" 
    android:onClick="ClickButton" 
    style="@style/ButtonAnswer"/> 

助けてください。

答えて

4
while (System.currentTimeMillis() < endTime) {} 

ビジーループが悪いあります。絶対にしないでください。 Wiki quote

一般的に[...]紡糸抗パターンとみなされ、 を避けなければならない、別のタスクではなく、無駄な活性に浪費さ を実行するために使用することができるプロセッサ時間として。戻る質問へ

答えが正しいと赤であれば私が選ばれたボタンを2秒間オレンジに色を変更したいの答えをクリックして、緑になるとあれば答え

Runnableを使用して、必要な遅延で投稿することができますが、あまり心配する必要はありません。遅延が発生すると、Runnableが処理されます。単にあなたのクラスのメンバーにこれを追加します。

Handler mHandler = new Handler(); 

してから作成し、ポストは、必要に応じて遅れるほどの取り扱いを持つ実行可能な言った:

Runnable runnable = new Runnable() { 
    @Override 
    public void run() { 
     changesColorsToGreenOrRed(); 
    } 
}; 
mHandler.postDelayed(runnable, 2000); 

ともたらす前に、別の2秒間、このような状況にとどまります新しい質問

は、仕事をしてから別のRunnableを投稿する必要があります。

+0

答えは明らかですが、ビジーループが悪い理由を説明することで改善されます。 – nhouser9

+0

https://en.wikipedia.org/wiki/Busy_waiting "しかし、一般に、回転は反パターンとみなされるので、別のタスクを実行するために使用できるプロセッサ時間が無駄な活動に無駄になるため避けてください。 " –

+0

ありがとう、それは動作します! –

関連する問題