From c5e223476398855dfd89f85d05301a2f1a3c9333 Mon Sep 17 00:00:00 2001 From: XenoAmess Date: Mon, 24 Aug 2020 16:50:41 +0800 Subject: [PATCH] use try with resource --- .../java/net/sourceforge/pmd/cpd/FileReporterTest.java | 8 +------- .../java/net/sourceforge/pmd/it/ZipFileExtractor.java | 10 ++-------- 2 files changed, 3 insertions(+), 15 deletions(-) diff --git a/pmd-core/src/test/java/net/sourceforge/pmd/cpd/FileReporterTest.java b/pmd-core/src/test/java/net/sourceforge/pmd/cpd/FileReporterTest.java index 97c7aade2b..9420578b58 100644 --- a/pmd-core/src/test/java/net/sourceforge/pmd/cpd/FileReporterTest.java +++ b/pmd-core/src/test/java/net/sourceforge/pmd/cpd/FileReporterTest.java @@ -54,9 +54,7 @@ public class FileReporterTest { } private String readFile(File file) throws IOException { - BufferedReader reader = null; - try { - reader = new BufferedReader(new FileReader(file)); + try (BufferedReader reader = new BufferedReader(new FileReader(file))) { StringBuffer buffer = new StringBuffer(); String line = reader.readLine(); while (line != null) { @@ -67,10 +65,6 @@ public class FileReporterTest { } } return buffer.toString(); - } finally { - if (reader != null) { - reader.close(); - } } } diff --git a/pmd-dist/src/test/java/net/sourceforge/pmd/it/ZipFileExtractor.java b/pmd-dist/src/test/java/net/sourceforge/pmd/it/ZipFileExtractor.java index d31051fd04..27a8b325d6 100644 --- a/pmd-dist/src/test/java/net/sourceforge/pmd/it/ZipFileExtractor.java +++ b/pmd-dist/src/test/java/net/sourceforge/pmd/it/ZipFileExtractor.java @@ -39,8 +39,7 @@ public class ZipFileExtractor { * @throws Exception if any error happens during extraction */ public static void extractZipFile(Path zipPath, Path tempDir) throws Exception { - ZipFile zip = new ZipFile(zipPath.toFile()); - try { + try (ZipFile zip = new ZipFile(zipPath.toFile())) { Enumeration entries = zip.getEntries(); while (entries.hasMoreElements()) { ZipArchiveEntry entry = entries.nextElement(); @@ -57,8 +56,6 @@ public class ZipFileExtractor { } } } - } finally { - zip.close(); } } @@ -70,15 +67,12 @@ public class ZipFileExtractor { */ public static List readZipFile(Path zipPath) throws Exception { List result = new ArrayList<>(); - ZipFile zip = new ZipFile(zipPath.toFile()); - try { + try (ZipFile zip = new ZipFile(zipPath.toFile())) { Enumeration entries = zip.getEntries(); while (entries.hasMoreElements()) { ZipArchiveEntry entry = entries.nextElement(); result.add(entry.getName()); } - } finally { - zip.close(); } return result; }