From ca17e0a0829fdb054b2e65151d9448c4cf83549c Mon Sep 17 00:00:00 2001
From: Marcus van Elst <marcusvebo@gmail.com>
Date: Sun, 5 Apr 2026 21:13:09 +0200
Subject: [PATCH] Optimierung: PropertiesConfigurationPortAdapter intern
 entflochten

---
 .../PropertiesConfigurationPortAdapter.java   | 24 ++++++++++++-------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/pdf-umbenenner-adapter-out/src/main/java/de/gecheckt/pdf/umbenenner/adapter/out/configuration/PropertiesConfigurationPortAdapter.java b/pdf-umbenenner-adapter-out/src/main/java/de/gecheckt/pdf/umbenenner/adapter/out/configuration/PropertiesConfigurationPortAdapter.java
index fcbc695..bc8e3b8 100644
--- a/pdf-umbenenner-adapter-out/src/main/java/de/gecheckt/pdf/umbenenner/adapter/out/configuration/PropertiesConfigurationPortAdapter.java
+++ b/pdf-umbenenner-adapter-out/src/main/java/de/gecheckt/pdf/umbenenner/adapter/out/configuration/PropertiesConfigurationPortAdapter.java
@@ -78,26 +78,34 @@ public class PropertiesConfigurationPortAdapter implements ConfigurationPort {
 
     @Override
     public StartConfiguration loadConfiguration() {
+        Properties props = loadPropertiesFile();
+        String apiKey = getApiKey(props);
+        return buildStartConfiguration(props, apiKey);
+    }
+
+    private Properties loadPropertiesFile() {
         Properties props = new Properties();
         try {
-            // Check if file exists first to preserve FileNotFoundException behavior for tests
             if (!Files.exists(configFilePath)) {
                 throw new java.io.FileNotFoundException("Config file not found: " + configFilePath);
             }
-            // Read file content as string to avoid escape sequence interpretation issues
             String content = Files.readString(configFilePath, StandardCharsets.UTF_8);
-            // Escape backslashes to prevent Java Properties from interpreting them as escape sequences
-            // This is needed because Windows paths use backslashes (e.g., C:\temp\...)
-            // and Java Properties interprets \t as tab, \n as newline, etc.
-            String escapedContent = content.replace("\\", "\\\\");
+            String escapedContent = escapeBackslashes(content);
             props.load(new StringReader(escapedContent));
         } catch (IOException e) {
             throw new RuntimeException("Failed to load configuration from " + configFilePath, e);
         }
+        return props;
+    }
 
-        // Apply environment variable precedence for api.key
-        String apiKey = getApiKey(props);
+    private String escapeBackslashes(String content) {
+        // Escape backslashes to prevent Java Properties from interpreting them as escape sequences.
+        // This is needed because Windows paths use backslashes (e.g., C:\temp\...)
+        // and Java Properties interprets \t as tab, \n as newline, etc.
+        return content.replace("\\", "\\\\");
+    }
 
+    private StartConfiguration buildStartConfiguration(Properties props, String apiKey) {
         return new StartConfiguration(
                 Paths.get(getRequiredProperty(props, "source.folder")),
                 Paths.get(getRequiredProperty(props, "target.folder")),