프로젝트 | 프로젝트 중 해결/고민한 문제들 정리! (feat. JPQL, Querydsl)

 

서론

음.. 크게 오류가 나진 않았고, 이전 프로젝트에 비해 해결하는 데에 몇 시간 걸리는 일도 없어서 무척 단순하고 간단한 것들일 수도 있다.

하지만 나는 새롭게 알게된 점들이 참 많다........

 

 

1. 성능 개선 

연관 관계 매핑? 메서드 체이닝으로 조회하던 부분을 JPQL로 조회하면서 select 문이 2개에서 1개로 줄었다.
왠지 성능 개선은 엄청난 차이로 개선해야하는 느낌인 듯하여..     이런 걸 성능 개선이라고 표현해도 될지 모르겠지만..? 

— 초기 코드

// Service
UUID ownerUser = menu.getRestaurant().getOwner().getUserId();

 

실행되는 select 문들

Hibernate: 
    select
        m1_0.rm_id,
        m1_0.created_at,
        m1_0.created_by,
        m1_0.deleted_at,
        m1_0.deleted_by,
        m1_0.is_public,
        m1_0.modified_at,
        m1_0.modified_by,
        m1_0.res_id,
        m1_0.rm_description,
        m1_0.rm_image_url,
        m1_0.rm_name,
        m1_0.rm_price 
    from
        p_restaurant_menu m1_0 
    where
        m1_0.rm_id=?
Hibernate: 
    select
        r1_0.res_id,
        r1_0.created_at,
        r1_0.created_by,
        r1_0.deleted_at,
        r1_0.deleted_by,
        r1_0.modified_at,
        r1_0.modified_by,
        r1_0.res_owner_id,
        r1_0.res_address,
        r1_0.res_image_url,
        r1_0.res_name,
        r1_0.res_phone 
    from
        p_restaurant r1_0 
    where
        r1_0.res_id=?

 

 

— 수정 후

// Service
UUID ownerUser = menuRepository.getRestaurantOwnerId(menu.getRmId());

// Repository
@Query("Select u.userId from User u join Restaurant r on r.owner.userId = u.userId join Menu m on m.restaurant.resId = r.resId where m.rmId = :rmId ")
UUID getRestaurantOwnerId(@Param("rmId") UUID rmId);

 

실행되는 select 문

Hibernate: 
    select
        u1_0.user_id 
    from
        p_users u1_0 
    join
        p_restaurant r1_0 
            on r1_0.res_owner_id=u1_0.user_id 
    join
        p_restaurant_menu m1_0 
            on m1_0.res_id=r1_0.res_id 
    where
        m1_0.rm_id=?

 

⚠️ 기존 코드의 문제점

가독성이 좋지 않고,

연관된 테이블을 타고, 타고 가서 userId를 조회했기 때문에 N+1 문제가 발생할 가능성이 있었다.
select문이 2번만 나간 것 보면 다행히 발생하진 않은 듯 하지만!

 

좋은 시도였다.

 

---

 

2. Querydsl BooleanExpression 

JPA Querydsl은 프로젝트하면서 주먹구구식으로 경험해본 거라 놓치고 있는 부분이 많은 것 같다.

	@Override
	public List<Menu> findMenusByResIdAndRole(UUID resId, UserDetailsImpl userDetails) {
		QMenu menu = QMenu.menu;
		UserRoleEnum role = userDetails.getRole();
		UUID userId = userDetails.getUserId();
		boolean isOwner = isOwner(userId, resId);

		BooleanExpression condition = menu.restaurant.resId.eq(resId);
		// role과 OWNER id 일치 여부에 따른 condition 추가
		if (role.equals(UserRoleEnum.CUSTOMER) || (role.equals(UserRoleEnum.OWNER) && !isOwner)) {
			log.info("role : {}", role);
			// condition.and(menu.isPublic.eq(true)); - 틀린 코드
			condition = condition.and(menu.isPublic.eq(true));
		}

		return jpaQueryFactory.selectFrom(menu).where(condition).fetch();
	}

BooleanExpression을 반환하는 condition 에다가 조건에 따라서 condition을 추가하는 코드를 작성했는데 제대로 작동하지 않았다.

BoleanExpression은 불변객체이기 때문에 새롭게 저장해줘야 한다.

그래서 

condition.and(menu.isPublic.eq(true)); 가 아닌

condition = condition.and(menu.isPublic.eq(true)); 로 적어서 꼭 값을 다시 넣어주어야 한다!!!!⭐⭐⭐⭐⭐⭐⭐⭐⭐

 

 

---

 

3. HTTP 응답값에 대한 고민 

PATCH 메서드로 데이터를 수정하는 api와 삭제하는 api 를 구현했는데 반환값을 어떻게 보내야할지 조금 고민했다.

- 수정의 경우, 수정한 값을 클라이언트가 다시 보여주어야 할 것 같아서 200 OK로 데이터 값을 응답에 보냈다.

- 삭제의 경우, 삭제한 값이 클라이언트에 반환될 필요가 없으니까 204 NO CONTENT가 맞는 것 같은데, 우리 프로젝트에서 응답 값을 보낼 때 CommonResponse 에 담아서 보내고, 이곳에는 SUCCESS CODE와 result가 담기도록 구현되어 있다. 그래서 보낼 데이터가 없는 경우에도 CommonResponse에 담아서 보내는 게 맞는지..? 그냥 nocontent().build()로 마무리하면 되는지 고민했다. 적합한 답이 뭔지는 찾지 못했다.. 그냥 방식을 결정했을 뿐! nocontent().build()로 작성했다.

 

 

 

 

< 200번대 HTTP 상태 코드 >

• 200 OK
  • 요청 성공, 요청이 정상적으로 처리됨
• 201 Created
  • 요청 성공, 새로운 리소스가 생성됨
• 202 Accepted
  • 요청 수락, 요청이 성공적으로 수락되었지만, 아직 완료되지 않음 (비동기적 처리)
• 204 No Content
  • 요청 성공, 반환할 내용이 없음 (주로 DELETE 요청에 응답)

 

 

Exception 또한 무신경하게 작성하다가 수정했다.

< Exception HTTP 상태 코드 >

• 401 UNAUTHORIZED 
  • 인증이 필요한 요청이거나 인증에 실패함
• 403 FORBIDDEN 
  • 리소스는 존재하지만, 클라이언트에게 해당 리소스에 대한 접근 권한이 없음
• 404 NOT_FOUND 
  • 요청한 리소스를 찾을 수 없음

 

느낀 점

예전에는 그냥 OK로 모두 보내버렸고, 각각의 차이도 잘 알지 못했는데 (알아볼 생각도 안하고 그냥 기능 구현에만 초점을 두었던 옛날..ㅠ) 진작 좀 찾아볼 걸 그랬다 ㅎㅎ..............