@Transactional 코드 개선 방법?

Question

다음과 같은 형태의 비즈니스 로직이 있다고 가정 할경우 제 생각으로는 주문이 생성되고재고를 차감하고쿠폰을 발급 하던중 -> 오류 발생@Transactional 처리로 인해 전부 롤백 처리이런식의 로직이 맞다고 생각합니다.@Transactional public void processOrder(OrderDto dto) { // 주문 생성 orderMapper.insertOrder(dto); // 재고 차감 inventoryMapper.decreaseStock(dto.getItemId(), dto.getQuantity()); // 쿠폰 발급 로직 if (dto.getUserId() != null && dto.getTotalPrice() > 100000) { couponMapper.issueCoupon(dto.getUserId()); } // 포인트 적립 pointMapper.addPoints(dto.getUserId(), calculatePoint(dto.getTotalPrice())); // 로그 기록 logMapper.insertLog("주문 처리 완료"); } 근데 위 코드는 좀 간결한 편입니다.다만 복잡한 비즈니스 로직이 들어가 있는 메소드(대략 100줄 가량) `@Transactional` 을 붙이는게 맞는지 좀 의문입니다. 오히려 로직을 좀 잘게 쪼개서 해당 부분에 만 붙이는게 옳은 방법이 아닐까.. @Transactional 을 현명하게 쓸려면 어떻게 해야할까요?

왕감쟈 · Answer

트랜잭션은 필요한 만큼만 유지되는 게 좋죠메서드를 분리해서 거기에 Transactional을 붙이거나 (자기호출 주의)트랜잭션 매니저를 사용하여 직접 제어하면 됩니다.

chiho · Answer

1~3번까지 로직이 데이터 변경이 있는건데 다 묶어야하져

cesium · Answer

Transactional중에 Required, Requies_new, nested ..등 찾아 보면 여러가지 효율적으로 사용하는 방법을 찾을수 있을꺼에요.

javaFactory · Answer

사실 Transactional 에 대한 문제라기 보다는 그거는 논리를 함수나 클래스로 어떻게 묶을까가 중요한 것 같아요 "관리차원에서 코드를 어떻게 클린하게 유지할까" 가 더 옳은 질문 같아요.왜냐면 결제 로직뿐만이 아니라 다른 로직에서도 다 같이 적용되는 문제기 때문입니다.가령```javavoid doSomeThing(){ ... 100 lines of code that setup .... ... 100 lines of code that check validation .... ... 100 lines of code that execute db query ....}```일케 되어 있는데 미래에, 그때 그 셋업에서 validation 로직 추가해 달라하면 틀린 그림 찾기하다가 망하거든요.그래서 이것을```void doProcess(){ doSetup(); doValidate(something); var processCmd = ProcessMapper.createProcessCmd(something); jobWriter.write(processCmd);}```이런 식으로 래핑하는 테크닉이 생활화되어 있으면 좋습니덩

개발잡부 · Answer

하나라도 오류나면 전체가 롤백되어야 하는 상황이라면 Transactional 하나만 붙이면 되고,그게 아니라면 필요한곳에 Transactional을 REQUIRES_NEW로 붙이고 트렌잭션을 따로 관리하면 됩니다.다만 같은 클래스내에서 메소드 호출은 적용이 안되므로, 클래스 자체를 따로 빼놔서 DI 시키고 호출해야합니다

wharup · Answer

말씀하신 방법대로 하는 게 정석입니다. 그런 것들 중에 Transaction을 유지하는 시간이 너무 길어져서 문제다 싶으면 중간에 커밋할 수 있겠죠.

@Transactional 코드 개선 방법?

6개의 답변

Q&A 목록