ROT: Zur Summe soll die Zahl ausgegeben werden - Logik + Test angepasst

[demos/kafka/training] / src / main / java / de / juplo / kafka / AdderBusinessLogic.java

diff --git a/src/main/java/de/juplo/kafka/AdderBusinessLogic.java b/src/main/java/de/juplo/kafka/AdderBusinessLogic.java
index 1f8ebc2..64fdb8c 100644 (file)
--- a/src/main/java/de/juplo/kafka/AdderBusinessLogic.java
+++ b/src/main/java/de/juplo/kafka/AdderBusinessLogic.java
@@ -22,24 +22,33 @@ public class AdderBusinessLogic
   }
 
 
-  public synchronized void startSum(String user)
+  public synchronized Optional<Long> getSum(String user)
   {
-    if (state.containsKey(user))
-      throw new IllegalStateException("Sumation for " + user + " already in progress, state: " + state.get(user));
-
-    state.put(user, 0l);
+    return Optional.ofNullable(state.get(user));
   }
 
-  public synchronized Optional<Long> getSum(String user)
+  public synchronized void addToSum(String user, Integer value)
   {
-    return Optional.ofNullable(state.get(user));
+    if (value == null || value < 1)
+      throw new IllegalArgumentException("Not a positive number: " + value);
+
+    long sum =
+        Optional
+            .ofNullable(state.get(user))
+            .orElse(0l);
+    state.put(user, sum + value);
   }
 
-  public synchronized Long endSum(String user)
+  public synchronized AdderResult calculate(String user)
   {
     if (!state.containsKey(user))
       throw new IllegalStateException("No sumation for " + user + " in progress");
 
-    return state.get(user);
+    return new AdderResult(66, state.remove(user));
+  }
+
+  protected Map<String, Long> getState()
+  {
+    return state;
   }
 }