[#54] 상품 조회 시 정렬/필터 POST 방식으로 수정 #60

JIWON27 · 2025-01-09T16:00:04Z

상품 조회 시 정렬/필터 POST 방식으로 수정

GET -> POST로 변경

- ProductResponse/ImageResponse 위치 이동 - 위치 이동에 따른 ProductResponse/ImageResponse 경로 수정

- GET 방식에서 POST 방식으로 변경 - requestParam을 requestBody로 변경하여 JSON 형태로 받도록 수정

- 필터링 조건들을 requestbody로 받을 수 있도록 POST 방식으로 수정 - 추천 서버에서 반환해주는 상품들은 ProductFindResponse의 recommended 필드를 true로 할당

- 일반 상품과 추천 상품을 합친 후, shuffle이 되어 정렬 조건이 무시되던 문제 수정 -추천 서버에서 추천 상품만 shuffle한 후 반환하도록 수정, 이후 일반 상품 리스트와 병합

sonarqubecloud · 2025-01-10T10:24:11Z

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

f-lab-moony

고생 많으셨습니다 ~
머지는 지원님이 됐다싶을때 해주시면 될 것 같아요 ~

f-lab-moony · 2025-01-14T12:58:28Z

module-api/src/main/java/com/example/moduleapi/controller/product/ProductApiController.java

@@ -48,16 +47,11 @@ public ProductFindResponse findById(@PathVariable Long id) {
        return productFacade.findById(id);
    }

-    @GetMapping
+    @PostMapping("/_search")


요거 _는 왜 들어간거에요 ?

조회와 POST 조회를 구분짓기위해 _추가했는데... 불필요한 부분이면 수정하겠습니다!

f-lab-moony · 2025-01-14T12:59:04Z

...pi/src/main/java/com/example/moduleapi/service/auction/HighestBidSseNotificationService.java

-                    .name("THE HIGHEST PRICE UPDATE") // 이벤트의 이름을 "THE HIGHEST PRICE UPDATE"로 지정
-                    .data(data) // 전송할 데이터
-                    .comment(comment)); // 이벤트에 대한 코멘트
+                                .id(productId.toString()) // 이벤트의 고유 ID (문자열 형태로 변환)


indent가 왜이렇게 차이가 많이나죠

엇 뭔가 이상해졌군요... 수정하겠습니다..

f-lab-moony · 2025-01-14T12:59:24Z

module-api/src/main/java/com/example/moduleapi/service/file/LocalFileService.java

-                .append(uuid)
-                .append(".")
-                .append(extension);
+            .append(uuid)


이건 또 들어왔네요 ㅋㅋ 신기하네

엇 뭔가 저장하면서 수정된거같군용...

f-lab-moony · 2025-01-14T16:03:34Z

module-api/src/main/java/com/example/moduleapi/service/file/LocalFileService.java

-                .append(userId)
-                .append(File.separator)
-                .append(savedImageFileName);
+            .append(baseUrl)


반복문 돌면서 만들어내는게 아니면 String.format 써도 좋을 것 같아요 ~

앗 넵 확인하고 수정하도록 하겠습니다!

f-lab-moony · 2025-01-14T16:03:49Z

...rc/main/java/com/example/moduleapi/service/httpClient/ProductRecommendationServerClient.java


 import io.github.resilience4j.circuitbreaker.annotation.CircuitBreaker;

 @FeignClient(name = "productRecommendationClient", url = "http://127.0.0.1:8085/api/v1/products", fallback = RecommendationFallBackMethod.class)
 @CircuitBreaker(name = "recommendationServer")
 public interface ProductRecommendationServerClient {

-    @GetMapping("/recommendations")
+    @PostMapping("/_recommendations")


여기도 언더바가 있네요 ?

추천 상품 조회 서버에도 POST 조회로 수정하면서 언더바 붙였는데 .. _가 의미가 없다면 수정하도록 하겠습니다!

f-lab-moony · 2025-01-14T16:05:53Z

...ion/src/main/java/com/example/modulerecommendation/service/ProductRecommendationService.java

            for (Map<ProductCondition, List<Long>> conditionMap : categoryMap.values()) {
                for (List<Long> productIds : conditionMap.values()) {
                    recommendationProductIds.addAll(productIds);
                }
            }
        } else {
            // 카테고리만 필터링된 경우
-            if (!categories.isEmpty()) {
-                for (Category category : categories) {
+            if (!productFilter.getCategory().isEmpty()) {


Filter를 추상화해서 종류별로 액션만 따로 정의해두면 여기서 if로 분기를 안해도 될 것 같아요

아하 넵 요 부분은 수정이 필요해보였는데 말씀해주신 방법대로 한번 적용해보도록 하겠습니다!

feat: ProductResponse에 추천상품인지 판단하는 필드 추가

634b361

- ProductResponse/ImageResponse 위치 이동 - 위치 이동에 따른 ProductResponse/ImageResponse 경로 수정

JIWON27 requested a review from f-lab-moony January 9, 2025 16:00

JIWON27 self-assigned this Jan 9, 2025

JIWON27 linked an issue Jan 9, 2025 that may be closed by this pull request

🚨 정렬/필터 고도화 #54

Closed

JIWON27 added 10 commits January 10, 2025 01:30

feat: ProductFilter/ProductFilterRequest 요청 dto 추가

8355dbd

refactor: 상품 조회 API GET -> POST 방식 변경 및 파라미터 처리 방식 수정

257b0a2

- GET 방식에서 POST 방식으로 변경 - requestParam을 requestBody로 변경하여 JSON 형태로 받도록 수정

refactor:상품 목록 조회시 ProductFacade로 User값을 꺼내서 반환하도록 수정

f67ce3c

refactor: 상품 목록 조회 API로부터 받는 User 타입 변경

346edee

refactor: ProductFacade, ProductService 테스트 코드 수정

86d3451

refactor: 추천 서버 API도 GET->POST로 수정

1185f6c

- 필터링 조건들을 requestbody로 받을 수 있도록 POST 방식으로 수정 - 추천 서버에서 반환해주는 상품들은 ProductFindResponse의 recommended 필드를 true로 할당

refactor: 수정된 추천 서버 API에 맞게 수정

9bd9c10

feat: ifNotEmpty 유틸 메서드 추가

5843cd5

refactor: repository계층에서 ProductFilterRequest를 받아서 처리하도록 수정

2888e2e

f-lab-moony requested changes Jan 14, 2025

View reviewed changes

JIWON27 changed the title ~~[#54] 상품 조회 시 정렬/필터 고도화~~ [#54] 상품 조회 시 정렬/필터 POST 방식으로 수정 Jan 17, 2025

JIWON27 merged commit 71f8e9b into main Jan 17, 2025
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#54] 상품 조회 시 정렬/필터 POST 방식으로 수정 #60

[#54] 상품 조회 시 정렬/필터 POST 방식으로 수정 #60

JIWON27 commented Jan 9, 2025 •

edited

Loading

sonarqubecloud bot commented Jan 10, 2025

f-lab-moony left a comment

f-lab-moony Jan 14, 2025

JIWON27 Jan 14, 2025 •

edited

Loading

f-lab-moony Jan 14, 2025

JIWON27 Jan 14, 2025

f-lab-moony Jan 14, 2025

JIWON27 Jan 14, 2025

f-lab-moony Jan 14, 2025

JIWON27 Jan 14, 2025

f-lab-moony Jan 14, 2025

JIWON27 Jan 14, 2025 •

edited

Loading

f-lab-moony Jan 14, 2025

JIWON27 Jan 14, 2025

[#54] 상품 조회 시 정렬/필터 POST 방식으로 수정 #60

[#54] 상품 조회 시 정렬/필터 POST 방식으로 수정 #60

Conversation

JIWON27 commented Jan 9, 2025 • edited Loading

상품 조회 시 정렬/필터 POST 방식으로 수정

sonarqubecloud bot commented Jan 10, 2025

Quality Gate passed

f-lab-moony left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JIWON27 Jan 14, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JIWON27 Jan 14, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JIWON27 commented Jan 9, 2025 •

edited

Loading

JIWON27 Jan 14, 2025 •

edited

Loading

JIWON27 Jan 14, 2025 •

edited

Loading