ループ文の終了について

数当てゲームで正解した時にプログラムを終了させたいのですが終わることができません。
初歩的なことで申し訳ありませんがお力添えをおねがいします。

public class Number {
public static void main (String[] args) {
System.out.println("数当てゲームをします");
for(;;){
System.out.println("一桁の数字を入力してください");
int number[] = {3,4,9};
for (int n: number){
int input = new java.util.Scanner(System.in).nextInt();
//正解した時の処理
if (input == n){
System.out.println("あたり");
System.out.println("ゲームを終了します");
break;
}
else {
System.out.println("残念もう一度");
}
}
}
}
}

行動規範の内容に同意します

回答3件

ベストアンサー

breakが内側のfor文にしか効かないので、無限ループの外側のfor文を抜け出すことができません。
何らかの方法で外側のforを抜ける必要があります。対処法としては
0. 外側のforでフラグを用意し、内側の条件でフラグを立て、内側を抜けた後判定

java
1for(;;){ 
2    System.out.println("一桁の数字を入力してください"); 
3    int number[] = {3,4,9}; 
4    //正解判定用フラグ
5    boolean right = false;
6    for (int n: number){ 
7        int input = new java.util.Scanner(System.in).nextInt(); 
8        //正解した時の処理 
9        if (input == n){ 
10            System.out.println("あたり"); 
11            System.out.println("ゲームを終了します"); 
12            right = true;
13            break; 
14        } else { 
15            System.out.println("残念もう一度"); 
16        } 
17    } 
18    if (right) {
19        break;
20    }
21}

外側のforにラベルを設定し、内側のbreakでラベル指定する(あまりお勧めしない)

投稿2017/05/28 04:03

編集2017/05/28 05:07

swordone

総合スコア20649

sankakusan

2017/05/28 04:28

とても早い回答ありがとうございます！つまり無限ループではなくて条件をつけてあげればいいという解釈でよろしいでしょうか？

swordone

2017/05/28 04:32

それでもいいですし、1.で判定の結果breakするという方法もありです。

sankakusan

2017/05/28 04:58

2.で解決できましたがあまりオススメされないとのことで何度も失礼だとは存じますがお手間でなければ１、についてもう少し教えていただいてもいいですか？

sankakusan

2017/05/28 05:15

ソースコード見ることができました本当にありがとうございます！なるほどーと関心の声しか出ません本当にありがとうございました！！

行動規範の内容に同意します

個人的には無限ループ部分を関数にして return で抜ける
って方がこのみかな。ただし、場合によりけりですが。
--- 追記 ---
解決したようですね。
ちなみに、こんなコードはどうでしょうか。

Java
1public class Number {
2  public static void main (String[] args) {
3    System.out.println("数当てゲームをします");
4    boolean keep = true;
5    int number[] = {3,4,9};
6    while(keep) {
7      keep = quest(number);
8    }
9    System.out.println("ゲームを終了します");
10  }
11  public static boolean quest(number) {
12    boolean ret = true;
13    System.out.println("一桁の数字を入力してください");
14    for (int n: number) {
15      int input = new java.util.Scanner(System.in).nextInt();
16      //正解した時の処理
17      if (input == n) {
18        System.out.println("あたり");
19        ret = false;
20        break;
21      }
22      else {
23        System.out.println("残念もう一度");
24      }
25    }
26    return ret;
27  }
28}