for文とif 文の組み合わせ

Question

public class Main {
    public static void main(String[] args) {
        Scanner a = new Scanner(System.in);
        int N = a.nextInt();
        int sc = 0;
        int bc = 0;
           for (int n = 0; n < N; n++){
            String s_n = a.nextLine();
           if (s_n.equals("strike")){ 
            System.out.println("strike!");
            sc += 1;
           }
            else {
             System.out.println("ball!");
             bc += 1;}
        }
        if (sc == 3) System.out.println("out!");
        if (bc == 4) System.out.println("fourball!");
      }
    }

N回数分入力された文字列の判定を行たいのですが1回目の判定が2回出力されてしまいます。
具体的には以下の通りです。
出力したい文字
ball!
strike!
ball!
strike!
出力される文字
ball!
ball!
strike!
ball!
strike!
なぜこうなってしまうのかわかりません。
どなたかご教授ください。

Answer 1

原因は既にseastar3さんが指摘されている通りだと思います。
しかし題材が題材なだけに、そもそもNの入力を受け付ける必要がないのでは？

import java.util.Scanner;

public class Main {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

        int sc = 0, bc = 0;
        while(true) {
            String s_n = scanner.nextLine();

            if (s_n.equals("s")) {
                System.out.println("strike!");
                sc += 1;
            }
            else if (s_n.equals("f")) {
                System.out.println("foul!");
                sc = Math.min(++sc, 2);
            }
            else {
                System.out.println("ball!");
                bc += 1;
            }
            System.out.println(sc + " " + bc);

            if (sc == 3) {
                System.out.println("out!");
                break;
            }
            if (bc == 4) {
                System.out.println("fourball!");
                break;
            }
        }
    }
}

実行例

s
strike!
1 0
f
foul!
2 0
f
foul!
2 0
b
ball!
2 1
b
ball!
2 2
f
foul!
2 2
s
strike!
3 2
out!

Comments

[sankakusan]

Nはfor文の条件に使う予定でしたがご提示いただいたコードの
ほうがスマートで正解なのですね。
私の書いたコードは誤作動を起こしましたし、、、
とても勉強になりました。ありがとうございます。

Answer 2

私も連続読み込みで矛盾がおきたことがあります。

記事「JavaのScannerとか.nextLine()の挙動をよく分かってなかった話」
のような挙動です。解決策も出ていますのでご参考に。

Comments

[sankakusan]

ご回答いただきありがとうございました。
とても参考になりました。
じっくりと読ませていただきます。

Answer 3

不可思議な挙動をするように見えるのは、最初に回数を入力する際のキー入力が２回に分けて使われているからです。
何が起きているのか、順にみていきましょう。

質問のプログラムを起動して、キーボードで　4↓　と入力します（ "↓"は、改行(Enter)キー）
そうするとScannerのバッファには、"4\n"　というデータが入ります。（"\n"は改行文字）

続いてnextIntメソッドで

1. 最初の文字 "4"は数字なので、取り出して数字に変換するための文字列(nextIntのコード中にある)に入れておく。
2. ２文字目は"\n"で数字ではないので、その手前までに得られた"4"という文字を整数 4 に変換して返します。Scannerのバッファには、"\n"　というデータが残ります。

次に、nextLineメソッドでバッファに残っている"\n"が返されます。Scannerのバッファは空になります。
"\n"は"Strike"とequalではありませんから、"ball!"が画面に表示されます。（これが１つ目の余分なball!です。）

これ以降は、キーボードから入力されるのがStrikeかballかによって、"strike!"もしくは"ball!"が表示されていきます。

＝＝＝
質問には「1回目の判定が2回出力されてしまいます」と書かれていますが、実際は１回目の判定の前に"ball!"が出力されているのです。

１回目の余分な"ball!"を出さないようにするには、nextIntメソッドのすぐ後にnextLineメソッドを実行してScannerのバッファを空にします。
具体的には質問のコードの

N = a.nextInt();

を

N = a.nextInt();
s_n=a.nextLine();

に変えればOK

Comments

[sankakusan]

懇切丁寧に解説いただいてありがとうございます。
とてもわかりやすく解決策まで提示していただいて
本当にありがとうございました！

Answer 4

入力がこういう形式ではありませんか？

4
ball
strike
ball
strike

この場合、nextIntで読み込んだあとのカーソルが4の直後になるため、最初のnextLineが空文字を返します。
このため、最初のループではelseに入ってしまうため、"ball!"を出力してしまうのです。
elseではなくelse ifで判定するなり、読み捨てするなりなどが必要です。

Comments

[sankakusan]

おっしゃる通りです。
else if　で条件をさらに限定して動作を安定させる
べきなんですね。勉強になりました。
回答いただきありがとうございました。