JAVAの配列がよくわかりません

Question

３桁の数字を入力して、数も場所もあっている（ヒット）のは何個で、数だけあっている（ブロー）は何個かを随時教えてくれるプログラムを書いています。
正解の数字は乱数で出しています。また、何回もできるようにループさせています。
答え合わせのしやすいように、初めに正解の数字は表示させています。
解答者が入力した数字をi[a]という配列(?)に代入したところくらいからよくわからなくなってしまいました。
現在何を入力しても０個と０個が表示されてしまいます。
java初心者のため、どこがどう違うのか分かりやすく教えていただけると幸いです。

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

public class Main {
  public static void main(String[] args) {
	  int[] answerNumList = new int[3];
	  System.out.println("３桁の数字を当ててください");
	  System.out.println("３桁の数字は重複しません");
	  System.out.println("０から始まる数字も可能です(例 012)");
	  System.out.println("数字と場所が合っている場合はヒット、数字だけが合っていたらブローとしてその数が表示されます");
	  System.out.print("正解 ");
	  //０～９が入ったList を生成
	  ArrayList<Integer> tempNumList = new ArrayList<Integer>();
	  for(int i = 0; i < 10; i++){
		  tempNumList.add(i);
	  }
	  //Listからランダムに数字を取り出して３桁の数を生成
	  for(int i = 0; i <answerNumList.length; i++){
		  int selectNum = (int)(Math.random() * tempNumList.size());
		  answerNumList[i] = tempNumList.remove(selectNum);
	  }
	  for(int i=0; i < answerNumList.length; i++){
		  System.out.print(answerNumList[i]);
		  if(i != answerNumList.length -1)
			  System.out.print( "  ");
	  }
	  //入力後のステップ
	  while(true){
		  try{
			  System.out.println();
			  System.out.println("３桁の数字を入力してください。");
			  BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
			  String str = br.readLine();
			  
			  int num = Integer.parseInt(str);
			  int a[]=new int[num];
			  
			  //エラーの初期化、エラーをfalseにしておく
			  boolean error = false;
			  int length = str.length();
			  //３桁でない数が入力されたときエラーをtrueにする
			  if (length != 3)
				  error = true;
			  //同じ数が入力されたときエラーをtrueにする
			  for(int i = 0 ; i < length; i++) {
				  for(int j =i +1; j < length; j++) {
					  if(str.charAt(i) == str.charAt(j))
						  error = true;
				}
			  }
			  //エラーの時の画面表示
			  if(error == true){
				System.out.println("入力エラーです。");
				System.out.println("０～９までの数値を３桁で重複しないように入力してください。");
			}
			  
			  if(error == false){
			  //hitの初期化
			  int hit = 0;
			  for(int i = 0; i < answerNumList.length; i++)
			  {
				  if ( a[i] == answerNumList[i])
				  {
					  hit++;
				  }
			  }
			  //blowの初期化
			  int blow = 0;
			  for(int i = 0; i < 3; i++)
			  {
				  for(int j = 0; j <answerNumList.length; j++)
				  {
					  if( a[i] == answerNumList[j])
					  {
						  blow++;
					  }
				  }
			  }
			  if(hit == answerNumList.length)
			  {
				  System.out.println("正解です.");
			  }
			  else
			  {
				  System.out.println("ヒット" + hit + "個," + "ブロー" + (blow - hit)+ "個");
			  }
			  }

			  
		  }
		  catch(IOException e){
		  e.printStackTrace();
		  }
	  }

  }
}

Answer 1

String str = br.readLine();
int num = Integer.parseInt(str);
int a[]=new int[num];

これは、例えば"123"と入力した際、長さ123の配列を作成している事になります。どんな数を入れようともこの配列の中身はすべて0なので、答えの数値に0が入っていないかぎりヒットもブローもしないことになります。

###ではどうするか？

• 文字列の各文字を取り出して、'0'との差で数値にする(数値であることのチェックなどが事前に必要)

String str = br.readLine();
int[] a = new int[answerNumList.length];

for (int i = 0; i < answerNumList.length; i++) {
    a[i] = str.charAt(i) - '0';
}

• 文字列を分解し、各文字をparseInt

String str = br.readLine();
int[] a = new int[answerNumList.length];
String[] split = str.split("");

for (int i = 0; i < answerNumList.length; i++) {
    a[i] = Integer.parseInt(split[i]);
}

• 数値変換し、剰余算で各桁を取り出す(コード略)

など、やりようはいくらかあります。

---

なお、他にも

• 正解の数設定が各桁ランダムになる保証がないので、shuffleなどを使って並べ替えてから取り出すべき
• ヒット、ブローの判定が二度手間。
• メソッドに切り分けて処理するべき(入力が適正かどうかの判定など)

など、いくつか問題を抱えているようです。

Comments

[ain4425]

回答ありがとうございます。では、配列そのものの中身に数を入れたければどう表記すればいいのでしょうか

[ain4425]

各文字の分解が自分なりに分かりやすいのでそれでやっていこうと思います。いろいろなやり方を提示していただきありがとうございました。
その他の問題点も含め、勉強していこうと思います！