4桁の数字を入れた配列をfor文で抽出したい

Question

前提・実現したいこと

乱数と入力した4桁の数字の配列をfor文で回して同じ桁に同じ数字がある場合〇違う桁に同じ数字がある場合△となるプログラムを作りたいのですがforまでいかず
数字を入力して無限ループになります

該当のソースコード

package game;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.Scanner;

public class MathGame {

    public static void main(String[] args) {
        // TODO 自動生成されたメソッド・スタブ
        System.out.println("ゲームを始めます");
        int answer[] = new int[4];
        answer[0] = (int) (Math.random() * 9) + 1;
        for (int i = 1; i < answer.length; i++) {
            int num = -1;
            for (boolean isNotExist = false; !isNotExist;) {
                num = (int) (Math.random() * 10);
                isNotExist = true;
                for (int j = 0; j < answer.length; j++) {
                    if (answer[j] == num) {
                        isNotExist = false;
                        break;
                    }
                }
            }
            if (num < 0) {
                // ここにきたらバグ
                throw new IllegalStateException();
            }
            answer[i] = num;
        }
        Arrays.stream(answer).forEach(System.out::println);

        int input[] = new int[4];
        boolean validated = false;
        while (!validated) {
            int a = 0;
            for (int n = 0; a < input.length; a++) {
                System.out.println("4文字の数字を入力してください");
                String sc = new Scanner(System.in).nextLine();
                if (sc.length() == 4) {
                    System.out.println("4文字だけ入力されました。");
                    validated = true;
                } else {
                    System.out.println("4文字だけ入力してください。");
                    break;
                }
              }
            }
                    List<Integer> result = new ArrayList<Integer>();
                    for(int a : answer){
                       for(int b : input){
                         if(a == b){
                        	 result.add(a);
                           break;
                }
            }
        }
        System.out.println("ListA = " + answer);
        System.out.println("ListB = " + input);
        System.out.println("ListC = " + result);
          }
        }

Comments

[tetutetu]

>入力した数字がfor文に入らないのですが
そもそもコンパイル通らないと思うのですが。

[oisii]

はいコンパイルエラーになりました。
intで回すことは出来ないのでしょうか？

[dodox86]

実際、Java8でコンパイル試したところコンパイルが通りませんでした。質問者さんが使っている正しい（？）コードを質問文中に貼り直してください。

[stdio]

> intで回すことは出来ないのでしょうか？
可能です。出来ればforの書き方を合わせて下さい。
「List<String> reList = new ArrayList<String>();」付近のプログラムが貴方のものではないのはバレバレです。

それと質問ですが、「for (boolean isNotExist = false; !isNotExist;) 」は何？

[oisii]

>実際、Java8でコンパイル試したところコンパイルが通りませんでした。
はい、このままだとコンパイルが通らなく困っています。知識がなく申し訳ございません

[tetutetu]

>はいコンパイルエラーになりました。
現在解決すべき問題は
>入力した数字がfor文に入らない
でも
>intで回すことは出来ないのでしょうか？
でもなく、まずコンパイルエラーを解消することでは？
つぎはぎで作った結果変数名、データ型、スコープが全てメチャクチャになってます。

[oisii]

>それと質問ですが、「for (boolean isNotExist = false; !isNotExist;) 」は何？
乱数生成時に4桁の乱数の数字が重複しないよう書きました。

[yoorwm]

質問する前にまずは真面目にJavaを学習するべきでは？

[jimbe]

> forまでいかず数字を入力して無限ループになります
どこの for でしょう.
どのような数字を入力して無限ループとなったのか, 具体的な入力と表示の流れをご提示願えますか.

Answer 1

最初に answer を求めていますが、その answer には 0 が含まれることがありません。
乱数で 0～9 を求めて、既に出た数と比較して同じものがあるかどうか確かめないと
いけないのに、4つの数すべてと比較していますから。

配列に乱数4桁を格納して画面出力が解決済みになっているのに、
その回答が活かされていませんね。

Arrays.stream(answer).forEach(System.out::println);
これは意味が分かって使っていますか？
なぜ元は print だったのを println にしたんですか？

List<Integer> result = newe ArrayList<Integer>(); 以降、
インデントが変なのはなぜですか？

Answer 2

現在の質問者様ですと希望の実装は難しいと思います。
実務で対応しているようでしたら上長に報告、勉強でしたら一度javaの基礎を固めた方がよろしいと思います。

• 参考

プロゲート

for文、拡張for文、forEachの使い方

Answer 3

以下は動作している Hit&Blow のコードですが, 参考のみでお願いします.
入力は inputAnswer メソッドです.
judge メソッドで二つの int 配列を比較して hit と blow をカウントしています.
表示の〇△の文字列化は, Judgment クラスの getImageString メソッドが行っています.

import java.util.*;

public class HitAndBlow {
  private static final int DIGITS = 4;
  public static void main(String[] args) {
    System.out.println("ゲームを始めます");
    try(Scanner scanner = new Scanner(System.in);) {
      int[] correct = createCorrect(DIGITS);
      while(true) {
        int[] answer = inputAnswer(DIGITS, scanner);
        Judgment result = judge(correct, answer);
        System.out.println(result.getImageString());
        if(result.exactMatch()) break;
        //System.out.println("correct = " + Arrays.toString(correct));
        //System.out.println("answer = " + Arrays.toString(answer));
        //System.out.println("result = " + result);
      }
    }
    System.out.println("ゲームを終わります");
  }
  private static int[] createCorrect(int digits) {
    List<Integer> values = Arrays.asList(0,1,2,3,4,5,6,7,8,9);
    do { Collections.shuffle(values); } while(values.get(0) == 0);
    int[] correct = new int[digits];
    for(int i=0; i<digits; i++) correct[i] = values.get(i);
    return correct;
  }
  private static int[] inputAnswer(int digits, Scanner scanner) {
    while(true) {
      System.out.println(digits+"桁の数字を入力してください");
      String line = scanner.nextLine();
      if(line.length() == digits) {
        try {
          int[] answer = new int[digits];
          for(int i=0; i<digits; i++) {
            answer[i] = Integer.parseInt(line.substring(i, i+1));
          }
          return answer;
        } catch(NumberFormatException ignore) {
          //retry
        }
      }
    }
  }
  private static Judgment judge(int[] correct, int[] answer) {
    if(correct.length != answer.length) throw new IllegalArgumentException();
    int hit = 0, blow = 0;
    for(int i=0; i<answer.length; i++) {
      for(int j=0; j<correct.length; j++) {
        if(answer[i] == correct[j]) {
          if(i == j) hit++; else blow++;
          break;
        }
      }
    }
    return new Judgment(correct.length, hit, blow);
  }
}
class Judgment {
  private int digits, hit, blow;
  Judgment(int digits, int hit, int blow) {
    this.digits = digits;
    this.hit = hit;
    this.blow = blow;
  }
  boolean exactMatch() { return hit == digits; }
  String getImageString() {
    if(hit == 0 && blow == 0) return "－";
    StringBuilder sb = new StringBuilder();
    for(int i=0; i<hit; i++) sb.append("〇");
    for(int i=0; i<blow; i++) sb.append("△");
    return sb.toString();
  }
  public String toString() {
    return super.toString()+"[digits="+digits+", hit="+hit+", blow="+blow+"]";
  }
}