File.createTempFile メソッドで作成した一時ファイルの削除について

Question

プログラムの概要

File.createTempFile メソッドで作成した複数の一時ファイルをzip形式で出力した後、一時ファイルを削除するというプログラムです。

起きている問題

zipに含めるファイル数が1つの場合だと削除まで問題なく完了しますが、
zipに含めるファイル数が複数になると、最後の一時ファイル以外、削除ができません。

試したこと、確認したこと

実行させたいJavaのバージョンは1.6ですが、一度Javaの実行環境を1.7に変え、Files.delete メソッドで例外を取得したところ、他プロセスが使用中というエラーでした。
また、zip出力が無ければ複数ファイルの削除ができました。

実行環境

Javaのバージョンは1.6
Eclipseのバージョンは4.4です。

解決策があればご教授お願いします。

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStream;
import java.io.PrintWriter;
import java.util.zip.ZipEntry;
import java.util.zip.ZipOutputStream;

import org.apache.commons.io.IOUtils;

public class FileMain {
	private static final String DIR = "./tmp/";
	public static void main(String[] args) {
		File file = new File(DIR);
		File file2 = new File(DIR);
		File file3 = new File(DIR);
		try {
			file = File.createTempFile("prefix", ".suffix", file);

			// ファイルに書き込み
			FileWriter fw = new FileWriter(file, true);
			BufferedWriter bw = new BufferedWriter(fw);
			PrintWriter pw = new PrintWriter(bw);
			pw.write("こんにゃく");
			pw.flush();
			pw.close();

			file2 = File.createTempFile("prefix", ".suffix", file2);

			// ファイルに書き込み
			FileWriter fw2 = new FileWriter(file2, true);
			BufferedWriter bw2 = new BufferedWriter(fw2);
			PrintWriter pw2 = new PrintWriter(bw2);
			pw2.write("たまねぎ");
			pw2.flush();
			pw2.close();

			file3 = File.createTempFile("prefix", ".suffix", file3);

			// ファイルに書き込み
			FileWriter fw3 = new FileWriter(file3, true);
			BufferedWriter bw3 = new BufferedWriter(fw3);
			PrintWriter pw3 = new PrintWriter(bw3);
			pw3.write("にんじん");
			pw3.flush();
			pw3.close();


			File[] files = {file, file2, file3};
		    ZipOutputStream zos = null;
		    try {
		        zos = new ZipOutputStream(new BufferedOutputStream(new FileOutputStream(new File(DIR + "hoge.zip"))));
		        createZip(zos, files);
		    } catch (IOException e) {
		        e.printStackTrace();
		    } finally {
		        IOUtils.closeQuietly(zos);
		    }


		} catch (IOException e) {
			e.printStackTrace();
		} finally {
			// ファイルを削除
			if (file != null && file.exists()) {
				file.delete();
			}
			if (file2 != null && file2.exists()) {
				file2.delete();
			}
			if (file3 != null && file3.exists()) {
				file3.delete();
			}
		}
	}

	private static void createZip(ZipOutputStream zos, File[] files) throws IOException {
		byte[] buf = new byte[1024];
		InputStream is = null;
		try {
			for (File file : files) {
				ZipEntry entry = new ZipEntry(file.getName());
				zos.putNextEntry(entry);
				is = new BufferedInputStream(new FileInputStream(file));
				int len = 0;
				while ((len = is.read(buf)) != -1) {
					zos.write(buf, 0, len);
				}
			}
		} finally {
			IOUtils.closeQuietly(is);
		}
	}
}

Answer 1

createZipメソッドの中で それぞれのファイル読み込みした最後でclose()しておらず、ループの最後のInputStreamしか閉じていないのが理由です。

for (File file : files) {
  ...

  is = new BufferedInputStream(new FileInputStream(file));

  ....

  while ((len = is.read(buf)) != -1) {
    zos.write(buf, 0, len);
  }
  is.close();
}

Comments

[syachiku]

回答の通りで解決できました。ありがとうございます。