From 82df080747170017e0fb86241fe68d73af1ce233 Mon Sep 17 00:00:00 2001
From: Philipp Riemer <ruderphilipp@gmail.com>
Date: Sat, 22 Jan 2022 16:17:52 +0100
Subject: [PATCH] Refactoring: extrahiere Logik aus HBCIInstitute::extractKeys
 in eigene Methode
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Hier werden stets -- unabhängig vom `keyType` -- aufwändig die Daten extrahiert,
diese jedoch nur in zwei Fällen (Wert `S` oder `V`) weiterverarbeitet. Somit
kann der gesamte Block in `if (keyType.equals("S") || keyType.equals("V"))`
gekapselt werden (vgl. Zeile 118 neu). Dies führt dazu, dass die Berechnungen
seltener ausgeführt werden müssen.

Weiterhin unterscheidet sich der Ablauf ausschließlich im letzten Schritt, wo
entschieden wird, ob `setInstSigKey` oder `setInstEncKey` verwendet werden
soll. Alles davor ist lediglich die notwendige Arbeit, um das richtige
Übergabeformat zu erzeugen. Sie kann somit in eine eigene Methode ausgelagert
werden.

Als Ergebnis bildet die Schleife nun klar die Businesslogik ab:
- Wenn Typ = null, dann direkt zum nächsten Datensatz.
- Wenn Typ = S oder V, dann setze Werte.
- Wenn Typ = anderer Wert, dann nichts tun.
---
 .../kapott/hbci/manager/HBCIInstitute.java    | 65 ++++++++++++-------
 1 file changed, 40 insertions(+), 25 deletions(-)

diff --git a/src/main/java/org/kapott/hbci/manager/HBCIInstitute.java b/src/main/java/org/kapott/hbci/manager/HBCIInstitute.java
index 9ed512f3..4d3f24d1 100644
--- a/src/main/java/org/kapott/hbci/manager/HBCIInstitute.java
+++ b/src/main/java/org/kapott/hbci/manager/HBCIInstitute.java
@@ -21,15 +21,19 @@
 
 package org.kapott.hbci.manager;
 
+import java.io.UnsupportedEncodingException;
 import java.math.BigInteger;
 import java.security.Key;
 import java.security.KeyFactory;
+import java.security.NoSuchAlgorithmException;
+import java.security.spec.InvalidKeySpecException;
 import java.security.spec.KeySpec;
 import java.security.spec.RSAPublicKeySpec;
 import java.util.Arrays;
 import java.util.Date;
 import java.util.Enumeration;
 import java.util.Properties;
+import java.util.Objects;
 
 import org.kapott.hbci.callback.HBCICallback;
 import org.kapott.hbci.comm.Comm;
@@ -112,30 +116,13 @@ void extractKeys(Properties result)
                 if (keyType==null)
                     continue;
 
-                String keyCountry=result.getProperty(head+".KeyName.KIK.country");
-                String keyBLZ=result.getProperty(head+".KeyName.KIK.blz");
-                String keyUserId=result.getProperty(head+".KeyName.userid");
-                String keyNum=result.getProperty(head+".KeyName.keynum");
-                String keyVersion=result.getProperty(head+".KeyName.keyversion");
-
-                HBCIUtils.log("found key "+
-                        keyCountry+"_"+keyBLZ+"_"+keyUserId+"_"+keyType+"_"+
-                        keyNum+"_"+keyVersion,
-                        HBCIUtils.LOG_DEBUG);
-
-                byte[] keyExponent=result.getProperty(head+".PubKey.exponent").getBytes(Comm.ENCODING);
-                byte[] keyModulus=result.getProperty(head+".PubKey.modulus").getBytes(Comm.ENCODING);
-
-                KeyFactory fac=KeyFactory.getInstance("RSA");
-                KeySpec spec=new RSAPublicKeySpec(new BigInteger(+1,keyModulus),
-                                                  new BigInteger(+1,keyExponent));
-                Key key=fac.generatePublic(spec);
-
-                if (keyType.equals("S")) {
-                    passport.setInstSigKey(new HBCIKey(keyCountry,keyBLZ,keyUserId,keyNum,keyVersion,key));
-                    foundChanges=true;
-                } else if (keyType.equals("V")) {
-                    passport.setInstEncKey(new HBCIKey(keyCountry,keyBLZ,keyUserId,keyNum,keyVersion,key));
+                if (keyType.equals("S") || keyType.equals("V")) {
+                    final HBCIKey hbciKey = getHbciKey(result, head, keyType);
+                    if (keyType.equals("S")) {
+                        passport.setInstSigKey(hbciKey);
+                    } else if (keyType.equals("V")) {
+                        passport.setInstEncKey(hbciKey);
+                    }
                     foundChanges=true;
                 }
             }
@@ -152,7 +139,35 @@ void extractKeys(Properties result)
             acknowledgeNewKeys();
         }
     }
-    
+
+    private static HBCIKey getHbciKey(final Properties result, final String head, final String type)
+                    throws UnsupportedEncodingException, NoSuchAlgorithmException, InvalidKeySpecException
+    {
+        Objects.requireNonNull(result);
+        Objects.requireNonNull(head);
+        Objects.requireNonNull(type);
+
+        String country = result.getProperty(head + ".KeyName.KIK.country");
+        String blz = result.getProperty(head + ".KeyName.KIK.blz");
+        String userId = result.getProperty(head + ".KeyName.userid");
+        String keyNum = result.getProperty(head + ".KeyName.keynum");
+        String keyVersion = result.getProperty(head + ".KeyName.keyversion");
+
+        HBCIUtils.log("found key "+
+                          country+"_"+blz+"_"+userId+"_"+type+"_"+
+                          keyNum+"_"+keyVersion,
+                      HBCIUtils.LOG_DEBUG);
+
+        byte[] keyExponent = result.getProperty(head + ".PubKey.exponent").getBytes(Comm.ENCODING);
+        byte[] keyModulus = result.getProperty(head + ".PubKey.modulus").getBytes(Comm.ENCODING);
+
+        KeyFactory fac = KeyFactory.getInstance("RSA");
+        KeySpec spec = new RSAPublicKeySpec(new BigInteger(+1, keyModulus), new BigInteger(+1, keyExponent));
+        Key key = fac.generatePublic(spec);
+
+        return new HBCIKey(country, blz, userId, keyNum, keyVersion, key);
+    }
+
     private void acknowledgeNewKeys()
     {
         StringBuffer answer=new StringBuffer();