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

안녕하세요. 영한님

20만 수강생 진심으로 축하드립니다.^^🎉

프로젝트 도중에 막혀서 강의를 👀복습하던 중 질문드립니다..!

(영한님 강의 덕분에 저도 이러한 고민을 하게되다니... 감사의 말씀 드립니다.😊) 

페이징이 가능하고 item.name으로 order를 조회하려고 하면 어떠한 방식으로 코드를 작성 해야할까요?

일단 제가 생각한 방법은 다음과 같습니다.

V3.1 적용

요구사항은 회원(member), 결제(payment), 배송(delievery), 주문 상품(orderItem)을 포함해서 페이징이 되도록 주문(order)을 조회하고, 조건으로 상품 이름(item.name)으로 조회가 가능해야 하는 것 입니다.

orderItem은 일대다 관계 이기 때문에 페이징이 불가합니다.

그래서 V3.1의 방법인 hibernate.default_batch_fetch_size 을 이용하여 IN 쿼리로 orderItem을 조회하려고 합니다.

이렇게 하면 페이징이 가능하도록 order는 조회할수는 있지만, item.name을 조건으로 조회 할 수는 없습니다.

V5 적용

그래서 V5에서 알려주신 방법을 사용했습니다.

1. order와 orderItem을 분리한다.

2. order을 이용하여 orderItem을 조회한다.(추가로 item.name 조건도 넣는다.)

3. 조회한 결과를 하나의 List으로 만든다.

4. List을 다시 Page로 변환한다.
   

public Page<AdminOrderListQueryDto> findOrdersByAdmin(Pageable pageable, OrderSearchCondition condition) {

    // 주문 전체 조회
    List<AdminOrderListQueryDto> content = queryFactory
            .select(new QAdminOrderListQueryDto(order.id,
                    order.status.stringValue(),
                    order.safeKingPayment.amount,
                    order.createDate,
                    order.merchantUid,
                    new QAdminOrderListPaymentQueryDto(order.safeKingPayment.status.stringValue()),
                    new QAdminOrderListMemberQueryDto(order.member.name),
                    new QAdminOrderListDeliveryQueryDto(order.delivery.receiver, order.delivery.status.stringValue()))
            )
            .from(order)
            .leftJoin(order.safeKingPayment, safekingPayment)
            .leftJoin(order.delivery, delivery)
            .leftJoin(order.member, member)
            .where(
                    orderBetweenDate(condition.getFromDate(), condition.getToDate()),
                    deliveryStatusEq(condition.getDeliveryStatus()),
                    paymentStatusEq(condition.getPaymentStatus())
            )
            .orderBy(order.createDate.desc())
            .fetch();

    // 주문 아이디 저장
    List<Long> orderIds = content.stream()
            .map(o -> o.getId())
            .collect(Collectors.toList());

    // 상품명으로 검색 조건
    Map<Long, List<AdminOrderListOrderItemQueryDto>> orderItemMap = findOrderItemMap(orderIds, condition.getKeyword());

    // 주문객체에 주문 상품컬렉션 저장
    content.forEach(o -> o.setOrderItems(orderItemMap.get(o.getId())));

    // 주문상품이 null이 아닌 컬렌션으로 구성
    List<AdminOrderListQueryDto> resultContent = content.stream()
            .filter(o -> o.getOrderItems() != null)
            .collect(Collectors.toList());

    // List를 Page로 변환
    PageRequest pageRequest = PageRequest.of(pageable.getPageNumber(), pageable.getPageSize());
    int start = (int) pageRequest.getOffset();
    int end = Math.min(start + pageRequest.getPageSize(), resultContent.size());

    if(start > end) {
        throw new OrderException("데이터가 없습니다. 관리자에게 문의하세요.");
    }

    return new PageImpl<>(resultContent.subList(start, end), pageRequest, resultContent.size());
}

private Map<Long, List<AdminOrderListOrderItemQueryDto>> findOrderItemMap(List<Long> orderIds, String keyword) {
    // 주문 상품 검색(item.name 조건 포함)
    List<AdminOrderListOrderItemQueryDto> orderItems = queryFactory.select(new QAdminOrderListOrderItemQueryDto(orderItem.order.id, orderItem.id, orderItem.item.name))
            .from(orderItem)
            .leftJoin(orderItem.item, item)
            .where(
                    orderItem.order.id.in(orderIds),
                    keywordContains(keyword)
            )
            .fetch();

    // Map 으로 변환
    Map<Long, List<AdminOrderListOrderItemQueryDto>> orderItemMap = orderItems.stream()
            .collect(Collectors.groupingBy(orderItemQueryDto -> orderItemQueryDto.getOrderId()));

    return orderItemMap;
}

 // 아이템이름 포함 조건
private BooleanExpression keywordContains(String keyword) {
        return hasText(keyword) ? item.name.contains(keyword) : null;
}

이러한 방식을 사용하는게 맞을까요? 🤔

감사합니다.^^