2
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
public class LoToi1 {
    public static void main(String[] args) {
            //入力するための準備
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        System.out.println("入力された年が閏年か判別します。");
        for(;;){
        //メッセージを表示する
            System.out.println("西暦を整数入力してください。");
            try {
                //入力する情報を取得
                String line = br.readLine();
                //文字列を数値に変換
                int a = Integer.parseInt(line);
                //閏年の判定処理
                if (a%4 == 0 && a%100 != 0 || a%400 == 0) {
                    System.out.println("閏年です。");
                } else { //それ以外の処理
                    System.out.println("閏年ではありません。");
                }
                System.out.println("続けますか?Y/N");
                boolean yn = false;
                while(!yn) {
                    String input = br.readLine();
                    if(input.equals("Y") || input.equals("y")) {
                        yn=true;
                    } else if(input.equals("N") || input.equals("n")) {
                        yn=false;
                 break;
                    } else {
                        System.out.println("YかNを入力してください。");
                    }    
                }    
            } catch (IOException e) {
                //入出力時に失敗があった時に行われる処理
                e.printStackTrace();
            } catch(NumberFormatException e){
                //数値以外の値が入力された場合の例外処理
                System.out.println("整数で入力してください");
            }
        }
    }
}

else if のbreak;なぜ効かないですか?

1
  • 1
    この場合のbreak は、直近のwhileをブレイクします。for(;;)ではありません。 直接関係ないですが、breakの行の前部分に全角スペースがあります。半角スペースを使うようにして下さい。
    – BLUEPIXY
    Commented 2015年7月21日 1:30

2 件の回答 2

6

質問の意図からすると、外側のfor()ループをbreakで抜けようとしているのですが、直近のwhile(!yn)ループを抜けるだけになっていますね。

あまり使いませんが、ラベル付きbreakにすると解決します。

public class LoToi1 {
    public static void main(String[] args) {
        //入力するための準備
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        System.out.println("入力された年が閏年か判別します。");
        outLoop: for(;;){ // ラベルをつける
            //メッセージを表示する
            System.out.println("西暦を整数入力してください。");

            // 中略

            boolean yn = false;
            while(!yn) {
                String input = br.readLine();
                if(input.equals("Y") || input.equals("y")) {
                    yn=true;
                } else if(input.equals("N") || input.equals("n")) {
                    yn=false;
                    break outLoop; // ラベル指定でbreak
                } else {
                    System.out.println("YかNを入力してください。");
                }    
            }

            // 中略    
        } // end of for(;;)
    } // end of void main()
}
3
  • この回答のほうがフラグをつかうよりエレガントですね。 Commented 2015年7月21日 4:58
  • 2
    ・_・)そんなときには+票を。。。w
    – fkm
    Commented 2015年7月21日 5:18
  • 入れ子のループからの脱出は、GOTO文が叩かれているときの反証としてよく上がっている気がします。breakは制限付きのGOTOと言えますね。もう少し複雑な例なら例外を投げてtry-catchでしょうか? Commented 2015年7月21日 8:07
1

breakを置いているのにループが継続してしまう理由としては、
この場合のbreak は、直近のwhileをブレイクするからで上層のforはブレイクしません。

boolean yn は、大本のループを継続するかどうかを表す真偽値ですから
次の様に

boolean yn = true;
for(;yn;){

for ループの前に置いてループの条件に使うようにします。
あるいは、

for(boolean yn = true; yn; ){

のようにします。

whileループは、YyNnのどれかが入力されるまでは繰り返すので、
無限ループにしてYyNnのどれかが入力されたら真偽値をynにセットしてループを抜けるようにします。

while(true){
    String input = br.readLine();
    if(input.equals("Y") || input.equals("y")){
        yn=true;//既定値なので必要は無い
        break;
    } else if(input.equals("N") || input.equals("n")){
        yn=false;
        break;
    }else{
        System.out.println("YかNを入力してください。");
    }
}
2
  • 解決できました、ありがとうございます
    – kjkkkk
    Commented 2015年7月21日 2:17
  • この回答で満足されたのであれば承認してください。
    – BLUEPIXY
    Commented 2015年7月21日 2:32

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。