メインメソッドでの入力と、違うクラスでの入力の際の挙動が違う

Question

###前提・実現したいこと
最初に二つの数を入力してもらい、その二数の積だけ整数値の入力を受け取るプログラムを書いていました。

例として、以下の値をコンソールにコピーアンドペーストしました。
3 3
1 2 -2
4 5 -1
7 3 6

この例だと3*3=9個の入力を受け取る(9個の入力は下の1,2,-2,4,5,-1,7,3,6です)
###発生している問題・エラーメッセージ

上手くsc.nextInt()が入力を受け取ってくれない

import java.util.Scanner;

public class Main {

	public static void main(String[] args){
		Scanner sc = new Scanner(System.in);
		int n = sc.nextInt();
		int m = sc.nextInt();
		Calc keisan = new Calc();
		keisan.seki(n,m);
		sc.close();
	}
}

class Calc{

    void seki(int n, int m){
        Scanner sc = new Scanner(System.in);
        int[] kazu = new int[n*m];
        for(int i =0;i<n*m;i++){
            kazu[i] = sc.nextInt();
        }
        sc.close();
    }
}
```Java


###試したこと
発生している問題にある通り、上の書き方だと入力待ち(最下段の7 3 6だけは入力されてる)状態になってしまい
思ったように動きません。
この入力部分を全てメインメソッドに移して、呼び出す代わりにメインメソッド内で処理させると上手く動作しましたが、その理由がわかりません。

###補足情報(言語/FW/ツール等のバージョンなど)
Eclipseを使って勉強していました。

Answer 1

Scannerクラスはコンストラクターに渡した入力ストリームから文字を「先読み」します。
先読みというのは実際にデータが必要になる前にずっと先の方まで読み込んでしまう動作のことを言います。

この例でいいますとmainメソッドの最初のnextIntの呼び出し時にコンソールから入力した4行分の文字は全てmainメソッドで生成したScannerインスタンスが読み込んでしまいます。

同じ入力元(System.in)からスキャンするのであれば別々のScannerインスタンスを生成してはならず、常に同一のScannerインスタンスを用いなければなりません。

質問者さんのコードに当てはめるなら、sekiメソッドへmainメソッドからScannerインスタンスを引数として渡せば期待通りに動きます。

---

追記：他の方の回答と矛盾してはいないと思うので追記しました。

自分が動かしたときは、質問者さんと同様に「最初に4行分をコピー＆ペーストでコンソールへ張り付ける」ということをしました。こうするとmainメソッドの最初のnextIntでSystem.inから4行分全部読み込まれてしまいます。

ScannerはInputStreamから文字をデコードするためにデフォルトで1024文字分のバッファーを持っており、InputStreamから読み込めるだけそのバッファーへ文字をデコードしてしまいます。よって最初に1行しか入力しない状態で動かせばオリジナルのコードのままでも動きます。それが他の方の回答にある内容だと思います。

しかし前述のとおり最初から4行分をコピー＆ペーストで入力すると期待通りには動きませんので、Scannerのインスタンスを複数作らないのがプログラムとしての正しい対処だと思います。

Comments

[CR_GARO]

Scannerクラスの性質が分かり、また何故そのような挙動になってしまったかも理解でき、勉強になりました。ありがとうございます。

Answer 2

全部入力された値は入っているようですよ。

提示されたコードには、入力された値がどこにも出力されていないのでデバッグモードで試しています。
どこにもデータを出力されていないコードなので、再度書いたコードと入力の方法を確認すると良いでしょうか…。

Comments

[CR_GARO]

回答ありがとうございます。出力を書き忘れていました。申し訳ありません。

Answer 3

いまいちやりたい事がわかりませんが、
Scanner sc = new Scanner(System.in);
が void sekiの中にもあるためではないでしょうか？

コピペで入力数字を全部いれるのではなく、最初に"3 3"と打ちこんで、その後3×3の数字をコピペすれば望みの結果になりそうですよね？

Comments

[CR_GARO]

質問の意図がわかりにくく、申し訳ありません。
動作の方はその通りに入力したら希望する動作をしました。