devonfw#799: clean code improvements (devonfw#884)

cli/src/main/java/com/devonfw/tools/ide/io/FileAccessImpl.java

@@ -608,32 +608,35 @@ public void extractZip(Path file, Path targetDir) {
       for (Path root : fs.getRootDirectories()) {
         size += getFileSizeRecursive(root);
       }
-      try (IdeProgressBar bp = this.context.newProgressbarForExtracting(size)) {
-        PathCopyListener listener = (source, target, directory) -> {
-          if (directory) {
-            return;
-          }
-          if (!context.getSystemInfo().isWindows()) {
-            try {
-              Object attribute = Files.getAttribute(source, "zip:permissions");
-              if (attribute instanceof Set<?> permissionSet) {
-                Files.setPosixFilePermissions(target, (Set<PosixFilePermission>) permissionSet);
-              }
-            } catch (Exception e) {
-              context.error(e, "Failed to transfer zip permissions for {}", target);
-            }
-          }
-          bp.stepBy(getFileSize(target));
-        };
+      try (final IdeProgressBar progressBar = this.context.newProgressbarForExtracting(size)) {
         for (Path root : fs.getRootDirectories()) {
-          copy(root, targetDir, FileCopyMode.EXTRACT, listener);
+          copy(root, targetDir, FileCopyMode.EXTRACT, (s, t, d) -> onFileCopiedFromZip(s, t, d, progressBar));
         }
       }
     } catch (IOException e) {
       throw new IllegalStateException("Failed to extract " + file + " to " + targetDir, e);
     }
   }
 
+  @SuppressWarnings("unchecked")
+  private void onFileCopiedFromZip(Path source, Path target, boolean directory, IdeProgressBar progressBar) {
+
+    if (directory) {
+      return;
+    }
+    if (!context.getSystemInfo().isWindows()) {
+      try {
+        Object attribute = Files.getAttribute(source, "zip:permissions");
+        if (attribute instanceof Set<?> permissionSet) {
+          Files.setPosixFilePermissions(target, (Set<PosixFilePermission>) permissionSet);
+        }
+      } catch (Exception e) {
+        context.error(e, "Failed to transfer zip permissions for {}", target);
+      }
+    }
+    progressBar.stepBy(getFileSize(target));
+  }
+
   @Override
   public void extractTar(Path file, Path targetDir, TarCompression compression) {
 

cli/src/main/java/com/devonfw/tools/ide/io/PathCopyListener.java

@@ -13,7 +13,7 @@ public interface PathCopyListener {
   /**
    * An empty {@link PathCopyListener} instance doing nothing.
    */
-  static PathCopyListener NONE = (s, t, d) -> {
+  PathCopyListener NONE = (s, t, d) -> {
   };
 
   /**