для такого мультивалютного счета, я бы лучше написал такой код (беглый рефакторинг)
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
public class Account {
ConcurrentMap<String, AtomicLong> currencyBallance = new ConcurrentHashMap<String, AtomicLong>();
public void transfer(String currency, int value) {
AtomicLong currencyValue = getPrevCurrencyBalance(currency);
long oldValue, newValue;
do {
oldValue = currencyValue.get();
newValue = oldValue + value;
if (newValue < 0) {
throw new IllegalArgumentException();
}
} while (!currencyValue.compareAndSet(oldValue, newValue));
}
private AtomicLong getPrevCurrencyBalance(String currency){
if(currencyBallance.get(currency) == null)
currencyBallance.putIfAbsent(currency, new AtomicLong(0));
return currencyBallance.get(currency);
}
}
Плюс в том, что изменение баланса идет исключительно в одном месте.
getPrevCurrencyBalance выдает предыдущий баланс мультивалютного счета в разрезе переданной валюты. Если баланс по этой валюте не был инициализирован - инициализируется 0;
Минус - если первоначалка отрицательная, мы напрасно инициализируем остаток. Если он недопустим - то в метод getPrevCurrencyBalance протягиваем сумму..., но на мой взляд это лишнее.
по поводу исключения. IllegalArgumentException вам тут не подходит, поскольку исключение - явное нарушение бизнеслогики. Следовательно нужно создать свое исключение, причем проверяемое.
