javaでじゃんけんGame3回勝負あっちむいてホイを作っているのですがあっちむいてほいの負けたときのあなたの負けが表示されない

Question

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

public class test32 {

	public static void main(String[] args) {
		int win = 0;
		int game = 3;

		for(int i = 0; i < game; i++) {
			win += sub ();
		}

		System.out.println(game + "回勝負中、あなたは" + win + "回勝ちました。");
	}

	static Scanner stdIn = new Scanner(System.in);
	static Random rand = new Random();

	private static int sub() {
		String[] com1 = new String[] {"グー", "チョキ",  "パー"};
		int hantei = 0;

		while(true) {

			while(true) {
				System.out.println("じゃんけん勝負\n3回勝負です\n\n初めにじゃんけん！です\n数字で入力してね");
				System.out.println("0:グー\n1:チョキ\n2:パー\n");

				System.out.print("最初はグー、じゃんけん : ");

				try {
					String str1 = stdIn.next();
					int pry1 = Integer.parseInt(str1);
					int cpu1 = rand.nextInt(3);

					switch (pry1) {
					case 0:
						if (cpu1 == 0) {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "グーで… " + "あいこだよ！\n");
							hantei = 0;
						} else if (cpu1 == 1) {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "グーで… " + "あなたの勝ち\n");
							hantei = 1;
						} else {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "グーで… " + "あなたの負け\n");
							hantei = 2;
						}
						break;
					case 1:
						if (cpu1 == 0) {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "チョキで… " + "あなたの負け\n");
							hantei = 2;
						} else if (cpu1 == 1) {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "チョキで… " + "あいこだよ！\n");
							hantei = 0;
						} else {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "チョキで… " + "あなたの勝ち\n");
							hantei = 1;
						}
						break;
					case 2:
						if (cpu1 == 0) {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "パーで… " + "あなたの勝ち\n");
							hantei = 1;
						} else if (cpu1 == 1) {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "パーで… " + "あなたの負け\n");
							hantei = 2;
						} else {
							System.out.println(com1[cpu1] + "(COM)と" + "(Player)" + "パーで… " + "あいこだよ！\n");
							hantei = 0;
						}
						break;
					default:
						System.out.println("範囲外の数字です。0～2で入力してね。");
						break;
					}

					if(hantei != 0) {
						break;
					}
				}catch(NumberFormatException e) {
					System.out.println("数字で入力してね。");
				}
			}

			/*|----- ここまでが、じゃんけん！の領域 -----|*/

			/*|----- ここから、あっちむいてホイ！の領域 -----|*/
			String[] com2 = new String[] {"上", "下",  "左",  "右"};

			while(true) {
				System.out.println("次はあっちむいてホイ！です\n数字で入力してね");
				System.out.println("0:上\n1:下\n2:左\n3:右\n");

				System.out.print("始めるよ！ あっちむいてホイ : ");
				try {
					String str2 = stdIn.next();
					int pry2 = Integer.parseInt(str2);
					int cpu2 = rand.nextInt(4);

					switch (pry2) {
					case 0:
						if (cpu2 == 0) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "上だね\n");
							if(hantei == 1) {
								System.out.println("あなたの勝ち！");
								return 1;
							}else if(hantei == 2) {
								System.out.println("あなたの負け！");
								return 0;
							}
						} else if (cpu2 == 1) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "上だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 2) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "上だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 3) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "上だね\nもう一度じゃんけんからだよ！");
						}
						break;
					case 1:
						if (cpu2 == 0) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "下だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 1) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "下だね\n");
							if(hantei == 1) {
								System.out.println("あなたの勝ち！");
								return 1;
							}else if(hantei == 2) {
								System.out.println("あなたの負け！");
								return 0;
							}
						} else if (cpu2 == 2) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "下だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 3) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "下だね\nもう一度じゃんけんからだよ！");
						}
						break;
					case 2:
						if (cpu2 == 0) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "左だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 1) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "左だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 2) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "左だね\n");
							if(hantei == 1) {
								System.out.println("あなたの勝ち！");
								return 1;
							}else if(hantei == 2) {
								System.out.println("あなたの負け！");
								return 0;
							}
						} else if (cpu2 == 3) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "左だね\nもう一度じゃんけんからだよ！");
						}
						break;
					case 3:
						if (cpu2 == 0) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "右だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 1) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "右だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 2) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "右だね\nもう一度じゃんけんからだよ！");
						} else if (cpu2 == 3) {
							System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + "右だね\n");
							if(hantei == 1) {
								System.out.println("あなたの勝ち！");
								return 1;
							}else if(hantei == 2) {
								System.out.println("あなたの負け！");
								return 0;
							}
						}
						break;
					default:
						System.out.println("範囲外の数字です。0～3で入力してね。");
						break;
					}
				}catch(InputMismatchException e) {
					System.out.println("エラーです！０～３の数字で入力してね");
				}catch(NumberFormatException e) {
					System.out.println("エラーです！０～３の数字を入力してね");
				}
				return 0;
			}
		}
	}
}
コード

あっちむいてほいのあなたの負けを表示したいです。

Comments

[yureighost]

試してみましたけど、ちゃんと表示されますよ。

[ta.tatu]

本当ですか？！Σ（￣□￣;）
すいません確認不足かな⁉️

[ta.tatu]

今確認しましたがじゃんけんの時のあなたの負けは表示されるのですが、あっちむいてほいの時のあなたの負けが表示されませんでした。

[jimbe]

あっちむいてほいは, pry と cpu が同じ値の時に, じゃんけんの結果を「勝ち負け」と表示して, pry と cpu が違う値の時には(勝ち負けは無く)「もう一度じゃんけん」と表示していますが, これはどういうことでしょう.

[jimbe]

すいません, あっちむいてほいの流れを忘れていました.

Answer 1

まず, 長すぎです.
ご質問の件は, 条件を並べ過ぎてどの場合にどれが表示されるのかが分からなくなっているためと思います.
例えば switch(pry2) { ～ } の部分は

  if(pry2 < 0 || com2.length < pry2) {
    System.out.println("範囲外の数字です。0～3で入力してね。");
  } else {
    System.out.println(com2[cpu2] + "(COM)と" + "(Player)" + com2[pry2] + "だね");
    if (pry2 == cpu2) {
      if(hantei == 1) {
        System.out.println("あなたの勝ち！");
        return 1;
      }else if(hantei == 2) {
        System.out.println("あなたの負け！");
        return 0;
      }
    } else {
      System.out.println("\nもう一度じゃんけんからだよ！");
      break; //これが必要では?
    }
  }

と纏まります.
そして, おそらく「もう一度じゃんけんから」となった場合は(3回勝負のカウントを上げずに再度じゃんけんをする為に) あっちむいてほいの while ループを抜けるため break を書かなくてはいけないと思うのですが, case 内では書けませんので, このように纏めたほうがよいのではと思います.
また, ご提示のコードの最後にある return 0 は削除しないと, 数字以外や範囲外の数値を入力等の場合に3回勝負のカウントが上がってしまいます.

モロモロ想像・修正すると, 以下の様になりました.
(遊びで日本語メソッド名を使いました. 問題ありましたらテキトウな英文字にしてください.)

import java.util.*;

class Player {
  private Scanner scanner = new Scanner(System.in);

  int select(String[] choices, String prompt) {
    while(true) {
      for(int i=0; i<choices.length; i++) System.out.printf("%d:%s\n", i, choices[i]);
      System.out.print(prompt);
      try {
        String str = scanner.next();
        int pry = Integer.parseInt(str);
        if(0 <= pry && pry < choices.length) return pry;

        System.out.println("範囲外の数字です。0～" + choices.length + "で入力してね。");
      }catch(NumberFormatException e) {
        System.out.println("エラーです！0～" + choices.length + "の数字を入力してね");
      }
    }
  }
}

class Computer {
  private Random rand = new Random();

  int select(String[] choices) {
    return rand.nextInt(choices.length);
  }
}

enum Settlement {
  NOT_SETTLED, WON, LOST
}

public class test32 {
  public static void main(String[] args) {
    Player player = new Player();
    Computer computer = new Computer();
    int win = 0;
    int game = 3;

    System.out.println("じゃんけん勝負\n" + game + "回勝負です");

    for(int i=1; i<=game; i++) {
      System.out.println("\n---- " + i + "回戦 ----");
      Settlement settlement;
      do {
        settlement = あっちむいてホイ(player, computer, じゃんけん(player, computer));
      } while(settlement == Settlement.NOT_SETTLED);
      if(settlement == Settlement.WON) win++;
    }

    System.out.println("\n---- 終了 ----");
    System.out.println(game + "回勝負中、あなたは" + win + "回勝ちました。");
  }

  private static boolean じゃんけん(Player player, Computer computer) {
    String[] handStr = new String[] {"グー", "チョキ",  "パー"};
    int[] hantei = new int[] { 1, 2, 0 }; //ドレに勝つか

    System.out.println("\n初めにじゃんけん！です\n数字で入力してね");

    while(true) {
      int pry = player.select(handStr, "最初はグー、じゃんけん : ");
      int cpu = computer.select(handStr);
      System.out.print(handStr[cpu] + "(COM)と" + "(Player)" + handStr[pry] + "で… ");

      if (pry != cpu) return printResult(hantei[pry] == cpu);

      System.out.println("あいこだよ！\n");
    }
  }

  private static Settlement あっちむいてホイ(Player player, Computer computer, boolean playerAttack) {
    String[] dirStr = new String[] {"上", "下",  "左",  "右"};

    System.out.println("\n次はあっちむいてホイ！です\nあなたは"+(playerAttack?"攻撃側":"防御側")+"です\n数字で入力してね");

    int pry = player.select(dirStr, "始めるよ！ あっちむいてホイ : ");
    int cpu = computer.select(dirStr);
    System.out.println(dirStr[cpu] + "(COM)と" + "(Player)" + dirStr[pry] +"だね");

    if(pry != cpu) {
      System.out.println("もう一度じゃんけんからだよ！");
      return Settlement.NOT_SETTLED;
    }
    return printResult(playerAttack) ? Settlement.WON : Settlement.LOST;
  }

  private static boolean printResult(boolean result) {
    System.out.println("あなたの" + (result?"勝ち":"負け") + "！");
    return result;
  }
}

Answer 2

コンソールのスクショで申し訳ありませんが、
この動作の想定ではないんですかね。

Comments

[ta.tatu]

有り難うございます。出てますね！
何故だろう？
ベストアンサーにさせて頂きます。