간단한 spring-boot 코드리뷰 부탁드립니다!

Question

안녕하십니까!계좌에 관한 로직을 구현하고 있는데&#64;Override &#64;Transactional public void withdrawPoint(String userId, WithdrawRequestDto req) { Account account &#61; accountRepository.findById(userId).orElseThrow(UserNotFoundException::new);// 출금하려는 금액이 예치금&#43;수수료보다 많을 때 예외 if(account.getPoint()<(req.price()&#43;req.charge())) throw new ExceedsBalanceException(); account.setPoint(account.getPoint()-(req.price()&#43;req.charge())); accountRepository.save(account); // 거래내역에 추가 createWithdrawHistory(userId, req); }이런 로직이 있는데 accountRepository에서 계좌 정보를 찾아왔는데이때 찾아온 값으로 로직을 타는 중간에 다른 사람이 값 변경을 한다면 무결성이 깨지는 문제가 있을거라고 생각합니다.이럴땐 무결성 유지를 위해 어떻게 처리하는게 좋을까요?그리고 언뜻 듣기론 save가 update와 save기능 두개가 있기 때문에 위험하다고 들었습니다.(이것도 중간에 값이 변경될 시 위험하기 때문이라고 들었습니다)이 위험한걸 안하려면 어떤 방법으로 변경해야 하는건지 궁금합니다.

kenu,전독시 · Answer

&#64;Transactional이게 있는데 왜 깨진다고 생각하시나요?그런 경우의 테스트 코드를 만들어서 돌려 보시면 상황이 보일 겁니다.개발자는 코드로 증명해야죠.

Clickin · Answer

무결성이 깨질 일은 없고 계좌 잔액을 별도 컬럼으로 가져갔을 시 트랜잭션 격리 정도에 따라서 phantom read에 의한 계좌 잔액이 다를 수는 있겠죠그건 락을 걸거나 애초에 계좌 잔액을 이력의 sum으로 구현하거나 해볼 수 있습니다.

간단한 spring-boot 코드리뷰 부탁드립니다!

2개의 답변

Q&A 목록