commit 30da70772ce91d16420f7510f07da968a111b6b5
author Gary Gregory <garydgregory@gmail.com> Sat Apr 27 09:41:42 2024 -0400
committer Gary Gregory <garydgregory@gmail.com> Sat Apr 27 09:41:42 2024 -0400
tree 9ed786768335b506535e271dd22c20af67408694
parent bfa837420dbbf5acafae10ade60504d8a4b63905

Refactor duplicate code

src/main/java/org/apache/commons/compress/archivers/ar/ArArchiveOutputStream.java

diff --git a/src/main/java/org/apache/commons/compress/archivers/ar/ArArchiveOutputStream.java b/src/main/java/org/apache/commons/compress/archivers/ar/ArArchiveOutputStream.java
index a0c8779..fe0bf6c 100644
--- a/src/main/java/org/apache/commons/compress/archivers/ar/ArArchiveOutputStream.java
+++ b/src/main/java/org/apache/commons/compress/archivers/ar/ArArchiveOutputStream.java
@@ -60,6 +60,15 @@
     }
 
     /**
+     * @throws IOException
+     */
+    private void checkFinished() throws IOException {
+        if (finished) {
+            throw new IOException("Stream has already been finished");
+        }
+    }
+
+    /**
      * Calls finish if necessary, and then closes the OutputStream
      */
     @Override
@@ -76,9 +85,7 @@
 
     @Override
     public void closeArchiveEntry() throws IOException {
-        if (finished) {
-            throw new IOException("Stream has already been finished");
-        }
+        checkFinished();
         if (prevEntry == null || !haveUnclosedEntry) {
             throw new IOException("No current entry to close");
         }
@@ -90,9 +97,7 @@
 
     @Override
     public ArArchiveEntry createArchiveEntry(final File inputFile, final String entryName) throws IOException {
-        if (finished) {
-            throw new IOException("Stream has already been finished");
-        }
+        checkFinished();
         return new ArArchiveEntry(inputFile, entryName);
     }
 
@@ -103,9 +108,7 @@
      */
     @Override
     public ArArchiveEntry createArchiveEntry(final Path inputPath, final String entryName, final LinkOption... options) throws IOException {
-        if (finished) {
-            throw new IOException("Stream has already been finished");
-        }
+        checkFinished();
         return new ArArchiveEntry(inputPath, entryName, options);
     }
 
@@ -132,9 +135,7 @@
 
     @Override
     public void putArchiveEntry(final ArArchiveEntry entry) throws IOException {
-        if (finished) {
-            throw new IOException("Stream has already been finished");
-        }
+        checkFinished();
         if (prevEntry == null) {
             writeArchiveHeader();
         } else {