作ったビンゴアプリが強制終了してしまいます

Question

前提・実現したいこと

android studioでandroidアプリ開発のための学習をしている初学者です。
言語はjavaです。

「たった1日で基本が身に付く　androidアプリ開発超入門」という本のビンゴアプリを作成しました。
問題なく完成したのですが、タップをし続けると強制終了してしまいます。
この原因を突き止めて強制終了を回避したいのです。

発生している問題

出目の最大値は初期値として75を設定してあるのですが、任意の数字に変更できます。
アプリが完成した後に夫に実機で試してもらいました。

夫がタップしながら「これってちゃんと最大値まで全部出るのかな」というので、
「じゃあ最大値を5に設定して全部の出目がでるかどうか見てみたら」とすすめて15全部の数字が出るか確認してもらいました。
ランダムに15の数字が出た後もタップしていたら、フリーズしたあとに強制終了しました。
エミュレーターでも同じようにフリーズしたあとに強制終了しました。

ログで出た内容は以下の通りです。
出目が揃ったあとは「重複したので再生成」と「uid=10135～」をずっと繰り返してます。

D/MainActivity: maxNumber: 5
D/MainActivity: [4]
D/MainActivity: [4, 2]
D/MainActivity: 重複したので再生成
D/MainActivity: [4, 2, 5]
D/MainActivity: [4, 2, 5, 3]
D/MainActivity: 重複したので再生成
D/MainActivity: [4, 2, 5, 3, 1]

D/MainActivity: 重複したので再生成
I/chatty: uid=10135(com.example.mybingo2) identical 77 lines
D/MainActivity: 重複したので再生成
D/MainActivity: 重複したので再生成
I/chatty: uid=10135(com.example.mybingo2) identical 34958 lines
・
・
・

該当のソースコード

package com.example.mybingo2;

import androidx.appcompat.app.AppCompatActivity;

import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.Button;
import android.widget.EditText;
import android.widget.TextView;

import java.util.ArrayList;


public class MainActivity extends AppCompatActivity {

//    最大値を管理するための変数を用意する。これはインスタンス変数・・アクティビティ内のすべてのメソッドからアクセスできる。他にローカル変数がある・・メソッドの｛｝で囲まれた範囲内で有効
//    アクセス修飾子privateはクラスの中でしか使えない。publicはクラスの外からでも使える。intはデータ型。maxNumberは変数名。75を代入。
    private int maxNumber = 75;
//    「数字の履歴」のインスタンス変数を宣言
//    privateはクラスの中でしか使えないアクセス修飾子。ArrayList<String>は文字列型のデータ型。historyは変数名。
    private ArrayList<String> history = new ArrayList<>();
//    「最大値の入力欄」のインスタンス変数を宣言
//    privateはクラスの中でしか使えないアクセス修飾子。EditTextはデータ型。maxNumberEditTextは変数名。
//    EditText や Button は、Android が提供するクラスであり、データ型の一種。
    private EditText maxNumberEditText;
//    「最大値の設定ボタン」のインスタンス変数を宣言
//    privateはクラスの中でしか使えないアクセス修飾子。Buttonはデータ型。registerMaxNumberButtonは変数名。
    private Button registerMaxNumberButton;
//    「次の数字を出すボタン」のインスタンス変数を宣言
//    privateはクラスの中でしか使えないアクセス修飾子。Buttonはデータ型。nextNumberButtonは変数名。
    private Button nextNumberButton;
//    「現在の数字を表示するTextView」のインスタンス変数を宣言
//    privateはクラスの中でしか使えないアクセス修飾子。TextViewはデータ型。currentNumberTextViewは変数名。
    private TextView currentNumberTextView;


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

//        ビューの変数を初期化する
        maxNumberEditText = findViewById(R.id.max_number);
        registerMaxNumberButton = findViewById(R.id.register_max_number);
        nextNumberButton = findViewById(R.id.next_number);
        currentNumberTextView = findViewById(R.id.current_number);


//      最大値の初期値をEditTextにセットする。画面に75が表示される。
        maxNumberEditText.setText("" + maxNumber);

//      最大値を更新する
        registerMaxNumberButton.setOnClickListener(new View.OnClickListener(){

            @Override
            public void onClick(View v){
//                入力値を文字列で取り出す
                String maxNumberString = maxNumberEditText.getText().toString();
//                int型に変換してから代入する。EditTextは入力値を文字列でしか扱うことができない。
                maxNumber = Integer.valueOf(maxNumberString);

                Log.d("MainActivity", "maxNumber: " + maxNumber);

            }
        });

//        表示中の数字を更新する
        nextNumberButton.setOnClickListener(new View.OnClickListener(){
            @Override
            public void onClick(View v){
                onClickNextNumber();
            }
        });
    }

//    nextNumberButtonがタップされたときの処理
    private void onClickNextNumber(){
//        Log.d("MainActivity","onClickNextNumber");

//      maxNumberを考慮したランダムな数値
        int nextNumber = createRandomNumber();

//      重複している数値だった場合は数値の生成をやり直す
        while (history.contains("" + nextNumber)){
            Log.d("MainActivity", "重複したので再生成");
            nextNumber = createRandomNumber();
        }

//      nextNumberを文字列に変換する
        String nextNumberStr = "" + nextNumber;

//      nextNumberを画面に表示する
        currentNumberTextView.setText(nextNumberStr);

//      履歴を残す
        history.add(nextNumberStr);
        Log.d("MainActivity", history.toString());

//        「1～75のランダムな数値を生成したい」
//        0.0~74.0の数値を生成する（最大値が初期値の75の場合）
//        Math.randomメソッドは0.0～1.0未満までのdouble型の小数値をランダムに生成する。これに75（-1しない。75のまま）をかけて、0.0～75.0未満にする。
//        double randomNumber = Math.random() * (maxNumber);
//        1～75の整数値を生成する（最大値が初期値の75の場合）
//        doubleからintへの縮小変換キャストをして、0～74にする。（小数点から整数にする）さらに+1で1～75にして、nextNumberへ代入。
//        int nextNumber = (int) randomNumber + 1;

//        nextNumberを画面に表示する。
//        currentNumberTextView.setText("" + nextNumber);

//        Log.d("MainActivity","nextNumber: " + nextNumber);
    }

//      maxNumberを考慮したランダムな数値を生成する
        private int createRandomNumber(){
//        0.0~74.0の数値を生成する
            double randomNumber = Math.random() * (maxNumber);
//        1~75の整数値を生成する
            return (int) randomNumber + 1;
        }



//         Log.d("MainActivity", "maxNumber: " + maxNumber);
}

試したこと

①出版元のサポートページを確認しましたが、とくに該当の記述はありませんでした。
②入力ミスがないかチェックして、最初から順番にエラーの確認をしました。
以下のコードの後に問題のエラーが出たように思いました。

//      重複している数値だった場合は数値の生成をやり直す
        while (history.contains("" + nextNumber)){
            Log.d("MainActivity", "重複したので再生成");
            nextNumber = createRandomNumber();
        }

何か間違って入力しているのか、足りないコードがあるのかわかりません。
よろしくご教示ください。
ビンゴアプリなので、1つずつ数字を出していって、最大値以上にタップすることはあまり無いかとは思いますが、
もし対策があるなら試してみたいのです。

Comments

[1T2R3M4]

1回すべての数字が出てしまうと無限ループになるのでは。

Answer 1

通常のビンゴであれば75個数字が出きってしまう前にビンゴしますから、数字を出し尽くした際の考慮はしてなかったのでしょうね。

打ち止めを検知する方法ですが、いろいろやり方はあります。簡単なやり方を2つ紹介します。

• 使用した数字の数をチェックする

今回の場合はhistoryに履歴が残っていますので、historyのsize()と maxNumber を比較し、同数になったら打ち止めにすることができます。

• 予め出現予定のリストを作る

質問のコードでは毎回ランダムに数字を抽選していますが、予め 1からmaxNumberまでの数字が入ったリストを用意し、リストの内容をシャッフルします。数字の抽選に変えてリストの中身を先頭から1つずつ取り出してremoveしていきます（最後から取り出しても良い）。このリストが空になったらゲーム終了です。

前者のほうが今の実装を大部分活かせるので、まずは前者の方式で打ち止めを検知してみてください。

Comments

[curacura]

ありがとうございます！打ち止め方式でやってみます。

Answer 2

//      重複している数値だった場合は数値の生成をやり直す
        while (history.contains("" + nextNumber)){
            Log.d("MainActivity", "重複したので再生成");
            nextNumber = createRandomNumber();
        }

数字が全部出尽くしてしまった場合、次に生成した数がいくつであろうと再抽選することになるので、
このループから一生抜け出せなくなります。つまりフリーズします。

で、「どうしたらよいか」なのですが、「数字が全部出尽くしたらどうしたいのか」を決めてそのとおり実装してください。

Comments

[curacura]

数字が全部出尽くしたら「次の数字を出す」ボタンが押せないようにしたいのですが、これは可能でしょうか？

[ozwk]

はい可能です。頑張ってみてください

[curacura]

ありがとうございます！やってみます。