JpaItemRepositoryV1의 findAll() 로직 질문 있습니다

2022-08-01T21:28:06.623Z

0

정확히 List<Object> param = new ArrayList<>(); 과 param.add(itemName) 이 부분이 JpaRepository 에서는 query.setParameter()로 파라미터값을 넘겨주고 있으니까 필요가 없는것 같아서요, 그건 아닐까요?

0

if (StringUtils.hasText(itemName)) {
    jpql += " i.itemName like concat('%',:itemName,'%')";
    param.add(itemName);
    andFlag = true;
}

TypedQuery<Item> query = em.createQuery(jpql, Item.class);
if (StringUtils.hasText(itemName)) {
    query.setParameter("itemName", itemName);
}

이 두개의 코드를 보신 것 같은데요,

두 코드 itemName이 존재할 때, 즉 검색을 이용할 때 동작합니다.

if (StringUtils.hasText(itemName)) 

이 조건을 만족하지 않으면

query.setParameter("itemName", itemName);

를 수행하지 않습니다.

TypedQuery<Item> query = em.createQuery(jpql, Item.class);

여기서 query는 jpql 변수로 만드는데, 추가적으로 

if (StringUtils.hasText(itemName)) 

이 조건을 만족해야 if문안의 query.setParameter가 수행됩니다.

0

    @Override
    public List<Item> findAll(ItemSearchCond cond) {
        String jpql = "select i from Item i";
        Integer maxPrice = cond.getMaxPrice();
        String itemName = cond.getItemName();
        if (StringUtils.hasText(itemName) || maxPrice != null) {
            jpql += " where";
        }
        boolean andFlag = false;
        List<Object> param = new ArrayList<>(); // 여기
        if (StringUtils.hasText(itemName)) {
            jpql += " i.itemName like concat('%',:itemName,'%')";
            param.add(itemName); // 여기
            andFlag = true;
        }
        if (maxPrice != null) {
            if (andFlag) {
                jpql += " and";
            }
            jpql += " i.price <= :maxPrice";
            param.add(maxPrice); // 여기
        }
        log.info("jpql={}", jpql);
        TypedQuery<Item> query = em.createQuery(jpql, Item.class);
        if (StringUtils.hasText(itemName)) {
            query.setParameter("itemName", itemName);
        }
        if (maxPrice != null) {
            query.setParameter("maxPrice", maxPrice);
        }
        return query.getResultList();
    }

음.. 그러면 저는 // 여기 라고 표시해놓은곳이 지워져도 된다고 생각하고, 잘못 들어간 부분이라고 생각하는데, 그건 아닌건가요??

0

주석으로 위치를 표기해주셔서 이해하였습니다.

param.add를 말씀해주셨는데, jpql 부분에만 신경을 썼네요..

네, 해당 코드는 지우셔도 상관없는 코드입니다.

JdbcTemplateItemRepositoryV1 코드를 복사해오는 과정에서 추가된 코드로 보이네요 ^^

JdbcTemplateItemRepositoryV1 여기서는 template에서 쿼리로 param을 넘기기 때문에 필요합니다.

  List<Object> param = new ArrayList<>();
        if (StringUtils.hasText(itemName)) {
            sql += " item_name like concat('%',?,'%')";
            param.add(itemName);
            andFlag = true;
        }

        if (maxPrice != null) {
            if (andFlag) {
                sql += " and";
            }
            sql += " price <= ?";
            param.add(maxPrice);
        }

        log.info("sql={}", sql);
        return template.query(sql, itemRowMapper(), param.toArray());

0

아 그렇군요 답변 감사합니다 ^^