java Scannerのcloseするタイミング

Question

package iroiro;

import java.util.Scanner;

public class Tecfulnisesatu {

public static void main(String[] args) {

	int i;

	Scanner sc = new Scanner(System.in);

	int honmono = sc.nextInt();
	int nisemono = sc.nextInt();
	int honretu[] = new int[honmono];
	int niseretu[] = new int[nisemono];

	
	for(i = 0; i < honmono; i++) {
		honretu[i] = sc.nextInt();
	}
	for(i=0;i<nisemono;i++) {
		niseretu[i] = sc.nextInt();
	}
	
	sc.close();
}
}

最後に「sc.close();」を入力しましたが、scが閉じられることがないと注意されてしまいます。for文の手前に置くと、閉じられるようです。
言葉足らずになっていると思いますが、ご教授お願い致します。

Comments

[int32_t]

> scが閉じられることがないと注意

その「注意」をそのまま質問文にコピペしてください。
コンパイル時に注意されるということで合ってますか? コンパイルしているのは本当に質問文のままのコードですか?

[mochi1234]

「リソース・リーク: 'sc' が閉じられることはありません」こちらになります。

「sc.close();」を入力した時点で、上記の文が表示されます。

[int32_t]

「sc.close();」がない場合にその注意が出るのは理解できるのですが、足すと出るというのは妙ですね。

[mochi1234]

申し訳ございません。言葉不足でした。
ない場合にも注意が出ていて、「sc.close();」を入力しても注意が出ている状態です。

[int32_t]

ああ、なるほど。では maisumakun 氏の回答の通りですね。

Answer 1

sc.nextInt()が例外を投げた場合に、最後のsc.close()までたどり着きません。

try with resourcesで書くか、finallyで閉じるかの必要があります。

Comments

[mochi1234]

maisumakunさん回答ありがとうございます。
「sc.nextInt()が例外を投げた場合」とはどういうときでしょうか？知識が浅いので理解できませんでした。申し訳ございません。

ご教授お願い致します。

[maisumakun]

> 「sc.nextInt()が例外を投げた場合」とはどういうときでしょうか？

・InputMismatchException - 次のトークンがInteger正規表現に一致しないか範囲外である場合
・NoSuchElementException - スキャンする入力がなくなった場合
・（IllegalStateException - このスキャナがクローズしている場合）

https://docs.oracle.com/javase/jp/8/docs/api/java/util/Scanner.html#nextInt--

[mochi1234]

知識がまだ浅かったです。ありがとうございました。

[momon-ga]

一応、コメント残しておきます。
ランタイムの例外は、たぶんコンパイラは感知しないのじゃないかと思います。
実際に
Scanner sc = new Scanner(System.in);

int honmono = sc.nextInt();
System.out.println(honmono);

sc.close();
の場合には、警告にならないのですよね。

Answer 2

どこかに悪い教本でもあるのだろうか・・・
C言語の名残？

スコープの広い変数を作るのはやめた方がいいです。

	int i;
	for(i = 0; i < honmono; i++) {
		honretu[i] = sc.nextInt();
	}
	for(i=0;i<nisemono;i++) {
		niseretu[i] = sc.nextInt();
	}

を

//	int i;
	for(int i = 0; i < honmono; i++) {
		honretu[i] = sc.nextInt();
	}
	for(int i=0;i<nisemono;i++) {
		niseretu[i] = sc.nextInt();
	}

にすればよいかと。

Comments

[maisumakun]

質問の問題とは全く無関係かと思われます。

[momon-ga]

どうしてもｉを事前に宣言したいなら、アノテーションで回避することも可能です
https://teratail.com/questions/337590

[mochi1234]

momon-gaさん回答ありがとうございます。上記のコードに書き直したら警告が表示されなくなりました。
以前にC言語を勉強していました。

質問が２点あります。
１．なぜmomon-gaさんのコードだと「sc.close();」が実行されるのか
２．javaであれば、その都度変数を宣言すべきなのか

ご教授お願い致します。

[momon-ga]

＞ 質問の問題とは全く無関係かと思われます。
実際に警告が消えますけど・・・
記述の問題で警告の有無が変わるなら、関係はありそうですが。

[momon-ga]

> 「sc.close();」が実行されるのか

これは、maisumakunさんの回答にありますが

実行されるか、どうかはわかりません。
例外が発生した場合は実行されないでしょう。

ただ、コンパイラが警告を出さなくなるのはご確認している通り事実です。

> javaであれば、その都度変数を宣言すべきなのか
javaに限らず、無駄にスコープを広げないのが普通かと思います。

[maisumakun]

> 実際に警告が消えますけど・・・

そうだったのですね、失礼しました（てっきり別な問題を指摘しただけの回答かと思いこんでいました）。

[momon-ga]

いえいえ。お気になさらずに。

[mochi1234]

最後に一つよろしいでしょうか。

momon-gaさんのコードであれば、例外の可能性が消えるから警告が消えるということですか？

[momon-ga]

あ、例外の話はたぶん関係ないと思います。
Maisumakunさんのコメントにつけてますが・・・

Scanner sc = new Scanner(System.in);

int honmono = sc.nextInt();
System.out.println(honmono);

sc.close();
は、警告でないはずです。

[mochi1234]

ありがとうございます。

https://teratail.com/questions/337590
先ほど送っていただいたURLのベストアンサーである文の、「変数iのスコープが広いのでループがあるとscan.close()にたどり着けない可能性があるとコンパイラが判断しているのではなかろうかと。」が理解できません。変数iのスコープが広いことで、「close」にたどり着けない可能性があるのはなぜですか？

何度も質問をして申し訳ございません。

[momon-ga]

すみません。端的にいうと、わかりません。

ちなみに
”たどり着けない可能性がある”　のでなく、”可能性があるとコンパイラが判断しているっぽい”です

コンパイラが、そう判断してる”っぽい”だけで、なぜ？と言われると・・・・
答えられないです。ごめんなさい。
※インスタンスフィールドを参照してるとかだとマルチスレッドがぁ。。。とかなりそうですが。

[mochi1234]

ありがとうございます。
多くの質問に回答して下さり、とても勉強になりました。
これからも調べながらコードを書くようにしていきます。

[int32_t]

Eclipse のバグとして登録されているみたいです。
https://github.com/eclipse/eclipse.jdt.ls/issues/1692
https://bugs.eclipse.org/bugs/show_bug.cgi?id=561589

[mochi1234]

int32_tさんありがとうございます。
すっきりしました。