はい いいえ の選択後の動作が正しく動かない

Question

前提・実現したいこと

好きな果物を選択した後に「はい」「いいえ」で再度選択するのですが
「はい」を選択すると問題なく動きますが、一度「いいえ」を選択したあとに
再び「はい」を選択すると、もう一度果物の選択が表示されてしまいます。

このような動作をしてしまう理由が分からないので教えて頂きたいです。

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

次の中から好きな果物を選んでください
1:【メロン】
2:【りんご】
3:【バナナ】
1
メロンですね
1:【はい】
2:【いいえ】
2
もう一度選択してください
1:【メロン】
2:【りんご】
3:【バナナ】
1
メロンですね
1:【はい】
2:【いいえ】
1
正しいキーを入力してください　//ここから
1:【メロン】　　　　　　　　　　　・
2:【りんご】　　　　　　　　　　　・
3:【バナナ】　　　　　　　　　//ここまでが動いてしまう理由がわかりません
1
メロンですね
1:【はい】
2:【いいえ】
1
あなたの好きな果物はメロンですね

該当のソースコード

public class test4
{
	public static void main(String[] args)
	{
		System.out.println("次の中から好きな果物を選んでください");
		
		String fruit = name();
		
		System.out.println("あなたの好きな果物は" + fruit + "ですね");
	}
	
	public static String name()
	{
		String m = "メロン";
		String r = "りんご";
		String b = "バナナ";
		
		System.out.println("1:【" + m + "】");
		System.out.println("2:【" + r + "】");
		System.out.println("3:【" + b + "】");
		int ans = new java.util.Scanner(System.in).nextInt();
		
		if(ans == 1){
			System.out.println(m + "ですね");
			int ans2 = Choose();			
				if(ans2 == 1){
				return m;
			}else{
				 ng();
			}
		}
		if(ans == 2){
			System.out.println(r + "ですね");
			int ans2 = Choose();			
				if(ans2 == 1){
				return r;
			}else{
				 ng();
			}
		}
		if(ans == 3){
			System.out.println(b + "ですね");
			int ans2 = Choose();			
				if(ans2 == 1){
				return b;
			}else{
				 ng();
			}
		}else{
			System.out.println("正しいキーを入力してください");
			return name();
		}
		return name();
	}
	
	public static int Choose(){
			System.out.println("1:【はい】");
			System.out.println("2:【いいえ】");
			int ans2 = new java.util.Scanner(System.in).nextInt();
			//System.out.println("もう一度選んでねorz");
			return ans2;
	}
	
	public static String ng(){
		System.out.println("もう一度選択してください");
		return name();
	}
}

試したこと

ここに問題に対して試したことを記載してください。

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

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

Answer 1

既に解決されてますが、今後の参考ということで回答してみます。

• 選択肢から選ぶ
• 「はい」で終わり
• 「いいえ」または不正な選択肢が選ばれれば再度選択肢を表示

という仕様なので、もうちょっとシンプルに書けるかもしれません。
既定の選択肢以外はNOとする「ホワイトリスト」の考え方ですね。

その定義あたりと配列を使います。
あとtry-catchを使って「予期しない選択肢」も捕捉できそうです（特に全角文字列とか）

細かいところを言えば、変数名やメソッド名など、もう少し要件に寄せたほうがいいかもしれません。
例：name()というメソッド名をつけているが実行結果は「選択したフルーツ名」なので「フルーツを選択する」のが分かるほうがいい

import java.util.HashMap;
import java.util.Map;

public class Fruit {

    private static Map<Integer, String> fruitList = new HashMap<>();
    static {
    	fruitList.put(1, "メロン");
    	fruitList.put(2, "りんご");
    	fruitList.put(3, "バナナ");
    }
    private static Map<Integer, String> answerList = new HashMap<>();
    static {
    	answerList.put(1, "はい");
    	answerList.put(2, "いいえ");
    }
	public static void main(String[] args) {
        System.out.println("次の中から好きな果物を選んでください");

        String fruit = selectFruit();

        System.out.println("あなたの好きな果物は" + fruit + "ですね");
	}
	static String selectFruit() {
		try {
			fruitList.forEach((selectNumber, fruitName) -> System.out.println(selectNumber+":【" + fruitName + "】"));
			int ans = new java.util.Scanner(System.in).nextInt();
			if(fruitList.get(ans) == null) {
				return selectWrongNumber();
			}
			System.out.println(fruitList.get(ans) + "ですね");
			return finalAnswer(ans);
		}catch(Exception e) {
			return selectWrongNumber();
		}
	}
	static String finalAnswer(int ans) {
		try {
			answerList.forEach((selectNumber,answer) -> System.out.println(selectNumber+":【" + answer + "】"));
			int fAns = new java.util.Scanner(System.in).nextInt();
			if(fAns == 1) {
				return fruitList.get(ans);
			}
			return selectFruit();
		}catch(Exception e) {
			return finalAnswer(ans);
		}
	}
	static String selectWrongNumber() {
		System.out.println("正しいキーを入力してください");
		return selectFruit();
	}
}

※ざっとしか動作確認してないのでおかしいところあるかもしれませんけど・・
※思ってたより時間がかかってしまった。色々機能探りつつデバッグこまかくやりすぎたかも。

• HashMap:要素の格納と取り出し
• メンバ変数
• staticメソッド／static変数／staticイニシャライザ
• 例外処理
• Javaの例外処理について
• java.lang.Exception

Comments

[m.ts10806]

細かいですが再”帰”処理です。再起では全く意味が変わってしまう・・・。

[BluOxy]

>もうちょっとシンプルに書けるかもしれません。
ですね。
私は、シンプルに書くことについては質問で求められていなかったため、自分の回答内容では割愛していました。mts10806さんの回答ではそこが触れられており、根本を解決するという観点で丁寧な内容でしたので高評価しました。

To:tomoxさん
個人的にはコードの繰り返し部分がやはり1番気になるポイントになるので、「m」「r」「b」という個別の変数ではなく配列を覚えて使えるようになると良いです。
そうすると、ifによる分岐が3つから1つにまとまったり、変数が少なくなったりして、幸せになれると思います。

[m.ts10806]

BluOxyさん
コメントありがとうございます。
「選択肢」というところから「配列」を使うのは中々すぐには発想がいかないところではありますが、
配列を覚えると繰り返す処理とかコードをより簡単に書くことにつながるので、覚えておきたいですね。
あとデバッグ。

余談：
あとで読み返したら全部どうせ再入力促すわけだしInputMismatchExceptionなんてニッチなやつじゃなくットップのExceptionで良いんじゃないかと思ったわけです。（想定の動作も確認できたし）

[tomox]

mts10806さん
「再帰」のご指摘ありがとうございます(;'∀')
恥ずかしい。。
見本まで作成して頂いてありがとうございます！
本見ながら、ネット見ながら理解できるように頑張ります。

[tomox]

BluOxyさん
アドバイスありがとうございます。
配列を使ったコードも作ってみますね！
分からなかったまた伺いに来てしまいます。。

[m.ts10806]

フローチャート書いてみると良いかもしません。
そうすると流れを整理できて冗長な処理も減るかもしれません。

Answer 2

もし最初に3を入力して1を入力したら正しく終了しませんか？

次の中から好きな果物を選んでください
1:【メロン】
2:【りんご】
3:【バナナ】
3
バナナですね
1:【はい】
2:【いいえ】
1
あなたの好きな果物はバナナですね

下記のelseがif(ans == 3)に対して行われているため、1や2を入れたときに必ずelseの処理が実行されてしまいます。
つまり、「入力が3でなければelseの処理が実行される」というコーディングになっているということです。

if(ans == 1){
 //~
}
if(ans == 2){
 //~
}

//NOTE: ansが1や2のときでもelseの処理は実行される
if(ans == 3){
 //~
}else{
 //~
}

解決するにはif(ans == 2)やif(ans == 3)の前にelseを追加してelse if(~)の形式に変更してください。
これで「入力が1や2や3でなければelseの処理が実行される」というコーディングになります。

Comments

[tomox]

早速の回答ありがとうございます。
ご指摘の通り下記のように変更をしてみましたが
変化はありませんでした。（処理が違っていたらすみません）

else if(ans == 2){
else if(ans == 3){

元々、どの果物を選択しても最初に「はい」と答えると
正しく動作します。
ーーーーーーーーーーーーーーーーー
次の中から好きな果物を選んでください
1:【メロン】
2:【りんご】
3:【バナナ】
1
メロンですね
1:【はい】
2:【いいえ】
1
あなたの好きな果物はメロンですね
ーーーーーーーーーーーーーーーー

果物選択後に、「いいえ」を選んだ後の動作に不具合が出てしまいます。
お時間ありましたら、再度ご教示頂けると幸いです。

[BluOxy]

なるほど。
よく見るとngやnameメソッドの戻り値でnameメソッドを呼ぶよう作られています。
これは再帰処理と言います。

ng();と書いてあるメソッドを削除し、その行を全て
System.out.println("もう一度選択してください");
と置き換えれば動作するかと思います。

[tomox]

ありがとうございます(^^)/
無事に解決できました。

同じ処理なのでメソッドを作ったほうがコードがスッキリするかと
思ったのですが、「再起処理」なんですね(;'∀')・・（これから勉強します！！）

何度もありがとうございました！

[BluOxy]

return name();を書くのは、elseの中だけに留めた方が良いですね。
nameメソッドの至るところでnameメソッドを呼んでしまうと、何回呼ばれるか全然分からなくなってしまいますから。

[tomox]

アドバイスありがとうございます！
本当に始めたばかりで、質問する仕方にも迷ってるぐらいです・・
頑張ります！！

[BluOxy]

お節介かもしれませんが、今回の質問のような論理エラーを無くしたいのであれば、デバッガーというツールの使い方を覚えてみてください。

そうすると論理エラーに関する質問の大半はなくなるはずなので。
詳しくは「Java デバッグ」で検索してみてください。

デバッグが出来れば生産効率は2倍、3倍…否、桁倍は上がります。なので、やはりエンジニアは皆、デバッグが当たり前にできます。

[tomox]

色々と教えて頂きありがとうございます。
「java　デバック」勉強します！！

Answer 3

ここがおかしいんじゃないですかねぇ

if(ans == 1){
   System.out.println(m + "ですね");
   int ans2 = Choose();            
   if(ans2 == 1){
       return m;
   }else{
       ng();
   }
}

if(ans == 2){
   System.out.println(r + "ですね");
   int ans2 = Choose();            
   if(ans2 == 1){
        return r;
   }else{
        ng();
   }
}

if(ans == 3){
   System.out.println(b + "ですね");
   int ans2 = Choose();            
   if(ans2 == 1){
       return b;
   }else{
       ng();
   }
} else {  // このelse文
   System.out.println("正しいキーを入力してください");
   return name();
}

これだと３を選ばなかった時は必ず正しいキーを入力して下さいになります

やりたいことは恐らく↓なんじゃないでしょーかねぇ

if(ans == 1){
   System.out.println(m + "ですね");
   int ans2 = Choose();            
   if(ans2 == 1){
       return m;
   }else{
       ng();
   }

} else if(ans == 2){
   System.out.println(r + "ですね");
   int ans2 = Choose();            
   if(ans2 == 1){
       return r;
   }else{
       ng();
   }

} else if(ans == 3){
   System.out.println(b + "ですね");
   int ans2 = Choose();            
   if(ans2 == 1){
       return b;
   }else{
       ng();
   }

} else {  // 1,2,3以外で判定したいならこうじゃないのかな
   System.out.println("正しいキーを入力してください");
   return name();
}

Comments

[tomox]

いつも回答ありがとうございます(^^♪
先程、別の方にも同じご指摘を頂きましたが「再起処理」？（これから勉強する内容なので。。）
ということで、動作に変化がありませんでした。

別の処理を教えて頂き解決できました！
また質問沢山出るかと思いますが、これからも宜しくお願いします。