HitBlowのコードについてわからないです。

Question

import java.util.Random;
import java.util.Scanner;
public class hitblow1 {
	static Scanner sc=new Scanner(System.in);
    static char[] secretNumber=new char[4];
    static int i;
    static int hitCount=0;
    static int blowCount=0;
    static char [] inputNumber=new char[4];
    static int loopCount=0;
    static int substitution=0;
    public static void main(String[] args) {

        System.out.println("シークレットナンバーをせっとしました");
        secretArraySet();
        debugSecret();
        do {
            inputArraySet();
            hitCountJob();
            blowCountJob();
            hitblowMessage();

            repeatNumber();

        }while(hitCount!=4);
        loopCount();

        System.out.println("4Hit");
    }
    //何回繰り返される
    public static void loopCount() {
        loopCount=0;
          while(true) {
              if(hitCount!=4) {
                  loopCount++;
                }else {
                    break;
                }
          }
          System.out.println("正解まで"+loopCount+"回かかりました");

    }

    //ここでsecretNumberを表現している
    public static void debugSecret() {

        for(i=0;i<secretNumber.length;i++) {
            System.out.print(secretNumber[i]);
        }
        System.out.print("\n");
    }
    public static void secretArraySet() {
        Random rnd=new Random();
        char [] arrayNumber= {'1','2','3','4','5','6','7','8','9'};
        int location01,location02;
        char tmp;//temporaly 一時的ファイル

        //無作為に50回arrayNumberの値を入れ替えた
        for(i=0;i<50;i++) {
            location01 = rnd.nextInt(arrayNumber.length);
            location02 = rnd.nextInt(arrayNumber.length);
            tmp = arrayNumber[location01];
            arrayNumber[location01]=arrayNumber[location02];
            arrayNumber[location02] = tmp;
        }
        //arrayNumberの前4つをsecretNumberに代入
        for(i=0;i<secretNumber.length;i++) {
            secretNumber[i]=arrayNumber[i];
            }
        }
    //ここでinputををする
    public static void inputArraySet() {

            System.out.println("4桁の異なる数値を入力してください");
            String inputStr=sc.next();
            inputNumber=inputStr.toCharArray();
            debugInput();
    }
    //ここでinputNumberを表現している
    public static void debugInput() {

        for(i=0;i<inputNumber.length;i++) {
            System.out.print(inputNumber[i]);
        }
        System.out.print("\n");
    }
    //hit
    public static void hitCountJob() {
        hitCount=0;
        for(i=0;i<secretNumber.length;i++) {
            if(secretNumber[i]==inputNumber[i]) {
                hitCount++;
            }
        }
    }
    //hit&blowの表示
    public static void hitblowMessage() {

        System.out.print("Hit:"+hitCount+"     blow:"+blowCount+"\n");
    }
    //blow
    public static void blowCountJob() {
        blowCount=0;

        for(int j=0;j<secretNumber.length;j++) {
            for(i =0;i<inputNumber.length;i++) {
                if(secretNumber[j]==inputNumber[i] && i!=j) {
                    blowCount++;
                }
            }
        }
    }
   

  

          }
    }
}

正解までにかかった回数と同じ数値が入力された場合の処理がしたいのですが、このコードではできないのでわかる方教えてください。
実行すると正解までにかかった回数が０回になるのでそこを回数が出るようにするにはどうすればいいですか？

Comments

[y_waiwai]

提示のコードではどういうふうな動作になるんでしょうか

[Aratamicky]

実行すると正解までにかかった回数が０回となります。

[y_waiwai]

それを質問に追記してください

[Aratamicky]

loopcountが動いてないので０回にはなるんだと思うんですけどどうすればいいですか？

[y_waiwai]

それを質問に書きましょう。質問は編集できます。
わからないので教えろ、といわれても、なにがわからないのかわからんので教えようがないですが、
結果がこうなってしまうのはなぜか、そうならないようにするためにはどうすればいいのか、と具体的なことを書くと回答しやすいですよ

[Aratamicky]

わかりました。

[m.ts10806]

質問テンプレート使って書き直してもらったほうが回答を得やすくなると思います。
https://teratail.com/help/question-tips#questionTips3-8

Answer 1

ほぼ同じコードを別の質問で見たのですが、何かの課題でしょうか？

ならば私の指摘もほぼ同じです。

Answer 2

うーん?
Javaで書く必要、ある?

すべてstaticで、すべてC言語でいう関数みたいな使い方しかしていない。

意味ない気がするが...
まあいいけど。

loopCountメソッドの方(変数も同じ名前のやつがあるから紛らわしい...)は、なんでwhileの外?

それじゃあ、一回しか動かないでしょ。

今回のは文を追っていったらわかるよ。
不自然な部分がいくつもある。

特に loopCountメソッド内のwhile文。
不自然すぎ。

何がしたいのか意味不明。

処理する(ループありで) -> ループから抜ける -> loopCountメソッドが呼び出される
-> (loopCountメソッド内) whileで無限ループ -> (無限ループ内で)hitCountフィールドが4でなければ...それ以外なら... -> 最初は 0 なので "それ以外"が選択される -> break -> printlnメソッドで出力...

意味不明。
ロジックは書いて考えたほうがいいよ?
普通に考えて、

High or low を入力させる -> 正しいかどうか調べる -> 間違っているならカウンタを上げて、再度入力させる...

みたいな感じじゃないの?

それともそういう仕様で書いているの?

なんか、でたらめに書いている感じが強すぎる...

また、フィールドはグローバル変数として使うものではないです。

C言語ですら非推奨です。

C言語の説明では 参考1, Javaでの説明は参考2です。

ほかにもありますが。