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

안녕하세요 ! 이번에 기본편을 제외한(기본편은 책으로 했습니다 !) 강사님 강의를 완강하고

jpa활용 1편 애플리케이션을 스프링데이터jpa와 queyrdsl을 활용해서 혼자 만들어보고 있습니다.

그 중에 주문 목록 검색 시, 주문자나 주문상태로 검색하는것을 querydsl로 짜봤습니다.

package com.example.springjpa.repository;

import com.example.springjpa.domain.*;
import com.querydsl.core.BooleanBuilder;
import com.querydsl.core.types.dsl.BooleanExpression;
import com.querydsl.jpa.impl.JPAQueryFactory;
import org.springframework.stereotype.Repository;
import org.springframework.transaction.annotation.Transactional;
import org.springframework.util.StringUtils;

import javax.persistence.EntityManager;
import java.util.List;
import java.util.function.Supplier;

import static com.example.springjpa.domain.QDelivery.delivery;
import static com.example.springjpa.domain.QMember.member;
import static com.example.springjpa.domain.QOrder.order;

@Repository
@Transactional
public class SearchQueryRepository {

    private final EntityManager em;
    private final JPAQueryFactory queryFactory;

    public SearchQueryRepository(EntityManager em){
        this.em = em;
        queryFactory = new JPAQueryFactory(em);
    }

    public List<Order> findAllByEntity(OrderSearch orderSearch){
        List<Order> query = queryFactory.select(order)
                .from(order)
                .join(order.member, member).fetchJoin()
                .join(order.delivery, delivery).fetchJoin()
                .where(orderStatusEq(orderSearch.getOrderStatus()), membernameEq(orderSearch.getMemberName()))
                .fetch();

        return query;
    }

    private BooleanBuilder orderStatusEq(OrderStatus orderStatus) {
        return nullSafeBuilder(()->order.status.eq(orderStatus));
    }

    private BooleanBuilder membernameEq(String memberName) {
        return nullSafeBuilder(()->member.name.contains(memberName));
    }

    public static BooleanBuilder nullSafeBuilder(Supplier<BooleanExpression> f) {
        try {
            return new BooleanBuilder(f.get());
        } catch (Exception e) {
            return new BooleanBuilder();
        }
    }


}

본래 fetch join시 루트엔티티말고 조인하는 entity를 where로 검사하면 데이터 부정합이 생길수 있다고 하셨는데요

위의 경우 order와 member, delivery는 외래키가 있으니, 부정합이 일어날것 같지 않은데, 혹시 저게 옳은 코드일까요?

테스트를 돌려봤을때는 이상이 없습니다만 ㅠㅠ 

아무리 생각해도 fetch join되는 테이블에 조건을 거는게 꺼림직해서.... 고민이 됩니다.

그냥 join으로 조회한 다음 dto에서 order.getDelivery().getAddress() 등을하는게 더 옳은 선택일까요?