Java HitandBlow

Question

import java.io.Console;
import java.util.Scanner;
public class sampul1{
    public static void main(String[] args) {
    	
        //ゲーム説明
        System.out.print("このゲームは３桁の数字を当てるゲームです");
        System.out.println("正解するまでの回数が少なかった方が勝利です");
        System.out.println("数字と桁が両方合っていればHit,数字のみならばBlowと表示されます");
        
        //数字設定
        Scanner sc=new Scanner(System.in);
        Console cons=System.console();
        if(cons==null) {
            System.out.println("cons==null");
            return;
        }
        String name1="player1";
        System.out.println(name1+"さん3桁の数字を設定してください");
        while(true) {
        boolean number=true;
       char[] answer1=cons.readPassword();
       String input=new String(answer1);
        char []a=input.toCharArray();
        int il=input.length();
        for(int i=0; i<3; i++) {
            for(int j=i+1; j<3; j++) {
                if(a[i]==a[j]||il!=3) {
                    number=false;
                }
            }
        }
        if(number==false) {
            System.out.println("数字は重複せず、３桁です");
            continue;
        }
        String name2="player2";
         System.out.println(name2+"さん3桁の数字を設定してください");
        while(true) {
        	boolean num=true;
         char[] answer2=cons.readPassword();
         String reinput=new String(answer2);
         char[]b=reinput.toCharArray();
         int rl=reinput.length();
         for(int i=0; i<3; i++) {
             for(int j=i+1; j<3; j++) {
                 if(b[i]==b[j]||rl!=3) {
                     num=false;
                 }
             }
         }
         if(num==false) {
             System.out.println("数字は重複せず、３桁です");
             continue;
         }
        //player1
        System.out.println(name1+"さん相手の３桁を予想してください");
         int count=1;
        while(true) {
        	boolean nu=true;
         String c=sc.nextLine();
         char[] d=c.toCharArray();
         int cl=c.length();
         for(int i=0; i<3; i++) {
             for(int j=i+1; j<3; j++) {
                 if(d[i]==d[j]||cl!=3) {
                     nu=false;
                 }
             }
         }
         if(nu==false) {
             System.out.println("数字は重複せず、３桁です");
             continue;
         }
         
           int hit=0, blow=0;
           for(int i=0; i<3; i++) {
              if(d[i]==b[i]) {
                  hit++;
              }else{
                for(int j=0; j<3; j++) {
                    if(d[i]==b[j]) {
                    blow++;
                    }
                }
            }
           }
          if(hit!=3) {
                 System.out.println(hit+"Hit "+blow+"Blow");
                 count++;
             }
             if(hit==3) {
                 System.out.println(name1+"さんは"+count+"回目で正解しました。");
                 break;
        }
        }
         
       
        
        //player2
        System.out.println(name2+"さん相手の３桁の数字を予想してください");
         int recount=1;
        while(true) {
        	boolean n=true;
        String e=sc.nextLine();
        char[] f=e.toCharArray();
        int el=e.length();
        for(int i=0; i<3; i++) {
             for(int j=i+1; j<3; j++) {
                 if(f[i]==f[j]||el!=3) {
                     n=false;
                 }
             }
         }
         if(n==false) {
             System.out.println("数字は重複せず、３桁です");
             continue;
         }
           int ahit=0, ablow=0;
           for(int i=0; i<3; i++) {
              if(f[i]==a[i]) {
                  ahit++;
              }else{
                for(int j=0; j<3; j++) {
                    if(f[i]==a[j]) {
                    ablow++;
                    }
                }
            }
           }
          if(ahit!=3) {
                 System.out.println(ahit+"Hit "+ablow+"Blow");
                 recount++;
             }
             if(ahit==3) {
                 System.out.println(name2+"さんは"+recount+"回目で正解しました。");
                 if(count==recount) {
                     System.out.println("引き分けです");
                 }
                 if(count<recount) {
                     System.out.println(name1+"さんの勝利です！");
                 }
                 if(count>recount) {
                     System.out.println(name2+"さんの勝利です！");
                 }
                 break;
        }
        }
        
        
        
            
        
        
           
        
        }
        }

    }
}

       
         

java初心者です。一応完成形？に近い形まで出来ました。
①readPassword以外で自分で数字を設定できてかつ非表示にする方法はないのでしょうか？（readPasswordはターミナルでしか実行できなくなるので。）
②このコードは同じ構文を繰り返していますが、他にもっと短縮簡素化できるコードは何かありますか？

Answer 1

①readPassword以外で自分で数字を設定できてかつ非表示にする方法はないのでしょうか？（readPasswordはターミナルでしか実行できなくなるので。）

たしかにIDEですごくデバッグしずらいですね（何か方法はあるんでしょうが）
JPasswordFieldを使うとか？^^;

②このコードは同じ構文を繰り返していますが、他にもっと短縮簡素化できるコードは何かありますか？

まずwhileのネストが深く、非常に読みにくいです。

while (true) {
    // player1 設定

    while (true) {
        // player2 設定

        while (true) {
            // 相手(player2)の予想

            // 正解でbreak
            break;
        }

        while (true) {
            // 相手(player1)の予想

            // 正解でbreak 結果判定
            break;
        }
    }
}

現状このような構成になっています。

しかしこのようなネストは不要のはずですし、バグ（想定していないであろう挙動）があります。
結果判定後breakで抜けるのは一番内側のwhileですから、player2 設定から繰り返しています。

何度も遊べるようにplayer1 設定から繰り返すならわかりますが、おそらくそういうつもりではなかったと推測します。

while (true) {
    // player1 設定
    break;
}
while (true) {
    // player2 設定
    break;
}
while (true) {
    // 相手(player2)の予想

    // 正解でbreak
    break;
}
while (true) {
    // 相手(player1)の予想

    // 正解でbreak
    break;
}
//  結果判定

こういう構成で十分なはずです（変数のスコープの問題でこうできなかったのかな？）

明らかに重複している部分をどうメソッドに切り出すべきかですが、「こうなってくれたらいいのにな」というのをまず先に定義してそうなるように後からメソッドを作るといい気がします。

今どきのIDEはリファクタリング機能（メソッドの抽出等）もよくできているので、いろいろ試してもいいと思います。

とくにmainメソッドはだらだら書いてしまいがちなので、最初から全体の大きな流れをコメントで書いてそれを埋めるようにメソッドを作っていくと自然と分けられる気がします。

ループがネストしてややこしいところや大域脱出（多重ループを一気に抜ける）したくなった時（Javaはlabelでジャンプできますが）は、大概メソッドに分けるとすっきり書けます。

---

わたしだったらmainはこのくらいになったらうれしいなぁと考えます。
頭のメモリが少ないので１メソッド１画面（３０行くらい）じゃないと把握できないのです^^;

public static void main(String[] args) {
    p("このゲームは３桁の数字を当てるゲームです");
    p("正解するまでの回数が少なかった方が勝利です");
    p("数字と桁が両方合っていればHit,数字のみならばBlowと表示されます");


    String name1 = "player1";
    p(name1 + "さん3桁の数字を設定してください");
    char[] answer1 = getSecretNumber();

    String name2 = "player2";
    p(name2 + "さん3桁の数字を設定してください");
    char[] answer2 = getSecretNumber();


    p(name1 + "さん相手の３桁を予想してください");
    int count1 = challenge(answer2);
    p(name1 + "さんは" + count1 + "回目で正解しました。");

    p(name2 + "さん相手の３桁の数字を予想してください");
    int count2 = challenge(answer1);
    p(name2 + "さんは" + count2 + "回目で正解しました。");


    if (count1 < count2) {
        p(name1 + "さんの勝利です！");
    } else if (count1 > count2) {
        p(name2 + "さんの勝利です！");
    } else {
        p("引き分けです");
    }
}

「なるほどそうだな」と思ったら、そうなるようにメソッドを作ってみてください。
「うーんちょっと違うな」と思ったら、robinjakusonさんの理想の分け方を考えてみてください^^;

Comments

[退会済みユーザー]

ありがとうございます。
自分で見ても何か長くてダサいなと思ったいたので、もう少し見やすいコードに改善していきます。

[TN8001]

着実に学習が進んでおられるようで、すばらしいですね。

そのため余計な？（ぶれるような）ことをしないようにした結果、中途半端なコード提示になってしまいましたがもし「参考までに見てみたい」ということでしたらおっしゃってください（別にそんな素晴らしいコードというわけではありませんがｗ

あと書き忘れましたが、現状数字以外も入力できてしまいます（それはそれで面白そうですがおそらく難しすぎですね^^;

Answer 2

入力、データチェック、NGならどーこー、の部分を無限ループに入れといて、
データチェックの結果OKならそのループを抜ける、というふうにしましょう。

ついでに、そこんところを関数にしてしまえば、いろんなところで使いまわしできます

Answer 3

readPassword以外で自分で数字を設定できてかつ非表示にする方法

コンソール版であればそれが一番確実なのではないでしょうか。
コンソール以外に移植することを前提とするのであれば、入出力に関する部分をインターフェースとして定義してそのコンソール版実装クラスを作成しておくのが良いかと思います。

このコードは同じ構文を繰り返していますが、他にもっと短縮簡素化

まず、処理毎に分類し、それぞれをメソッドにしてください。その際、何をパラメータとして渡し何を戻り値として返すようにするとメソッドの独立性が確保されるかを考えられると良いかと思います。
メソッドの分け方が適切であれば、自然と再利用できるようになり、全体として小さくなるはずです。

---

入出力を IO インターフェースに纏める形にしてみました。
Player, Numbers もクラス化すると思ったより大き目になった感じです。(toString() とかありますしね…。)
副作用的に NAMES を増やせば 3 人以上でもプレイできると思います。(試してはいません。)

package teratail_java.q375509;

import java.io.Console;
import java.io.PrintWriter;
import java.util.*;

//ユーザーインターフェース
interface IO {
  void println(String s);
  char[] input();
  char[] inputPassword();
}

//ユーザーインターフェース(コンソール版)
class ConsoleIO implements IO {
  private Console cons;
  private PrintWriter out;
  ConsoleIO() {
    cons = System.console();
    if(cons == null) {
      throw new IllegalStateException();
    }
    out = cons.writer();
  }
  @Override
  public void println(String s) {
    out.println(s);
  }
  @Override
  public char[] input() {
    return cons.readLine().toCharArray();
  }
  @Override
  public char[] inputPassword() {
    return cons.readPassword();
  }
}

//ユーザーインターフェース(System版)
class SystemIO implements IO {
  private Scanner sc;
  SystemIO(){
    sc = new Scanner(System.in);
  }
  @Override
  public void println(String s) {
    System.out.println(s);
  }
  @Override
  public char[] input() {
    return sc.nextLine().toCharArray();
  }
  @Override
  public char[] inputPassword() {
    return sc.nextLine().toCharArray();
  }
}

//三桁の数字
class Numbers {
  static boolean checkNumbers(char[] s) {
    if(s.length != 3 || s[0] == s[1] || s[1] == s[2] || s[2] == s[0]) return false;
    for(int i=0; i<s.length; i++) if("0123456789".indexOf(s[i]) < 0) return false;
    return true;
  }

  static class HitAndBlow {
    final int hit, blow;

    HitAndBlow(int hit, int blow) {
      this.hit = hit;
      this.blow = blow;
    }

    @Override
    public String toString() {
      return hit + "Hit " + blow + "Blow";
    }
  }

  private final char[] numbers;
  Numbers(char[] numbers) {
    if(!checkNumbers(numbers)) throw new IllegalArgumentException();
    this.numbers = numbers;
  }

  HitAndBlow checkHitAndBlow(Numbers other) {
    //ロジックはオリジナル尊重
    int hit = 0, blow = 0;
    for(int i = 0; i < numbers.length; i++) {
      if(numbers[i] == other.numbers[i]) {
        hit++;
      } else {
        for(int j = 0; j < numbers.length; j++) {
          if(numbers[i] == other.numbers[j]) {
            blow++;
          }
        }
      }
    }
    return new HitAndBlow(hit, blow);
  }

  @Override
  public String toString() {
    return Arrays.toString(numbers);
  }
}

//プレイヤー
class Player {
  final String name;
  private final Numbers numbers;
  private int predictCount;
  Player(String name, Numbers numbers) {
    this.name = name;
    this.numbers = numbers;
    this.predictCount = 0;
  }

  void incrementPredictCount() { predictCount++; }
  int getPredictCount() { return predictCount; }

  Numbers.HitAndBlow checkHitAndBlow(Numbers other) {
    return numbers.checkHitAndBlow(other);
  }

  @Override
  public String toString() {
    StringJoiner sj = new StringJoiner(",", "Player[", "]");
    sj.add("name=" + name);
    sj.add("numbers=" + numbers);
    sj.add("predictCount=" + predictCount);
    return sj.toString();
  }
}

public class Sample1 {
  private static final String[] NAMES = new String[]{ "player1", "player2" };

  public static void main(String[] args) {
    IO io = new ConsoleIO();
    //IO io = new SystemIO(); //テスト用 (inputPassword の入力が見えてしまう)

    //ゲーム説明
    io.println("このゲームは３桁の数字を当てるゲームです");
    io.println("正解するまでの回数が少なかった方が勝利です");
    io.println("数字と桁が両方合っていればHit,数字のみならばBlowと表示されます");

    //プレイヤー
    Player[] players = new Player[NAMES.length];
    for(int i=0; i<NAMES.length; i++) {
      io.println(NAMES[i] + "さん3桁の数字を設定してください");
      Numbers numbers = inputNumbers(io, true);
      players[i] = new Player(NAMES[i], numbers);
    }

    //プレイ
    for(int i=0; i<players.length; i++) {
      Player target = players[(i+1)%players.length]; //予想する相手
      io.println(players[i].name + "さん, " + target.name + "さんの３桁を予想してください");
      while(true) {
        Numbers predict = inputNumbers(io, false);
        players[i].incrementPredictCount();
        Numbers.HitAndBlow hab = target.checkHitAndBlow(predict);
        if(hab.hit == 3) {
          io.println(players[i].name + "さんは" + players[i].getPredictCount() + "回目で正解しました。");
          break;
        }
        io.println(hab.toString());
      }
    }

    //結果
    List<Player> winnerList = judgement(players);
    if(winnerList.size() == players.length) { //全員勝利=同点
      io.println("引き分けです");
    } else {
      StringJoiner sj = new StringJoiner(",", "", "さんの勝利です！");
      for(Player p : winnerList) sj.add(p.name);
      io.println(sj.toString());
    }
  }

  //数字入力
  static Numbers inputNumbers(IO io, boolean hide) {
    char[] s;
    while(true) {
      s = hide ? io.inputPassword() : io.input();
      //System.out.println("s="+Arrays.toString(s));
      if(Numbers.checkNumbers(s)) break;
      io.println("数字は重複せず、３桁です");
    }
    return new Numbers(s);
  }

  //最終判定
  static List<Player> judgement(Player[] players) {
    List<Player> winnerList = new ArrayList<Player>(Arrays.asList(players));
    Collections.sort(winnerList, (o1,o2) -> o1.getPredictCount()-o2.getPredictCount());
    //for(Player p : winnerList) System.out.println(p);
    for(int i=winnerList.size()-1; winnerList.get(i).getPredictCount()>winnerList.get(0).getPredictCount(); i--) winnerList.remove(i);
    return winnerList;
  }
}

Comments

[退会済みユーザー]

ありがとうございます。
特に移植する予定もないのでこのままにしとこうと思います。

[退会済みユーザー]

わざわざありがとうございます。
同じ機能でも違った書き方があって参考になります。