java じゃんけんゲーム 出力されない

Question

前提

javaでじゃんけんゲームを作っています。コンピューターの出力と比較して勝者を決定します。

実現したいこと

・入力する側が勝つまでじゃんけんを続け、入力者が勝ったらじゃんけんを終了する。

発生している問題・エラーメッセージ

じゃんけんの結果が出力されない

該当のソースコード

import java.io.*;
class Sample{
	public static void main(String[] args)throws IOException{
		
			BufferedReader br = new BufferedReader(new InputStreamReader(System.in));	
			
			for(int i = 0; i >= 0; i++){
			
			System.out.print("パー(P),グー(G),チョキ(T)>");
			String myHand = br.readLine();
			
			int compHand = (int)(Math.random()*3)+1;
			System.out.print("コンピュータ:");
			
			if(compHand == 1){
				System.out.println("P");
			}else if(compHand == 2){
				System.out.println("G");
			}else if(compHand == 3){
				System.out.println("T");
			}
			
			if((myHand == "G" && compHand == 3) || (myHand == "T" && compHand == 1) || (myHand == "P" && compHand == 2)){
				System.out.println("あなたの勝ち");
				break;
			}else if((myHand == "G" && compHand == 2) || (myHand == "T" && compHand == 3) || (myHand == "P" && compHand == 1)){
				System.out.println("あいこ");
				continue;
			}else if((myHand == "G" && compHand == 1) || (myHand == "T" && compHand == 2) || (myHand == "P" && compHand == 3)){
				System.out.println("コンピュータの勝ち");
				continue;
			}
		}		
	}
}

試したこと

break;を使うことで入力者が勝った時にじゃんけんゲームが終了すると考えた。すると勝敗が出力されないままゲームが続いた。なぜかわからないのでご教授いただきたい。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

文字の比較なら == が使えます。
例えば myHand == "G" を myHand.charAt(0) == 'G' に書き換えて下さい。

for(int i = 0; i >= 0; i++){
変なfor文ですね。i は　0 から始めてどんどん増やすのですから、正であるのは当然です。
無限ループなら while (true) { または for (;;) { でいいでしょう。

myHand も compHand も 0、1、2 にすると簡単になります。

import java.util.Scanner;

class Sample{
	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		while (true) {
			System.out.print("パー(P),グー(G),チョキ(T)>");
			int myHand = "PGT".indexOf(sc.next());
			int compHand = (int)(Math.random()*3);
			System.out.println("コンピュータ:" + "PGT".charAt(compHand));
			int d = (compHand - myHand + 3) % 3;
			if (d == 1) {
				System.out.println("あなたの勝ち");
				break;
			}
			if (d == 0)
				System.out.println("あいこ");
			else
				System.out.println("コンピュータの勝ち");
		}		
	}
}

Answer 2

文字列の比較には == ではなく .equals を使ってください。
具体的には、例えば myHand == "G" を myHand.equals("G") に書き換えて下さい。

おまけ

たぶん私ならこう書きます。
勝ち負けの判定とか、文字や数値からの変換とか、逐一考えるのはめんどくさそうなので...

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.io.IOException;
import java.util.Random;

enum JankenHand {
	P("P"), G("G"), T("T");
	
	private String name;
	private JankenHand(String name) {
		this.name = name;
	}

	public boolean win(JankenHand other) {
		if(this == P && other == G) return true;
		if(this == G && other == T) return true;
		if(this == T && other == P) return true;
		return false;
	}

	@Override
	public String toString() {
		return name;
	}

	public static JankenHand of(String name) {
		for(JankenHand hand: JankenHand.values()) {
			if(hand.name.equals(name)) return hand;
		}
		throw new IllegalArgumentException(
			String.format("不正な引数： %s", name));
	}

	public static JankenHand getRandomHand(Random gen) {
		return JankenHand.values()[gen.nextInt(JankenHand.values().length)];
	}
}

class Sample {
	public static void main(String[] args) throws IOException {
		Random randGen = new Random();

		try(BufferedReader br = new BufferedReader(new InputStreamReader(System.in))) {
			while(true) {
				// 自分の手を入力
				System.out.print("パー(P),グー(G),チョキ(T)>");
				final JankenHand myHand;
				try {
					myHand = JankenHand.of(br.readLine());
				}
				catch(IllegalArgumentException e) {
					System.err.println(e.getMessage());
					continue;
				}
				
				// コンピュータの手を取得
				final JankenHand compHand = JankenHand.getRandomHand(randGen);
				System.out.println("コンピュータ:" + compHand);
				
				// 勝ち負けを判定
				if(myHand.win(compHand)) {
					System.out.println("あなたの勝ち");
					break;
				}
				if(compHand.win(myHand)) {
					System.out.println("コンピュータの勝ち");
					continue;
				}

				System.out.println("あいこ");
			}		
		}		
	}
}

getRandomHandの引数をStrategyにしてやると癖あるコンピューターが作れます。
また、日本の3すくみのジャンケン以外も比較的簡単に作れます。

いろいろ改造してみると楽しいですよ。

Comments

[73T-beginner]

初歩的なミスでした。気をつけます。ありがとうございました。

[jimbe]

JankenHand.of メソッドは enum.valueOf で良いように思います。

[LouiS0616]

@jimbe さん
確かに今回は識別子と名前が完全一致しているのでそれでもいけますね。