[Feat] Our Shoot 조회 API 구현

[cowboysj]

#️⃣ 관련 이슈

• [Feat] OUR SHOOT 조회 구현 #65

💡 작업내용

쿼리 파라미터 tab으로 구별해 Our shoot을 조회하는 API 구현

• Shoot 상태(yet, doing, done) 받아 반환
• mentioned면 내가 태그된 슛 반환
• Querydsl로 구현하였습니다.

📸 스크린샷(선택)

📝 기타

(참고사항, 리뷰어에게 전하고 싶은 말 등을 넣어주세요)

[iiqcov]

수고하셨습니다!!

[iiqcov]

이 부분을 스위치문으로 변경하면 가독성이 더 좋아질 것 같은데 혹시 if-else문으로 작성한 이유가 있을까요?

return switch (tab) {
  case YET -> getShootByStatus(figma, Status.YET);
...

이런식으로요!

[cowboysj]

switch문은 지양해야 한다는 글을 봤었어서 사용하지 않았었는데, 다시 찾아보니 그 의미는 switch문 대신 if-else문을 쓰라는 게 아니라 OCP 원칙에 위배되지 않게 작성하라는 거였네요..!

이 블로그 글 참고해서 object-mapping 방식으로 바꿔봤습니다..!

 @Transactional(readOnly = true)
    public List<GetOurShootResponse> getOurShoot(String tab) {
        final Figma figma = figmaUtil.getCurrentFigma();
        if (tab == null || tab.isBlank()) {
            tab = YET;
        }

        Map<String, Function<Figma, List<GetOurShootResponse>>> tabMapping = Map.of(
            YET, f -> getShootByStatus(f, Status.YET),
            DOING, f -> getShootByStatus(f, Status.DOING),
            DONE, f -> getShootByStatus(f, Status.DONE),
            MENTIONED, this::getMentionedShoot
        );

        return tabMapping.getOrDefault(tab, f -> {
            throw new CustomException(ErrorCode.INVALID_TAB);
        }).apply(figma);
    }

[iiqcov]

object-mapping 방식은 저도 처음 본 방식이네요! 너무 좋은 방법인 것 같습니다 👍 👍 다음에 저도 적용해봐야겠어요~
수고하셨습니다!!

[sonarqubecloud]

Quality Gate passed

Issues
17 New issues
0 Accepted issues

Measures
1 Security Hotspot
5.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud