じゃんけんゲーム あいこの繰り返し

Question

前提・実現したいこと

　はじめまして。java初心者のものです。初歩的な質問かもしれないのですがお答えいただけたら幸いです。
javaを用いてじゃんけんゲームを作成しているのですが、あいこの際にじゃんけんの入力を繰り返しを行えるよう改良したいと考えています。もしよければどなたかご教授いただけませんか。ほかに必要な記載があれば教えてくださるとありがたいです。よろしくお願いします。

該当のソースコード

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

public class JankenIf {

	public static void main(String[] args) {
		while(true) {
		// ユーザの手をキーボードから入力してもらう
		int user = getUser();
 
		// コンピュータの手を乱数で作成
		int cpu = getCpu();
 
		// 勝敗の判定
		String result = judgeJanken(user,cpu);
 
		// 結果を表示
		showResult(user,cpu,result);
	}
 
	public static int getUser() {
		// キーボード入力の準備
		Scanner stdin = new Scanner(System.in);
 
		// 無限ループ
		while(true) {
			// メッセージの表示
			System.out.println("<じゃんけんゲーム>");
			System.out.println();
			System.out.println("あなたのじゃんけんの手を矢印の先に入力して下さい！");
			System.out.print("(グーの場合：0，チョキの場合：1、パーの場合：2)　--> ");
 
			// 入力されたデータが整数かどうかチェック
			if(stdin.hasNextInt()) {
				// 入力されたデータを整数として読み込む
				int number = stdin.nextInt();
				// 整数でも有効なのは「0，1，2」のみ
				if(number<=-1 || number >=3) {
					// 範囲外は無効なデータなのでやり直し
					System.out.println("【エラー】入力できるのは「0～2」です");
					continue;
				}else {
					// 0,1,2の場合、メソッドの結果として返す
					return number;
				}
			}else {
				// 整数以外の場合、無効なデータなのでやり直し
				System.out.println("【エラー】入力できるのは整数だけです");
				// 不要なトークンをバッファから取り除く
				stdin.next();
			}
		}
	}
 
	public static int getCpu() {
		// 乱数を準備
		Random rand = new Random();
		// 0,1,2のどれかを求め、メソッドの結果として返す
		return rand.nextInt(3);
	}
 
	public static String judgeJanken(int user,int cpu) {
		String result=""; // 判定結果を保存
 
		if((user==0 && cpu==1) || (user==1 && cpu==2) || (user==2 && cpu==0)) {
			result="あなたの勝ち！";
		}else if( (user==0 && cpu==2) || (user==1 && cpu==0) || (user==2 && cpu==1)) {
			result="あなたのまけ！";
		}else {
			result="あいこです！";
		}
 
		// 勝敗結果を返す
		return result;
	}
 
	public static void showResult(int user,int cpu,String result) {
		// じゃんけんの手を配列で定義
		String[] janken= {"グー","チョキ","パー"};
		// 結果の表示
		System.out.println("あなたの手:"+janken[user]+",コンピュータの手:"+janken[cpu]);
		System.out.println("結果は..."+result);
			if((user==0 && cpu==1)||(user==1 && cpu==2)||(user==2 && cpu==0)||(user==0 && cpu==2)||(user==1 && cpu==0)||(user==2 && cpu==1)) {
				break;
			}
	      }
	   }
	}
}

試したこと

while文を用いて試してみたのですが知識不足でうまく作用されませんでした。
while文ですべてくくり、じゃんけんの勝ちと負けの際にbreakで抜けられるようにするようコードを書いたつもりです。
現状
・Exception in thread "main" java.lang.Error: Unresolved compilation problem:
at JankenIf.main(JankenIf.java:6)
という表示が出るため実行に移せない。
・実行できないため、あいこの際に繰り返しが適切にされるかわからない。
理想
・じゃんけんの勝敗がつくまで繰り返しじゃんけんを行うコードを作る。

Comments

[jimbe]

試して「うまく」動作しないのを無くされては、丸投げ状態になります。
知識不足を自覚されているのであれば、何の知識が不足しているのかが分かる状態のコードをご提示ください。

また、「うまく」という表現は作者の想定動作に対しての表現であって、その想定動作を知らない側にとっては何のことかわかりません。
どう表示されてどう入力したらどうなるはずがどうなったのか、その原因はどこまでどのように調査したのかを **具体的に**「試したこと」にお書きください。

[oimo0505]

ご指摘ありがとうございます。
jimbe様の指摘をいただくまで客観的に考えることが出来ていませんでした。
今後からも参考にさせていただきます。
質問の方編集させていただきましたが、まだ至らぬ点があればお手数ですがまたご指摘いただけると幸いです。

[jimbe]

編集ありがとうございます。
追加されたコードを見ますと、java 初心者と言われるのは分かりますが、追加された部分以外とのコードの質の違いが感じられます。
つまり、まっさらから(追加まえの)コードを書かれた方であればこのように追加はしないと思うのですが、どこかの情報を元に（切り貼り等で）作成されているのでしょうか。

[oimo0505]

コメントありがとうございます。
現在課題作成の途中でインターネットを参考にしながら取り組んでよいとのことだったため、おっしゃる通りおおもとの部分に関してはインターネットのサイトを参考にさせていただいております。（https://ict-skillup.com/java/1107/）
参考したうえで自分なりにプログラムを変更していきたいと思っていたのですが、途中でつまづいてしまい現在質問させていただいております。

[m.ts10806]

mainメソッドのwhileが閉じられてないように見えますが、そこがエラーの原因では

[oimo0505]

コメントありがとうございます。
showResult(user,cpu,result);　の後で閉じるようにしてみたのですが同じようなエラーが出てしまいます。

[dodox86]

> 閉じるようにしてみた

いや、ああでもない、こうでもないとトライアル＆エラーでコードをこねくり回すのではなく、while()の基本の使い方と考え方を理解してから自分のコードに適用しましょうよ、、、

[oimo0505]

ご指摘ありがとうございます。
試行錯誤を行うのではなく、もう少し基本に立ち返ってみようと思います。

[dodox86]

while()の基本、とコメントしましたが、そうですらなく「どの行からどの行まで、どんな条件while(条件)で繰り返し、あるいは繰り返しを抜ける(break)か、すなわちそれが結果として{}の範囲」を明確に意識するのが大事なのです。知識不足とは関係ないのです。

[m.ts10806]

「ここからここまでは想定通り動く」という小さい塊を幾つも作って動作確認できてから組み合わせるのを強くすすめます。
いきなり流れがあるコードを作るのは結構難易度高いです。
「判定結果を返す」
「自分の手を決定」「相手の手を決定」
「〇〇するまで繰り返す」
など、それぞれ単体で確認できるもののはずなので。

[oimo0505]

ご丁寧・親切な解説ありがとうございます。
インデントについてもう一度丁寧に確認した後whileを追加したいと思います。

[oimo0505]

m.ts10806様
ご教授ありがとうございます。
少しずつ実行を行いエラーが出た場面で再確認するよう気を付けます。
このことは今後プログラミングを行う際にも重要になってくると思います。
貴重なご意見ありがとうございました。

Answer 1

修正しようと追加された while と if の組み合わせは、個々には間違っておりません。
ただ、その追加場所が問題となっています。

java では(まぁ他の言語でも大体そうだと思いますが) メソッドや for/while、 if 等で使われる "{" "}" の組み合わせは、その外側の "{" "}" に完全に含まれていなければなりません。

例えば、public class JankenIf { で始まるクラスの中の各メソッド(main や getUser から showResult まで)は全てクラスを囲む最後の } までの中にありますし、getUser の「無限ループ」の while(true) { も getUser 内に対応する } があります。

一方追加されたコードは、 main メソッド内から始まり、getUser 等を飛び越えて showResult までも含もうとされており、そのようなことは java では出来ません。

main メソッド内で while ループするのであれば、その終わりである "}" やその内部に書こうとする if 文やその "{" "}" までも全て、それぞれがそれぞれの外の内に含むようにしなければなりません。

若干蛇足ですが、コードを御覧頂けば分かります通り、"{" "}" の組み合わせといってもどの "{" がどの "}" に対応するのかは、幾つも "}" が並んでいると分かり難いです。
そのため、"{" が出たらそれ以降の文は少し右(2文字や4文字、もしくは1タブ等)にずらして書き始め、 "}" が出る時に戻すという書き方のルールがあり、"インデント" と呼ばれます。
追加前のコードはキチンと "{" "}" の対応が取れるように文字の位置が合っています。
開発ツールの中にはこのインデントを自動で正しく付け(直し)てくれる機能があり、それを用いることで "{" "}" の対応が確認できたりしますので、もしありましたら活用されては如何でしょうか。

Comments

[jimbe]

蛇足の補足(!)ですが、インデントは無くてもプログラム自体には問題ありません。あくまで人がコードを読み易くする為に作られたいわゆる"紳士協定"です。
ですので、インデントをどう扱うか、何文字をどのような場合に付けるか等は人(もしくは各種団体?)それぞれであり、深堀すると宗教戦争となりますので、取り扱いには注意してください(苦笑)

[oimo0505]

とても詳しい解説ありがとうございます。
参考にさせていただきます。