트랜잭션이 보장되고 처리되어야만 하는 N개의 작업이 있다면 별도의 서비스로 추출하라는 말이 헷갈립니다.

안녕하세요, 항상 강의 잘 보고 있습니다. 강의 듣기 전에는 모든 Layer에 단위테스트를 작성하느라 테스트가 싫었는데 덕분에 테스트가 재미있어졌습니다. 감사합니다 😀

아래 질문에 대한 답변을 읽다가 알쏭달쏭한 부분이 생겼습니다.

해당 질문(긴 작업일 경우 트랜잭션을 걸지 말아도 된다는 점이 이해가 잘 안갑니다)에서 강사님께서는 "메일 전송 이후에 해야하는 작업들이 1가지가 아니라 트랜잭션이 보장되고 처리되어야만 하는 N개의 작업이었다면, 별도의 서비스로 추출해 트랜잭션을 적용하는 방법을 생각해볼 수 있습니다." 라는 말씀을 하셨습니다.

그렇다면 아래 MailService 클래스를 예시로 생각했을 때, 아래와 같이 별도의 클래스로 분리하라는 말씀이 맞으실까요?

// 변경 전 
@RequiredArgsConstructor
@Service
public class MailService {

  private final MailSendClient mailSendClient;
  private final MailSendHistoryRepository mailSendHistoryRepository;

  public boolean sendMail(String fromEmail, String toEmail, String subject, String content) {

    boolean result = mailSendClient.sendEmail(fromEmail, toEmail, subject, content);
    if (result) {
      mailSendHistoryRepository.save(MailSendHistory.builder()
          .fromEmail(fromEmail)
          .toEmail(toEmail)
          .subject(subject)
          .content(content)
          .build()
      );
      return true;
    }

    return false;
  }
}

아래 코드에서 변경된 부분은 다음과 같습니다.

• MailSendHistoryService 클래스에 대한 의존관계가 생겼습니다.

• MailService의 sendEmail 메서드에는 readOnly 트랜잭션이 걸렸습니다.

• MailSendHistoryService의 추가적인_일을_하다 메서드에는 트랜잭션이 걸렸습니다.

// 변경 후(내 생각)
@RequiredArgsConstructor
@Service
public class MailService {

  private final MailSendClient mailSendClient;
  private final MailSendHistoryService mailSendHistoryService;

  @Transactional(readOnly = true)
  public boolean sendMail(String fromEmail, String toEmail, String subject, String content) {

    boolean result = mailSendClient.sendEmail(fromEmail, toEmail, subject, content);
    if (result) {
      mailSendHistoryService.추가적인_일을_하다();
      return true;
    }

    return false;
  }
}

@RequiredArgsConstructor
@Service
public class MailSendHistoryService {

  private final MailSendHistoryRepository mailSendHistoryRepository;

  @Transactional
  public void 추가적인_일을_하다() {
    ... DB CUD 작업 ...
  }
}