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

32. Анкета на кредит наличными в Сбере

Сложная

Java · 55 строк

Клиент заполняет анкету на кредит до 5 млн ₽ под 19,9 % годовых. Контроллер проверяет возраст 21–70 лет и что аннуитетный платёж не больше половины дохода, отсекает повторную анкету за 30 дней и зовёт скоринг.

CreditForm содержит ФИО, серию и номер паспорта, дату рождения, доход, сумму и срок; toString() сгенерирован Lombok. Зарплатные клиенты поле дохода не заполняют — его подтягивают из зарплатного проекта. Скоринг отвечает 2–8 секунд, пул Hikari — 20 соединений. Отметьте строки с ошибками.

В работе: Онлайн-заявка на кредит в СберБанк Онлайн: пик — вечер пятницы и дни зарплаты.

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

Источник: наша задача на данных Сбера: кредитная анкета и скоринг. Условие пересказано своими словами.

  1. @RestController
  2. @RequestMapping("/api/credit")
  3. public class CreditApplicationController {
  4. private static final Logger log = LoggerFactory.getLogger(CreditApplicationController.class);
  5. private static final BigDecimal MAX_AMOUNT = new BigDecimal("5000000");
  6. private static final double ANNUAL_RATE = 0.199;
  8. private final DataSource dataSource;
  9. private final ScoringClient scoring;
  11. public CreditApplicationController(DataSource dataSource, ScoringClient scoring) {
  12. this.dataSource = dataSource;
  13. this.scoring = scoring;
  14. }
  16. @PostMapping("/apply")
  17. public ResponseEntity<String> apply(@RequestBody CreditForm form) throws SQLException {
  18. log.info("Новая анкета: {}", form);
  19. int age = LocalDate.now().getYear() - form.getBirthDate().getYear();
  20. if (age < 21 || age > 70) {
  21. return ResponseEntity.badRequest().body("Возраст вне диапазона 21–70 лет");
  22. }
  23. if (form.getAmount().compareTo(MAX_AMOUNT) > 0) {
  24. return ResponseEntity.badRequest().body("Сумма больше 5 млн ₽");
  25. }
  26. BigDecimal payment = monthlyPayment(form.getAmount(), form.getTermMonths());
  27. if (payment.compareTo(form.getIncome().multiply(new BigDecimal("0.5"))) > 0) {
  28. return ResponseEntity.ok("Отказ: платёж больше половины дохода");
  29. }
  31. Connection conn = dataSource.getConnection();
  32. Statement st = conn.createStatement();
  33. ResultSet rs = st.executeQuery("SELECT count(*) FROM applications WHERE passport = '"
  34. + form.getPassport() + "' AND created_at > now() - interval '30 days'");
  35. rs.next();
  36. if (rs.getInt(1) > 0) {
  37. return ResponseEntity.ok("Анкета уже на рассмотрении");
  38. }
  39. ScoreResult score = scoring.check(form);
  40. PreparedStatement ins = conn.prepareStatement(
  41. "INSERT INTO applications(passport, amount, term, score) VALUES (?, ?, ?, ?)");
  42. ins.setString(1, form.getPassport());
  43. ins.setBigDecimal(2, form.getAmount());
  44. ins.setInt(3, form.getTermMonths());
  45. ins.setInt(4, score.getValue());
  46. ins.executeUpdate();
  47. return ResponseEntity.ok(score.getValue() >= 600 ? "Предварительно одобрено" : "Отказ");
  48. }
  50. private BigDecimal monthlyPayment(BigDecimal amount, int months) {
  51. double r = ANNUAL_RATE;
  52. double k = r * Math.pow(1 + r, months) / (Math.pow(1 + r, months) - 1);
  53. return new BigDecimal(amount.doubleValue() * k);
  54. }
  55. }