桁数を判定して、４桁の数が入力されるまで繰り返すメソッド

Question

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
Javaで４桁の乱数を当てるHitBlowを作っています。５桁以上なら「多すぎます。４桁になるようにしてください。」３桁以下なら、「桁が少ないです。４桁の整数を入力してください。」と表示され,４桁なら以降に続くようにしたいです。
###発生している問題・エラーメッセージ

エラーメッセージ
ArrayIndexOutOfBorderException
もともと指定した配列の要素数を超えた値を入力されたためエラーが出ました。

###該当のソースコード

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class HitBlow {
	public static void Ransuu() {
		//４桁の重複ない乱数を作り値を送るメソッド
		int[] answer = new int[4];
		for (int i = 0; i < answer.length; i++) {
			Loop: while (true) {
				answer[i] = (int) (Math.random() * 10); //0~9の数値を入れる
				for (int j = 0; j < i; j++) {
					//その前までの数値のどれかとかぶっていたら数値代入からやり直し
					if (answer[j] == answer[i] || answer[0] == 0)
						continue Loop;
				}
				break;
			}
		}
//とりあえず出力させています。
		for (int x : answer) {
			System.out.print(x);
		}
	}

	public static void Ketasuu(int n) {
		//桁数を判定するメソッド
		int digits = 0;
		while (n > 0) {
			digits++;
			n /= 10;
		}
		if (digits > 4) {
			System.out.println("桁数が多いです。桁数を４桁にしてください");
		} else if (digits < 4) {
			System.out.println("桁数が少ないです。桁数を４桁にしてください");
		}
	}

	public static void toInt(String[] n, int[] m) {
		//String型の要素をint型配列に格納するメソッド
		for (int i = 0; i < n.length; i++) {
			m[i] = Integer.parseInt(n[i]);
		}
	}

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
		Ransuu();
		BufferedReader a = new BufferedReader(new InputStreamReader(System.in));
		try {
			int num = 0;
			String[] input = new String[4];
			int[] input2 = new int[4];
			Flag: while (true) {
				String b = a.readLine();
				Ketasuu(num = Integer.parseInt(b));

				input = b.split("", 0);
				toInt(input, input2);
			}
		} catch (IOException e) {
			System.out.println(e);
		}
	}
}

###試したこと
課題に対してアプローチしたことを記載してください
boolean型のメソッドを作り、メソッド内で桁数を判定したのですが、true falseの２択で両方continue させたいので分岐が足りませんでした。４桁より多い、少ない、４桁の３択で分岐させたいです。
###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

単に『4桁であるか』が分かればいいのであれば、1000 <= x < 10000で充分なのでは。
ただし、期待される入力は正の整数とします。

サンプル

『数字に重複のない、n桁の数値を返す静的メソッド』のサンプル

static int retNDigitsNumWithoutOverlap(int digitNum) {
    if(digitNum <= 0) throw new IllegalArgumentException();
    if(10 < digitNum) throw new IllegalArgumentException();
    
    List<Integer> digits = Arrays.asList(0, 1, 2, 3, 4, 5, 6, 7, 8, 9);
    do {
        Collections.shuffle(digits);
    } while(digits.get(0) == 0);

    int ret = 0;
    for(int digit: digits.subList(0, digitNum)) {
        ret *= 10;
        ret += digit;
    }

    return ret;
}

---

『正の整数の桁数を計算する静的メソッド』のサンプル

static int computeNumOfDigit(int num) {
    if(num < 0)  throw new IllegalArgumentException();
    if(num == 0) return 1;

    return (int)Math.log10(num) + 1;
}

巨大な数が入力されないと保証できるのであれば、次のように書いても良いでしょう。

static int computeNumOfDigit(int num) {
    if(num < 0)  throw new IllegalArgumentException();
    for(int i = 1, p = 10; true; ++i, p *= 10) {
        if(num < p) return i;
    }
}

『n桁の数値が入力されるまで、再入力を求める静的メソッド』のサンプル

static int scanInteger(BufferedReader br, String message) throws IOException {
    while(true) {
        try {
            System.out.print(message);
            return Integer.parseInt(br.readLine());
        } catch(NumberFormatException e) {}
    }
}
static int scanNDigitInteger(BufferedReader br, String message, int digitNum) throws IOException {
    while(true) {
        int input = scanInteger(br, message);
        if(computeNumOfDigit(input) == digitNum) return input;
    }
}

実行イメージは以下のとおりです。

input 4 digit num: hoge
input 4 digit num: asffdsa
input 4 digit num: 23
input 4 digit num: 34234
input 4 digit num: 3425
3425

Comments

[TSho]

回答ありがとうございます。４桁より多いか少ないか４桁かの３択限定です。

[LouiS0616]

それならif(x < 1000) {} else if(x < 10000) {} elseが一番素直かと。

[LouiS0616]

いろいろと難しく考えすぎだと感じたので、ちょっとサンプルを書いてみました。

[TSho]

ありがとうございました。たしかに簡単なことを難しく考えてしまっていました。

Answer 2

読みにくいです。

ソースコードは"</>" で出てくるやつの 「中に」記述してください。

外側になっていますよ。

Boolは true/falseの2値です。

よって3択以上には適さないです。

ではどうするか。

intで代用する。

少ない: -1
ちょうど: 0
多い: 1

みたいに。

別に-1,0,1 じゃなくてもいいです。あるいは桁数を取得するメソッドを作り、

戻り値が 4 なら ちょうどいい。

...みたいに。

桁数を取得する場合は、

私なら

方法1:
数直線で考えて、4桁になるように考える

たとえば、

4桁って 1000～9999 ですよね。

この範囲に入っているかどうかで考える。

多い、少ない...だったら、

少ない: 1000 未満 ( 999以下 )
多い: 10000以上

とか。

後は邪道ですが、

方法2:
文字列として考える

文字列なら Stringのlengthメソッドで取得すればいい。

Comments

[TSho]

回答ありがとうございます。intを作ってその値を返すやり方でやってみます！