java のScannerクラスの使い方が誤っておりエラーが出てくる

Question

前提

数字を取得して合計するクラス（以降　Sumクラス）と
Sumクラスを使うmainクラスの両方でScannerクラスのインスタンスを作成しています。

実現したいこと

・SumクラスとMainクラスの両方でScannerクラスのnextInt()メソッドを使用したいです。

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

Exception in thread "main" java.util.NoSuchElementException
	at java.base/java.util.Scanner.throwFor(Scanner.java:941)
	at java.base/java.util.Scanner.next(Scanner.java:1598)
	at java.base/java.util.Scanner.nextInt(Scanner.java:2263)
	at java.base/java.util.Scanner.nextInt(Scanner.java:2217)
	at Main.sum(Main.java:49)
	at Main.main(Main.java:42)

該当のソースコード

public class Main {
   
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        int h = sc.nextInt();
        int w = sc.nextInt();
        sum();
    }
    
    public static int sum(){

        int sum=0;
        Scanner sc = new Scanner(System.in);
        sum = sc.nextInt();

        System.out.println("sum="+sum);
        return sum;
    }
}

試したこと

Overview (Java Platform SE 8 )を呼んだのですが、解決方法が分かりませんでした。
「各種のアクセッサ・メソッドによってスローされ、リクエストされている要素が存在しないことを示します。」
と記載がありましたが、入力はちゃんと行っております。
Scannerクラスのインスタンを両メソッドで生成しているのが間違いでしょうか?

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

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

Comments

[jimbe]

>数字を取得して合計するクラス（以降　Sumクラス）と
>Sumクラスを使うmainクラス

クラスは class Main のみです。
void main(…) と int sum() はメソッドです。
そして、どこにも"合計する"処理はありませんが…。

Answer 1

mainメソッドと sum メソッドの外に、static を付けて sc を書けばよいでしょう。

import java.util.Scanner;

public class Main {
	static Scanner sc = new Scanner(System.in);

	public static void main(String[] args) {
		int h = sc.nextInt();
		int w = sc.nextInt();
		sum();
	}
	
	public static int sum() {
		int sum = sc.nextInt();
		System.out.println("sum=" + sum);
		return sum;
	}
}

Comments

[chan...]

ありがとうございます。各メソッドの中でScannerのインスタンスを生成するのではなく
Staticをつけてヒープ領域に生成するのですね。
プログラムも動きました。
回答ありがとうございます。

[jimbe]

肝は「Staticをつけてヒープ領域に生成する」ことではなく「main/sub どちらのメソッドからも同じオブジェクトを使えるようにする」ことでしょう。
static なのは、メソッドが static だからだけではないかと思います。

[chan...]

jimbeさんありがとうございます。
＞static なのは、メソッドが static だからだけではないかと思います。
staticメソッドからは非staticフィールドにアクセスできないからですね。
勘違いをしていましたが訂正していただきありがとうございました。

Answer 2

同一のストリームから複数の Scanner を生成すると、それぞれの動作が絡まって意図しない動作をする可能性があります。
Main で作成したものを sum のパラメータとして渡す等で同一のオブジェクトを使用するようにしてください。

Comments

[chan...]

ご回答ありがとうございます。
初心者で、同一ストリームでScannerの複数生成ができないことを存じておりませんでした。
今後気をつけます。

[jimbe]

出来なくはありません。実際にプログラムは(意図した通りでなくても)動作はしていたようですので。
本質は、 Scanner は渡されたストリームが同時に他のオブジェクトでも使われることを想定していないだろうということです。(そしてストリームではそれが普通です。)
例えば動作を高速にするため、入力されたデータは全て自身の内部に取り込んでおき、 nextInt が呼ばれたらその中から数値を取り出す…ということをやっているかもしれません。
もしそのようなことをしていたら、二つ目の Scanner でデータを読もうとしても何も残っていないことになります。

Scanner はウインドウ等 GUI を使い始めると使わなくなる(使えなくなる)類ですので CUI で始める入門時によく使われますが、実はデリケートで難しいものです。