JavaでScannerをメソッド化してint配列を簡単に作成し、例外処理も行いたい。

Question

前提・実現したいこと

前提
スキャナーをメソッド化して各クラスで再利用したいです。
例外処理の練習をしたいです。

実現したいこと
Javaで標準入力で受け取った数の配列を作成し、そこに順次数値を代入し、文字列が入力した場合は
検知して再入力を促したいです。

例えば5個の要素数をもつ配列を作るとします。
int[] array = {a,b,c,d,e};

というのを楽に作りたいと思いました。

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

エラーメッセージ
Exception in thread "main" java.util.NoSuchElementException
	at java.base/java.util.Scanner.throwFor(Scanner.java:937)
	at java.base/java.util.Scanner.next(Scanner.java:1594)
	at java.base/java.util.Scanner.nextInt(Scanner.java:2258)
	at java.base/java.util.Scanner.nextInt(Scanner.java:2212)
	at sample.sc(sample.java:41)
	at sample.main(sample.java:8)

該当のソースコード

ソースコード
import java.util.InputMismatchException;
import java.util.Scanner;

public class sample {
	public static void main (String[] args){
	
		int num = sc();
		int[] array = array(num);
		
		for(int i = 0; i < array.length ; i++) {
		System.out.println(array[i]);
	}
		
	}
	
	//配列たくさん作る
	public static int[] array(int num) {
		
		//numの数値分の配列を作る
		int[] array = new int[num];
		
		//for文で中に数値を入れる
		for(int i = 0; i < array.length ; i++) {
			array[i] = sc();
		}
		
		return array;
		
	}
	
	

	public static int sc() {
		Scanner sc = new Scanner(System.in);
		
		int num = 0;
		
		//エラーのおそれがある処理
	    try{
	        num = sc.nextInt();
	        sc.close();
	      }catch (InputMismatchException e){
	        System.out.println("半角の数値を再度入力してください");
	        num = sc();
	      }
	    return num;
	}


}

試したこと

sc();は単体では正常な入力を受け取るまでループすることを確認しました。

for文のみでは配列の作成を行うことができました。
スキャナをメソッド化してから動かなくなりました。

問題がどこにあるのかを突き止めるために、プログラムの細部を何度か書き換え、おそらくsc();が二度目の呼び出し以降使えないのではないか？という今の能力で解決できない疑問に当たりました。
エラー文でグーグル検索をかけましたが、プログラミング歴が１か月程度で有効な回答にたどり着くkとができませんでした。

気になってしょうがないので、ぜひお答えいただければと思います。

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

openJDK17 Eclipse

Comments

[m.ts10806]

>再入力を促したい
「受け付ける値が入ってくるまでwhileでループ」ではダメなのでしょうか。

[Sango]

こんにちは。
回答ありがとうございます。

私が書いている最中に思いついたものが上記のコードでした。
調べるとwhileでのループもあるようですね。
この問題が解決したらwhileでのループでも書いてみたいと思います。
ただ、現在の私のやり方だと何がいけないのかを特に知りたいです。

よろしくお願いいたします。

[jimbe]

> sc();は単体では正常な入力を受け取るまでループすることを確認しました。

いえ、これは再帰していますので、外見的には正常に見えますが、時限爆弾を抱えています。
sc の中で sc を呼ばないでください。

Answer 1

現在の私のやり方だと何がいけないのか

おそらくこれ
Scannerを使った時に入力をしてないのにNoSuchElementExceptionが発生する

即close()しているから。
close()外すと一応エラーは出ません（想定通り動くかは別）
close()するのは利用が終わる時と思ってください。
Scanner sc = new Scanner(System.in);←ここはmainメソッドに書いたほうが良さそうです。

蛇足：

• Javaのコーディングルールとしてクラス名の１文字目は大文字です
• インデントがグチャグチャです。可読性は非常に大事です。

コードフォーマット機能や構文チェック機能のあるエディタ、例えばeclipseのようなIDEを利用しましょう。

Comments

[Sango]

はじめまして！
コーディングルールなどの指摘非常に助かります。
エディタとしてeclipseを使ってまだ1週間もたっていないのですが、今の時点では書いているうちにどんどんインデントがぐちゃぐちゃになり、自分でも読んでいられなくなってしまいます。
｛｝を自分ですら追えなくなってしまってコンパイルエラーになりがちです。。

きれいなコードの書き方の本なども読んでみようかと思います。

とりあえずeclipseがインデントを入れたら逆らわずにそのまま字下げしておくだけで幾分かましになるでしょうか？

[LouiS0616]

Ctrl + Shift + F でフォーマットかけると良いかもです。

[m.ts10806]

>eclipseがインデントを入れたら逆らわず
基本、自動で入るインデントに任せます。

[Sango]

お二人ともありがとうございます。
本当に初学者でなるべく自分の力で理解しようと思っていて人にコードを見せたのも初めてでした。
良い経験になりましたので、これからもぜひよろしくお願いいたします。

[m.ts10806]

自分で理解しようとするのは大事です。
ドキュメント、リファレンスを見に行く癖が付けられると良いと思います。
Scannerなら下記。
https://docs.oracle.com/javase/jp/8/docs/api/java/util/Scanner.html

Answer 2

標準入力に対するScannerは、一度closeするともう一回使えません。

Scannerインスタンスをmainメソッド側で作って引数で取り廻すか、
フィールドとして保持するようにすれば良いでしょう。

その場合、mainメソッド終了前にcloseしてあげて下さい。

エラー再現

import java.util.Scanner;

class Main {
    public static void main(String[] args) {
        Scanner sc;
        
        sc = new Scanner(System.in);
        System.out.println(sc.next());    // ここではちゃんと読める
        sc.close();

        sc = new Scanner(System.in);
        System.out.println(sc.next());    // エラー!!
        sc.close();
    }
}

実例

ダメな例 (2回目以降の呼び出し時にNoSuchElementExceptionを送出する)

int read() {
    Scanner sc = new Scanner(System.in);
    int ret = sc.nextInt();
    sc.close();
    return ret;
}

良い例

int read(Scanner sc) {
    int ret = sc.nextInt();
    return ret;
}

Comments

[Sango]

Scannerは1度closeすると再利用できない。勉強になりました！！
確かにcloseを消すと機能しました。
こういったケースもあるのだと学ばせていただきました。

インスタンス関連は勉強中なのですぐ実装できませんが、mainでscannerのインスタンス化までやってみたいと思います。

[Sango]

一番最初に解決策として提示してくださり、また次の勉強にもつながりました。
ありがとうございます。

Answer 3

Scanner オブジェクトを static に持つやり方もあります。

import java.util.InputMismatchException;
import java.util.Scanner;

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

    public static void main (String[] args){
        int num = sc();
        int[] array = array(num);
        for (int i = 0; i < array.length ; i++) {
			System.out.println(array[i]);
		}
    }

    public static int[] array(int num) { //配列たくさん作る
        int[] array = new int[num]; //numの数値分の配列を作る
        for (int i = 0; i < array.length ; i++) { //for文で中に数値を入れる
            array[i] = sc();
        }
        return array;
    }

    public static int sc() {
        int num = 0;
        try { //エラーのおそれがある処理
            num = sc.nextInt();
        } catch (InputMismatchException e) {
            System.out.println("半角の数値を再度入力してください");
			sc.next(); // 誤入力の読み飛ばし
            num = sc.nextInt();
        }
        return num;
    }
}

全角の数字を入力しても正しい数値として読み込まれます。
try-catch は、数字ではない文字を入力した場合への対応でしょう。
再入力でも誤入力の場合どうしますか？

Comments

[Sango]

おはようございます。
回答ありがとうございます。
ご指摘の通り、書いていた当時は半角以外はエラー返ってcatch側に処理が行くだろうと
予想しており、実現したいこと（半角数値取得）に対して間違ったコードを書いていました。

staticフィールドやインスタンスフィールドに対する理解がまだ勉強できておらずよくわかっていません。
今日も勉強していきますので勉強次第、ご指摘されたようなことを実現できるコードに回収したいと思います。

Answer 4

        num = sc.nextInt();

これでは数値以外の入力した時点で詰みます。
どんな入力をしてもイケるようにしましょう

Comments

[m.ts10806]

catchされるのでそういう詰み方はしません。

[LouiS0616]

@y_waiwai さん
例外機構ってご存じですか。
https://docs.oracle.com/javase/jp/8/docs/api/java/util/Scanner.html#nextInt--