public void upgradeLevels() throws SQLException {
// 트랜잭션 경계 설정
  TransactionStatus status = this.transactionManager.getTransaction(
      new DefaultTransactionDefinition());

  try {
  // 비즈니스 로직
    List<User> users = userDao.getAll();
    for (User user : users) {
      if (policy.canUpgradeLevel(user)) {
        upgradeLevel(user);
      }
    }
    
// 트랜잭션 경계 설정
    this.transactionManager.commit(status);
  } catch (RuntimeException e) {
    this.transactionManager.rollback(status);
    throw e;
  }
}

• 트랜잭션 경계 설정 코드와 비즈니스 로직 코드가 복잡하게 얽힌 듯 하지만 두 종류의 코드가 구분되어 있다.
  • 비즈니스 로직 코드를 사이에 두고 트랜잭션 시작과 종료를 담당하는 코드가 앞뒤에 위치
• 트랜잭션 경계 설정 코드와 비즈니스 로직 코드 간 서로 주고받는 정보가 없다.
  1. 비즈니스 로직 코드에서 직접 DB 를 사용하지 않는다.
  2. 트랜잭션 준비 과정에서 만들어진 DB 커넥션 정보 등을 직접 참조할 필요가 없다.
  • 해당 메서드에서 시작된 트랜잭션 정보는 트랜잭션 동기화 방법을 통해 DAO 가 알아서 활용

⇒ 비즈니스 로직과 트랜잭션 경계 설정 코드는 완벽하게 독립적인 코드

비즈니스 로직과 트랜잭션 경게 설정의 분리

public void upgradeLevels() throws SQLException {
    //트랜잭션 경계 설정
    TransactionStatus status = this.transactionManager.getTransaction(
        new DefaultTransactionDefinition());

    try {
      upgradeLevelsInternal();
      //트랜잭션 경계 설정
      this.transactionManager.commit(status);
    } catch (RuntimeException e) {
      this.transactionManager.rollback(status);
      throw e;
    }
  }
  
  private void upgradeLevelsInternal(){
    // 비즈니스 로직
    List<User> users = userDao.getAll();
    for (User user : users) {
      if (policy.canUpgradeLevel(user)) {
        upgradeLevel(user);
      }
    }
  }

변경점

• 코드를 분리하여 코드 가독성 향상
• 순수하게 사용자 레벨 업그레이드를 담당하는 비즈니스 로직만 독립적인 메서드에 남긴다.
  • 이해하기 편하고 수정 시 부담도 없다.
  • 실수로 트랜잭션 코드를 건드릴 염려도 없어졌다.

리팩토링 후 테스트! 아 맞다 테스트! 아 맞다 테스트! 아 맞다 테스트! 아 맞다 테스트!