closeをどこに書けばいいのか

Question

int型の配列に入力した数字を出力するものを作ろうとしましたが下記のエラーが出ます。「Scanner等を使うときはとりあえずclose」という考え方ではいけないのでしょうか。

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

リソース・リーク: 'scan' が閉じられることはありません	

該当のソースコード

import java.util.Scanner;

public class array1 {

	public static void main(String[] args) {
		Scanner scan = new Scanner(System.in);
		int array[] = new int[5];
		int i;
		
		for(i = 0; i < array.length; i++) {
			array[i] = scan.nextInt();
		}
		
		System.out.println("back");
		
		for(i = 0; i < array.length; i++) {
			System.out.println(array[i]);
		}
		
		scan.close();
	 }
}

Comments

[maisumakun]

> エラーメッセージ

このエラーメッセージは、どこへどのように表示されたものですか？

[tanatana32]

eclipseの問題の欄で警告として表示されてまいます。質問の意図が理解できていなかったらすみません

[BeatStar]

もしかして、コマンドプロンプトとかから起動していませんか?
それなら閉じられないのも当たり前かと。

[tanatana32]

コマンドプロンプトからはコンパイル、実行した場合は思った通りの動作をしてくれるのですがeclipseでは実行できないです。

Answer 1

eclipseのリソースリークは警告がでるだけでデフォルトの設定では実行できると思います。
※[java]-[コンパイラー]-[警告/宣言]で、設定できます

で、警告がでる理由は想像ですが、変数iのスコープが広いのでループがあると
scan.close()にたどり着けない可能性があるとコンパイラが判断しているのではなかろうかと。
※ちなみに、forループそのものがない場合も、警告になりませんでした。（iの初期化は必要）

一般的なiのループの書き方は以下のようにするかと思います。
この場合、警告がでませんでした。

	public static void main(String[] args) {
		Scanner scan = new Scanner(System.in);
		int array[] = new int[5];
//		int i;

		for (int i = 0; i < array.length; i++) { // int i の宣言はココ
			array[i] = scan.nextInt();
		} // int i の有効範囲は、ここまで

		System.out.println("back");

		for (int i = 0; i < array.length; i++) { // 上記のiのスコープ外なので、再度 iという名前で宣言できる
			System.out.println(array[i]);
		}

		scan.close();
	}

というわけで、スコープの問題かと以下のようにしたところ
警告になりませんでした。

public static void main(String[] args) {
	Scanner scan = new Scanner(System.in);
	{ // ここと
		int array[] = new int[5];
		int i;

		for (i = 0; i < array.length; i++) {
			array[i] = scan.nextInt();
		}

		System.out.println("back");

		for (i = 0; i < array.length; i++) {
			System.out.println(array[i]);
		}
	} // ここを中カッコで囲う。
	scan.close();
}

基本的にはスコープを広げるのはバグの元なので、いわゆる普通のforループの書き方をすれば
良い（警告がでない）と思います。

あと、eclipiseでの警告の当該行にランプのアイコンが出てると思いますが、
それをクリックするとクイックフィックスとして対応方法がでてきます。

ひとつあげると、アノテーション@SuppressWarnings("resource")を付与するとか・・・

繰り返しになりますが、おとなしくｉのスコープを最小限にした方が良いとは思います。

Comments

[tanatana32]

ありがとうございます！肝に銘じておきます！