1度ボタンを押して複数回表示

前提・実現したいこと

android studioで1度ボタンを押すとfor文が起動し、出た乱数によって方向の違う矢印を4回表示させるプログラムを書いています。しかし以下のプログラムで書いたところボタンを押すと1回目の矢印は出るのですが2回目、3回目と違う向きの矢印が出ません。実行時にはエラーが出ませんでした。
もしよろしければ4回矢印を表示させるにはどうすればいいかご教示頂けますでしょうか？

該当のソースコード

java
1import java.util.Random;
2
3
4public class MainActivity extends AppCompatActivity {
5
6    int i=0;
7    int x[] = new int[4];
8
9    @Override
10    protected void onCreate(Bundle savedInstanceState) {
11        super.onCreate(savedInstanceState);
12        setContentView(R.layout.activity_main);
13
14        Button button_sta=(Button)findViewById(R.id.button_sta);
15
16        final TextView textView = (TextView) findViewById(R.id.text);
17
18        button_sta.setOnClickListener(new View.OnClickListener(){
19            @Override
20            public void onClick(View v){
21                Random rand=new Random();
22                int num;
23                for(i=0;i<4;i++){
24                    num=rand.nextInt(4);
25                    if(num==0) {
26                        textView.setText("←");
27                        x[i] =num;
28                    }
29
30                    else if(num==1) {
31                        textView.setText("↑");
32                        x[i] = num;
33                    }
34
35                    else if(num==2) {
36                        textView.setText("→");
37                        x[i] = num;
38                    }
39                    else if(num==3) {
40                        textView.setText("↓");
41                        x[i] = num;
42                    }
43                }
44
45            }
46        });
47    }
48}

行動規範の内容に同意します

回答2件

ベストアンサー

問題

textView.setText が4回連続で表示されており, 現在の挙動は最後の4回目にrand.nextInt(4)で取得した値に対応するものが表示されているものと見受けられます. これは, 全4回の実行が間を置かずに実行されているためです.

解決策

全4回の実行に, それぞれ間を空けて実行をするとよいかと思われます. これを実現するための, 一つの方法としては, Handler.postDelayed を使用するのが最も楽かと思われます.

コード

次のように, forの中身を new Handler().postDelayed(() -> { ... }) で囲ってあげることで, for分の全4回の処理は即座に実行されますが, postDelayed によって, 第２引数に指定した時間(ミリ秒)後に実行させることができます. 次の例では, i * 1000L としており, これは button_sta をタップしてから0ms後に1回目, 1000ms後に2回目, 2000ms後に3回目, 3000ms後に4回目となります. もしこの間隔を変更したい場合は, i * 1000L の部分を適宜変更してください.

java
1for (i = 0; i < 4; i++) {
2  new Handler().postDelayed(new Runnable() {
3    @Override
4    public void run() {
5      int num = rand.nextInt(4);
6      // 値に応じた分岐処理と表示
7    }
8  }, i * 1000L);
9}

または, 以下のように記述することも出来るかと思います.

java
1for (int i = 0; i < 4; i++) {
2    new Handler().postDelayed(() -> {
3        int num = rand.nextInt(4);
4        // 値に応じた分岐処理と表示
5    }, i * 1000L);
6}

補足

値に応じた分岐処理と表示 の処理は, より簡潔に記述することができます. 具体的には以下のように記述できます.

java
1
2// クラス直下などに定義
3private String[] texts = {"←", "↑", "→", "↓"};
4
5// onClick 内の処理
6Random rand = new Random();
7for (i = 0; i < 4; i++) {
8  new Handler().postDelayed(() -> {
9    int num = rand.nextInt(4);
10    textView.setText(texts[i]);
11    x[i] = num;
12  }, i * 1000L);
13}