인프런 커뮤니티 질문&답변

안녕하세요 항상 양질의 강의 해주셔서 먼저 감사드립니다.

현재 코드 작성 중에 어떤게 더 나은 방법인지 궁금해서 문의드립니다.

카드 충전하는 서비스가 있다고 가정하면 컨트롤러를 통해 카드 충전 서비스를 호출할 예정입니다.

class Controller {
private final ChargeService chargeService;
 void charge() {
    chargeService.charge();
  }
}

위와 같이 충전 진행 시, 내부에서는 카드를 찾고 외부 API를 통해 충전을 진행 후 더티체킹을 통해 카드 상태를 업데이트 할 예정입니다.

class ChargeService {
  private final CardFindService cardFindService;
  private final ChargeHistoryService chargeHistoryService;
  void charge() {
    var card = cardFindService.findCard();
    cardApiService.charge(); // 외부 API 호출
    chargeHistoryService.save(); // 충전 내역 저장
    card.updateStatus(); // 카드 상태 변경
  }
}

여기서 궁금한게 충전을 진행하는 ChargeService 내에서 더티체킹을 통해 카드의 상태를 업데이트 하는게 좋을지, 아니면 컨트롤러단에서 ChargeService와 CardUpdateService를 각각 호출해주는게 좋을까요?

class Controller {
private final ChargeService chargeService;
private final CardUpdateService cardUpdateService;
 void charge() {
    chargeService.charge();
    cardUpdateService.updateCardStatus();
  }
}

OSIV를 꺼둔 상태에서 진행 시 chargeService 내 하나의 트랜잭션 내에서 충전 내역과 카드 상태까지 업데이트를 하는게 편리해 보이긴 하지만, chargeService 내에서 카드의 상태를 업데이트하는 역할까지 주어지는 것 같아 고민이 됩니다.

만약 서비스 2개를 각각 호출하게 되면 cardFind도 각 서비스 내에서 1회 씩 총 2번 호출되는 점도 마음에 걸립니다.

이와 같은 상황에서는 어떤 방식이 실무에 더 적합하고 좋을지 의견 부탁드리겠습니다.

감사합니다.