javaプログラムで、ヒット&ブローを作っています！

Question

import java.util.Random;
import java.util.Scanner;


public class hitandblow01 {

	public static void main(String[] args)throws Exception {
		Random random = new Random();
		var sc =new Scanner(System.in);

		ArrayList<Integer>ans = new ArrayList<Integer>();

		ArrayList<Integer>num1 = new ArrayList<Integer>();
		for(int i = 0; i < 10; i++) {
			num1.add(i);
		}
		for(int i = 0; i < 4; i++) {
			int index = random.nextInt(num1.size());
			ans.add(num1.remove(index));
		}
		System.out.println(ans);
		int count = 1;

		while(true) {
			ArrayList<Integer>player = new ArrayList<Integer>();

			System.out.print("4桁の数字を入力して下さい。");

			int num2 = sc.nextInt();
			System.out.println(num2);
			player.add((num2 / 1000)% 10);
			player.add((num2 / 100)% 10);
			player.add((num2 / 10)% 10);
			player.add(num2 % 10);

			int hit = 0;
			for(int i =0; i < 4; i++) {
				if(player.get(i)== ans.get(i)) {
					hit++;
				}
			}
			int blow = 0;
			for(int i = 0; i < 4; i++) {
				for(int j = 0; j < 4; j++) {
					if(player.get(i)==ans.get(j)) {
						blow++;
					}
				}
			}
			if(hit==4) {
				System.out.println("おめでとう！"+count+"回目で成功♪");
			}
			System.out.println("ヒット："+hit+"個、ブロー："+(blow - hit)+"個");
			System.out.println();
			count++;
		}
	}

}



コード

javaでヒット&ブローを作りました！
改良するために三回に一回使われている数字を出すにはどうすれば良いのか分からず困ってます。
どなたか教えてください！

Comments

[m.ts10806]

何をもとに何を教えれば良いのでしょうか。
「作ってくれ」という作業依頼でしょうか

[azuapricot]

教えて下さいって丸投げもいいところですしせめて自分で作ったコードは質問に提示しましょうね
こんな質問上司や先生にしたら怒られるにきまってます。
質問をする前に一度読み直してから投稿しましょうね

[m.ts10806]

せめて「質問するときのヒント」を読んで質問テンプレートを使ってください。
質問は編集できますので。

[jimbe]

> 三回に一回使われている数字を出す
の仕様が分からないのですが, 具体的にどういうことでしょう.

[m.ts10806]

マークダウンはヘルプなどを確認し正しく使ってください。平で提示されても読む気をなくすだけです。

[dit.]

1行目（import java.util.ArrayList;）消えちゃってるんで、```の後は改行してください。

Answer 1

ベストアンサーが付きましたので, 勉強中とのことで蛇足です.

※ 以下のコードは動作確認はしていません

  int hit = 0;
  for(int i =0; i < 4; i++) {
    if(player.get(i)== ans.get(i)) {
      hit++;
    }
  }
  int blow = 0;
  for(int i = 0; i < 4; i++) {
    for(int j = 0; j < 4; j++) {
      if(player.get(i)==ans.get(j)) {
        blow++;
      }
    }
  }

これでは hit 判定された数字が blow でもカウントされてしまうのではないでしょうか.
hit した数字は blow チェックをしないように

  int hit = 0;
  int blow = 0;
  for(int i = 0; i < 4; i++) {
    if(player.get(i) == ans.get(i)) {
      hit++;
    } else {
      for(int j = 0; j < 4; j++) {
        if(player.get(i) == ans.get(j)) {
          blow++;
        }
      }
    }
  }

とされては如何でしょう.

また, この場合は List.contains というメソッドで j のループを置き換えられると思います.

  int hit = 0;
  int blow = 0;
  for(int i = 0; i < 4; i++) {
    if(player.get(i) == ans.get(i)) {
      hit++;
    } else if(ans.contains(player.get(i))) {
      blow++;
    }
  }

さらについでに, "4" というのは ans の件数ですので,

  int hit = 0;
  int blow = 0;
  for(int i = 0; i < ans.size(); i++) {
    if(player.get(i) == ans.get(i)) {
      hit++;
    } else if(ans.contains(player.get(i))) {
      blow++;
    }
  }

とすると, 4桁以外に変えたとしてもこの個所はこのまま使用できることになります.

Comments

[ta.tatu]

丁寧な補足ありがとうございます。
助かります！

Answer 2

三回に一回使われている数字

何回か失敗していると、ヒントとして3番目の答えの数字を表示させるって考えでいいのかな？
後、このプログラムには終わりがないので、つけておきました。

if(hit==4) {
    System.out.println("おめでとう！"+count+"回目で成功♪");
    break;
}else{
    //追加
    if(count == 5){
        System.out.println("５回間違えたのでヒントだよ。３番目の数字は、" + ans.get(2) + "だ");
    }
}

Comments

[ta.tatu]

ありがとうございました。まだまだ勉強不足でした！
勉強始めてまだ3ヶ月なのでもっと勉強します！

[stdio]

どれだけ勉強しているのか分かりませんが、頑張って下さい。