オブジェクト破棄コードの要否

Question

質問

下記のコードを作成しました（既存のExcelファイルを開き、値を読み取るプログラム）。
プログラム自体は正常に終了するのですが、気になったのがfinally句に良く記述する「オブジェクト破棄コード」です。例示しているソースコードのコメントにも記述していますが、

		} finally {
			// ＧＣ対象外オブジェクトは存在しないので、原則オブジェクト破棄コードは不要（下記のようなコードは不要）
			// if(excel != null ) excel = null;
			// if(sheet != null ) sheet = null;
		}

上記の認識であっておりますでしょうか？

C#ですと、リソースの破棄ーC＃によるプログラミング

メモリは .NET Framework の「ガーベジコレクション」機能が自動的に管理していて、 プログラマが明示的に破棄してやる必要はないのですが、 ファイルなどは「ガーベジコレクション」の管理対象外で、 明示的な破棄が必要です。

とのことですので、ファイル関係オブジェクトはその破棄コードが必要だと思いますが、Javaはそのような破棄コードは不要と考えてよろしいでしょうか？

＜ソースコードサンプル＞

		try{
			//Excelファイル オープン
			Workbook excel = WorkbookFactory.create(new File(inputPath.toString()));
			
			//シート オープン
			Sheet sheet = excel.getSheet(settings.getProperty("SheetName"));
			・
			・
			・

			}
		}catch(Exception e){
			System.err.println("ファイル読込み失敗：" + e.toString());
		} finally {
			// ＧＣ対象外オブジェクトは存在しないので、原則オブジェクト破棄コードは不要（下記のようなコードは不要）
			// if(excel != null ) excel = null;
			// if(sheet != null ) sheet = null;
		}

Answer 1

C# での言及

ファイルなどは「ガーベジコレクション」の管理対象外で、 明示的な破棄が必要です。

は「ファイル自体」であって「Workbook や Sheet に当たるオブジェクト」のことでは無いように思います。
java でも GC はファイル自体は削除しません。

---

try(FileInputStream fis = new FileInputStream(args[0])) {
  //fis から読んでなんたらかんたら
}
//この時点で fis は自動で close され、かつスコープを抜けるため fis は GC による回収対象となるので
// finally で fis.close() したり fis = null する必要は無い。(例外が発生しても同様。)

import java.io.File;

import org.apache.poi.ss.usermodel.*;

public class Main {
  public static void main(String[] args) {
    try(Workbook excel = WorkbookFactory.create(new File(args[0]))) { //Workbook は AutoCloseable
      Sheet sheet = excel.getSheet("シート1"); //Sheet は close 無し
      //(ry
    } catch(Exception e) {
      System.err.println("ファイル読込み失敗：" + e);
    }
    //excel は close され、 sheet 共々スコープを抜けて GC 対象へ
  }
}

Comments

[nakanak]

引用した部分が悪かったですね、、、
上記ＵＲＬ確認いただくとわかるのですが、「ファイル」ではなく「ファイルリソース（下のコードで言えば、reader）」という意味です。C# では、このreaderを明示的に破棄するコードが必要になりますが、Java では、このようなコードは不要で良いでしょうか？という、質問内容になります。

FileStream reader = new FileStream(args[0], FileMode.Open);

[jimbe]

url はマークダウンが何か間違っているようでアクセス出来ません。

「破棄」という言葉は close のことを指していて、 GC によるメモリの回収のことでは無いのではないように思います。

そのような reader であれば、今の java であれば try-resource 等で close した上でスコープを抜けることで GC に破棄させることが定石かと思いますので、 finally で null を代入することは無いと思います。

[nakanak]

URL修正しました。すみませんでした。
「try-with-resources文」というものですね（初めて知りました）。この構文を使うことで、自動でクローズ＆GC対象になるということで理解しました。自分の作っているコードも修正しようと思います。ありがとうございました。

Answer 2

複数の観点が混じっているので回答もつらつらと書いた感じになりますが。

---

メモリは .NET Framework の「ガーベジコレクション」機能が自動的に管理していて、 プログラマが明示的に破棄してやる必要はないのですが、 ファイルなどは「ガーベジコレクション」の管理対象外で、 明示的な破棄が必要です。

は正しいとは思えません。
「"ファイルを開いている状態"を管理しているオブジェクト」はガベージコレクションの対象になります。それが他の何からも参照されていなければそのオブジェクトは破棄されます。そして破棄される時にファイルを閉じるはずです。
そういった意味で"ファイルなどは「ガーベジコレクション」の管理対象外"は正しい表現には思えないです。

---

「WorkbookFactory.createで作られたオブジェクト」が内部に「"ファイルを開いている状態"を管理しているオブジェクト」を保持しているかどうかは分かりません。
仮に保持していたとしたらそのファイルを閉じる操作が用意されていてしかるべきです。finally節で呼ぶべきなのはその操作です。
逆に言うとそのようなファイルを閉じる操作が用意されていないなら、ファイルはすでに閉じられていて、閉じる必要がない可能性の方が高いです。

仮に保持していたとしてもそのオブジェクトは「WorkbookFactory.createで作られたオブジェクト」だけが参照していると期待されます。（複雑なキャッシュ構造などを持っているなどと考えることはないでしょう）
であれば「WorkbookFactory.createで作られたオブジェクト」がガベージコレクションの対象になった時に、「"ファイルを開いている状態"を管理しているオブジェクト」も一緒にガベージコレクションの対象になって、ファイルが閉じられるはずです。

---

		try{
			//Excelファイル オープン
			Workbook excel = WorkbookFactory.create(new File(inputPath.toString()));
			
			//シート オープン
			Sheet sheet = excel.getSheet(settings.getProperty("SheetName"));
			・
			・
			・

			}
		}catch(Exception e){
			System.err.println("ファイル読込み失敗：" + e.toString());
		} finally {
			// ＧＣ対象外オブジェクトは存在しないので、原則オブジェクト破棄コードは不要（下記のようなコードは不要）
			// if(excel != null ) excel = null;
			// if(sheet != null ) sheet = null;
		}

このコードの終了後に「WorkbookFactory.createで作られたオブジェクト」がガベージコレクションの対象になるかどうかは、このコードだけでは分かりません。
このコードの外側がどう書かれているかに依ります。
excel = nullやsheet = nullに意味があるかどうかはこのコードだけで論じることはできません。
（ただ、最初に書いた通り、一般論としては「ファイルを閉じる操作があるならそれを呼ぶべき。ないならファイルは閉じられていてそのような操作をする必要はないだろう」なので、そもそも論じる意味がある気はしません）

Comments

[nakanak]

「ファイルを閉じる操作」と「オブジェクト破棄の操作」を混同して質問してしまい、すみませんでした。
Workbookオブジェクトには close メソッドがあるので、try-catch-finally 構文なら、finally 句に close メソッドを記述するべきということですね。また、close メソッドを書かなかったとしても、try 句を抜けるときに、GC対象となり、GC で オブジェクト破棄されるときに自動的にclose されるということですね。

ただ、上の方が回答いただいているように、try - with -resource 文を使えば、close 文すらいらなくなるとのことですので、try - with -resource 文を使うようにします。ありがとうございました。

[quickquip]

> try 句を抜けるときに、GC対象となり、
細かい話ですが、GC対象になるのは try 文を抜けた時ではないです。これはメソッドのローカル変数扱いなのでGC対象になるのはメソッドを抜けた後です。
（普段は違いを気にすることでもないですけれど、GCの話をしている時なので書きました）
（考えてみたら最初からそう書けばよかったですね）

Answer 3

POIですか？
POIのWorkbookはjava.lang.AutoClosableなので、closeを呼ぶ必要があります。
try-with-resouce(C#でのusing相当)の出番でしょう。

Comments

[nakanak]

仰る通りですね（初めて知りました）。

[nakanak]

一番早く回答いただけたので、ベストアンサーにさせていただきました
（ただ、私の質問わかりにくく、他の方も悩ませてしまい、申し訳ありませんでした）