close()をfinallyで囲む必要が無い場合

Question

スッキリ分かるJava入門 実践編9章の章末問題の解答に記載されているコードですが、
ファイルを閉じるtry-catch文がfinallyで囲まれていません。

closeは必ずfinallyで閉じなければならないと同シリーズ入門編で書かれていたのですが、
なぜこのコードの場合はfinallyをつける必要がないのでしょうか？

import java.io.*;
import java.util.zip.GZIPOutputStream.*;

public class Main {
	public static void main(String[] args) {
		String inFile = args[0];
		String outFile = args[1];
		FileInputStream fis = null;
		GZIPOutputStream gzos = null;
		
		try {
			fis = new FileInputStream(inFile);
			FileOutputStream fos = new FileOutputStream(outFile);
			BufferedOutputStream bos = new BufferedOutputStream(fos);
			gzos = new GZIPOutputStream(bos);
			int i = fis.read();
			while(i != -1) {
				gzos.write(i);
				i = fis.read();
			}
			gzos.flush();
			gzos.close();
			fis.close();
	} catch (IOException e) {
	System.out.println("ファイル操作に失敗しました"); 
	try {
		if(fis != null) {
			fis.close();
		}
		if(gzos != null) {
			gzos.close();
		}
	} catch (IOException ee) { }
    }
  }
}

Answer 1

かなり古いコードなので理解する必要はありません。
以下のような書き方で書かれている参考書に切り替えてください。
try-with-resources 文

Comments

[etrisotoko777]

ご丁寧にありがとうございます。
参考にさせていただきます

Answer 2

closeは必ずfinallyで閉じなければならない

そんなことはありません。要は「例外が起きても起きなくてもきちんとcloseする」を実現できればいいので、このコード例にありますように、catchでも閉じればfinallyは不要です。

とはいえ、tryの最後とcatchの両側に同じことを書くのは面倒ですし、将来のメンテ性も悪化させますので、実用的にはあまり適当ではない形です。

Comments

[maisumakun]

なお、Java 7ではtry-with-resourcesという構文が登場しています。こちらではcloseを自分で書く必要すらなくなっていますので、使える状況であればこちらを使うことをおすすめします。

https://docs.oracle.com/javase/jp/7/technotes/guides/language/try-with-resources.html

[etrisotoko777]

ご丁寧にありがとうございます。
参考にさせていただきます