javaプログラミングについて

質問

ただいまjavaプログラム勉強中でhit&blowを制作しております。
public static void main(String[] args) {

Scanner scan = new Scanner(System.in);
Random rand = new Random();
int[] answer = new int[4];
int[] input = new int[4];

int[] Number = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, };
int x = 10;
for (int i = 0; i < 4; i++) {
int select = (int) (Math.random() * x);
answer[i] = Number[select];
Number[select] = Number[x - 1];
x--;
}

int count = 1;

while (true) {
System.out.println("4桁の異なる数値を入力");
int str_input = scan.nextInt();
// 代入
for (int i = 0; i < 4; i++) {
input[i] = str_input;
}
// hit
int hit = 0;
for (int i = 0; i < 4; i++) {
if (input[i] == answer[i]) {
hit++;
}
}

// blow
int blow = 0;
for (int i = 0; i < 4; i++) {
for (int j = 0; j < 4; j++){
if (i != j && input[i] == answer[j]) {
blow++;
}
}
}

if (hit == 4) {

System.out.println("4hit");
System.out.println("正解まで " + count + "回");

break;
} else {
System.out.println("Hit:" + hit + " Blow:" + (blow - hit));
count++;
}
}
}
}
数字が4桁で作っているのですが、数字や4桁以上を入力してしまった場合の表示は後にしようとして、先にhitとblowの判定を作ろうとしているのですが、上記で実行したところhitとblowの判定がされずhit:０blow:0と表示されてしまいます。解決策を教えてください。
自分で作ってみたものの、hit blowの判定方法があっているかも自信がないです。

回答No.3

amanojaku1 · ベストアンサー

> int[] Number = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, };
> int x = 10;
> for (int i = 0; i < 4; i++) {
> int select = (int) (Math.random() * x);
> answer[i] = Number[select];
> Number[select] = Number[x - 1];
> x--;
> }

↑これは これで非常に正しいですが、この「Number」のように配列の要素を1つ抹消する場合は通常は「ArrayList」を使ったりします(「ArrayList」については下記ページ参照)。

クラス ArrayList<E>
https://docs.oracle.com/javase/jp/6/api/java/util/ArrayList.html

(1).(「hit」は容易に推測できますが)説明が非常に抽象的なので「blow」の処理が分かりにくいです。
(2).デバッグ用の「print」を各所に入れてチャント動いているか、チェックするようにしましょう(下記はデバッグ用の「print」を1つだけ残してあります)。

public static void main(String[] args) {
Scanner scan = new Scanner(System.in);
Random rand = new Random();
int[] answer = new int[4];
int[] input = new int[4];

int[] Number = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, };
int x = 10;
for (int i = 0; i < 4; i++) {
int select = (int) (Math.random() * x);
answer[i] = Number[select];
Number[select] = Number[x - 1];
System.out.println( // デバッグ用
"i="+i+"; "+
"x="+x+"; "+
"select="+select+"; "+
"answer[i]="+answer[i]+"; "+
"Number[select]="+Number[select]+"; "+
"");
x--;
}

int count = 1;

while (true) {
System.out.println("4桁の異なる数値を入力");
String str_input = scan.next();
char[] char_input = str_input.toCharArray();
// 代入
for (int i = 0; i < 4; i++) {
input[i] = Integer.parseInt(String.valueOf(char_input[i]));
}
// hit
int hit = 0;
for (int i = 0; i < 4; i++) {
if (input[i] == answer[i]) {
hit++;
}
}

// blow
int blow = 0;
for (int i = 0; i < 4; i++) {
for (int j = 0; j < 4; j++) {
if (i != j && input[i] == answer[j]) {
blow++;
}
}
}

if (hit == 4) {

System.out.println("4hit");
System.out.println("正解まで " + count + "回");

break;
} else {
// 「blow」を計算する上記のループでわざわざ
// 「if (i != j && input[i] == answer[j])」として
// 「hit」を排除しているのですから「(blow - hit)」のように
// 「blow」から「hit」を引く必要はないのでは？
System.out.println("Hit:" + hit + " Blow:" + blow);
count++;
}
}
}

回答No.2

maiko0333

＞ if (i != j && input[i] == answer[j]) {
input[i]はキー入力した４桁。anserは計算した１桁ですのでこの条件には当てはまらない。

回答No.1

y_shimizu

hit&blowとやらは知りませんが、Scanner.nextIntでどんな値を入力しているのかによるのでなんとも
入力値をinputの配列全部に同じ値入れてるので、そこかな？