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

안녕하세요 토비님!!

강의를 완강하고 제 프로젝트를 리팩토링하면서 피드백받고싶은점이 생겨 질문글을 올립니다.

Repository Port를 기술에 종속시키지 않기 위해 Adapter에서 JPA, MyBatis, QueryDSL을 조합하는 구조를 선택했는데, 이 설계 방향이 적절한지 조언을 구하고 싶습니다.

현재 구조

  core/
  └── domain/
  └── application/
      └── required/          ← Port 인터페이스
          └── ~Repository

  adapter/
  └── persistence/
      ├── ~RepositoryAdapter ← Port 구현체
      ├── ~JpaRepository     ← Spring Data JPA
      └── ~MybatisMapper     ← Mybatis 매퍼 인터페이스

의존 관계

  [Service] → [Repository] ←impl― [RepositoryAdapter] → [JpaRepository]
                                                      → [MybatisMapper]
                                                      → [JPAQueryFactory]
    (core)       (core)              (adapter)              (adapter)

예시

@Repository
@RequiredArgsConstructor
public class ItemRepositoryAdapter implements ItemRepository {
    private final ItemJpaRepository itemJpaRepository;
    private final ItemMybatisMapper itemMybatisMapper;

    @Override
    public List<Item> findBySearchRequest(ItemSearchRequest request) {
        return itemMybatisMapper.findBySearchRequest(request)
                .stream()
                .map(Item::from)
                .toList();
    }

    @Override
    public List<Item> saveAll(List<Item> items) {
        return itemJpaRepository.saveAll(items);
    }
}

@Repository
@RequiredArgsConstructor
public class AuctionRepositoryAdapter implements AuctionRepository {
    private final JPAQueryFactory jpaQueryFactory;
    private final AuctionJpaRepository auctionJpaRepository;

    @Override
    public void deleteAllByRegionAndRealmId(RegionType region, Long realmId) {
        QAuction qAuction = QAuction.auction;

        BooleanBuilder filter = new BooleanBuilder();
        filter.and(qAuction.region.eq(region));

        if (realmId == null) {
            filter.and(qAuction.realmId.isNull());
        } else {
            filter.and(qAuction.realmId.eq(realmId));
        }

        jpaQueryFactory.delete(qAuction)
                .where(filter)
                .execute();
    }

    @Override
    public int saveAll(List<Auction> auctions) {
        if (auctions.isEmpty()) return 0;

        return auctionJpaRepository.saveAll(auctions).size();
    }
}

이 구조를 선택한 이유

동적 쿼리, 벌크 연산 등 JPA만으로 해결하기 어려운 케이스가 있어 MyBatis와 QueryDSL을 병행 사용하고 있습니다.

일반적인 방식인 Port 인터페이스가 Spring Data JPA Repository를 상속하는 구조를 채택하지 않은 이유:

• MyBatis나 QueryDSL 기반 구현체를 만들 수 없음

• CustomRepository 인터페이스를 별도로 만들어야 하는 복잡도 증가

현재 방식의 장점:

• Port 인터페이스가 순수 Java 인터페이스로 유지됨

• RepositoryAdapter에서 상황에 맞는 기술을 자유롭게 조합 가능

• 추가 인터페이스 없이 단순한 구조 유지

저는 현재 구조가 의존 관계도 외부에서 내부로 향하고 테스트도 쉬워서 괜찮다고 생각하는데 토비님의 생각도 듣고싶습니다!! 좋은 강의 감사합니다!!