Hit&Blowのコードで、求めているのは正解まで何回繰り返えされかとそれと同じ数字が入力されたらメッセージが出るようにしたいです。

Question

Hit&Blowのコードです。
求めているのは正解まで何回繰り返したか
それと同じ数字が入力されたらメッセージが出るようにしたいです。
例　1123　"同じ数字が入力されています"
同じ数字が入力されたら繰り返しのカウンタに入らないようにしたいです。

import java.util.Random;
import java.util.Scanner;
public class HitBlow {

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

	
	//同じ数字が入力されたらメッセージが出る
	public static void repeatNumber() {
		for(i = 0; i <inputNumber.length; i++){
	        if(inputNumber[0]==inputNumber[1] || inputNumber[0]==inputNumber[2] || inputNumber[0]==inputNumber[3]){
	        	System.out.println("同じ数値が入力しています");
	        	break;
	        }else if(inputNumber[1]==inputNumber[0] || inputNumber[1]==inputNumber[2] || inputNumber[1]==inputNumber[3]){
	        	System.out.println("同じ数値が入力しています");
	        	break;
	        }else if(inputNumber[2]==inputNumber[0] || inputNumber[2]==inputNumber[1] || inputNumber[2]==inputNumber[3]) {
	        	System.out.println("同じ数値が入力しています");
	        	break;
	        }else if(inputNumber[3]==inputNumber[0] || inputNumber[3]==inputNumber[2] || inputNumber[3]==inputNumber[1]) {
	        	System.out.println("同じ数値が入力しています");
	        	break;
	        }else {
	        	continue;
	        }

	      }
	}

Comments

[swordone]

1. コードはマークダウンで書いてください。
　質問編集画面でコード部分を選択し、<code>というボタンを押し、「ここに言語を入力」の部分を「java」に書き換えてください。
2. 現状で何が問題なのかを明記してください。それが無いと丸投げの質問と同じになります。
　こうなってほしいのにこうなるとか、こういうエラーが出るとか、そういった情報を入れてください。

[m.ts10806]

>Hit&Blowのコードです。
タイトルには要件を記載してください。できれば「起きている問題」「つまずいていることを具体的に」書いてあること、ですね。
同じアプリケーションを作っていて次に質問するときもこのタイトルで質問して「タイトルだけで問題の分別がつけられるか」も考えてみてください。

Answer 1

あなたがこのHit&Blowのジャッジマンだとして、どのように判断してどのように処理をするのか、考えてみてください。

質問された点に限れば、

• ループカウントはゲームの進行と関係なく、ゲームが終わった後に数えようとしているので意味を成していません。
• 重複チェックはヒット、ブローの判定の後になっている。チェックも重複があったらメッセージを出すだけで、そのときのゲームでの処理に影響を及ぼせない。

Comments

[O11n11a11g1a]

ヒントを教えていただきありがとうございます。
プログラムを完成出来ました。