projects
/
demos
/
kafka
/
training
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
GRÜN: Neue Erwartungen umgesetzt
[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
c0a4332
..
cbd500d
100644
(file)
--- a/
src/main/java/de/juplo/kafka/AdderBusinessLogic.java
+++ b/
src/main/java/de/juplo/kafka/AdderBusinessLogic.java
@@
-22,14
+22,6
@@
public class AdderBusinessLogic
}
}
- public synchronized void startSum(String user)
- {
- if (state.containsKey(user))
- throw new IllegalStateException("Sumation for " + user + " already in progress, state: " + state.get(user));
-
- state.put(user, 0l);
- }
-
public synchronized Optional<Long> getSum(String user)
{
return Optional.ofNullable(state.get(user));
public synchronized Optional<Long> getSum(String user)
{
return Optional.ofNullable(state.get(user));
@@
-37,21
+29,22
@@
public class AdderBusinessLogic
public synchronized void addToSum(String user, Integer value)
{
public synchronized void addToSum(String user, Integer value)
{
- if (!state.containsKey(user))
- throw new IllegalStateException("No sumation for " + user + " in progress");
if (value == null || value < 1)
throw new IllegalArgumentException("Not a positive number: " + value);
if (value == null || value < 1)
throw new IllegalArgumentException("Not a positive number: " + value);
- long result = state.get(user) + value;
- state.put(user, result);
+ long sum =
+ Optional
+ .ofNullable(state.get(user))
+ .orElse(0l);
+ state.put(user, sum + value);
}
}
- public synchronized Long
endSum
(String user)
+ public synchronized Long
calculate
(String user)
{
if (!state.containsKey(user))
throw new IllegalStateException("No sumation for " + user + " in progress");
{
if (!state.containsKey(user))
throw new IllegalStateException("No sumation for " + user + " in progress");
- return state.
get
(user);
+ return state.
remove
(user);
}
protected Map<String, Long> getState()
}
protected Map<String, Long> getState()