Войти Регистрация
Код-ревью

1. Перевод между счетами с комиссией

Средняя

Java · 32 строк

Сервис переводит деньги между счетами клиентов и берёт комиссию 1,5 % с отправителя. Код пришёл на ревью перед релизом. Отметьте строки с ошибками.

Сервис крутится в трёх подах за балансировщиком, AccountRepository каждый раз читает счёт из базы и возвращает новый объект.

В работе: Переводы по номеру счёта в мобильном банке: десятки тысяч операций в минуту в пятницу вечером.

Как проверяем Нажмите на строку с ошибкой и коротко напишите, что не так. Баг засчитан, если отмечена хотя бы одна его строка. Зачёт — найдено не меньше 60 % багов с учётом серьёзности (критичный весит 3, обычный 2, мелкий 1), а лишних отметок не больше, чем багов в коде. Пустые строки, скобки и комментарии лишними не считаем. Ответ один на подход: после него показываем разбор.
деньгитранзакцииблокировкиSQL-инъекция

Источник: Т‑Банк: секция Java, код-ревью с доработкой под бизнес-задачу. Условие пересказано своими словами.

Давали на собеседовании? Ответов пока мало — ваш поможет.
  1. @Service
  2. public class TransferService {
  3. private static final Logger log = LoggerFactory.getLogger(TransferService.class);
  4. private static final double FEE_RATE = 0.015;
  6. private final JdbcTemplate jdbc;
  7. private final AccountRepository accounts;
  9. public TransferService(JdbcTemplate jdbc, AccountRepository accounts) {
  10. this.jdbc = jdbc;
  11. this.accounts = accounts;
  12. }
  14. public TransferResult transfer(String fromId, String toId, double amount, String cardNumber) {
  15. log.info("Перевод {} -> {}, сумма {}, карта {}", fromId, toId, amount, cardNumber);
  16. Account from = accounts.findById(fromId);
  17. Account to = accounts.findById(toId);
  18. double fee = amount * FEE_RATE;
  19. if (from.getBalance() < amount + fee) {
  20. return TransferResult.rejected("Недостаточно средств");
  21. }
  22. synchronized (this) {
  23. from.setBalance(from.getBalance() - amount - fee);
  24. to.setBalance(to.getBalance() + amount);
  25. }
  26. accounts.save(from);
  27. accounts.save(to);
  28. jdbc.update("INSERT INTO transfers(from_id, to_id, amount, fee) VALUES ('"
  29. + fromId + "', '" + toId + "', " + amount + ", " + fee + ")");
  30. return TransferResult.ok(fee);
  31. }
  32. }