#215 [refactor] 지원서 상세보기 api service 분리 #216

pkl0912 · 2024-01-30T16:20:08Z

해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)

해결하려는 문제가 무엇인가요?

지원서 상세보기 service 분리

어떻게 해결했나요?

modelRetrieve service 사용,
getIsSend 에서 offer 가 있는지 없는지 확인할 때 offerRetrieve 를 쓸까말까 고민했는데...
offerRetrieve 를 쓰면 applicationRetirieve 랑 순환참조가 나서 안되더라고요...
그래서 일단 getIsSend를 applicationRetrieve 에 넣어놨는데
좋은방법 있으면 공유 부탁드립니다!

KWY0218

p2

좋은방법 있으면 공유 부탁드립니다!

Response로 보낼 값을 컨트롤러 단에서 조립하는 것은 어떨까요?
현재 최종적으로 클라이언트에게 반환할 정보들을 보면 application, model, offerIsSend 정보로 보입니다.
DesignerController에서
ModelRetrieveService를 통해 ModelDto를
ApplicationRetrieveService를 통해 ApplicationDto를
OfferRetrieveService를 통해 isOffer정보를
반환받은 후
컨트롤러 단에서 해당 Dto들을 조합해서 Response로 변환한 다음에 클라이언트에게 반환하는 것은 어떨까요?

이렇게 하면 컨트롤러 단에 서비스 의존이 뒤죽박죽이 되겠지만...
서비스 단의 코드는 더욱 간결해질 것이고, 서비스간의 의존성은 줄어들 것이라고 생각이 듭니다..!

어떻게 생각하시나요???

KWY0218 · 2024-01-31T00:09:52Z

src/main/java/com/moddy/server/controller/designer/DesignerController.java

@@ -87,7 +87,7 @@ public SuccessNonDataResponse offerCreateRequest(
    public SuccessResponse<ApplicationDetailInfoResponse> getApplicationDetailInfo(
            @Parameter(hidden = true) @UserId Long userId,
            @PathVariable(value = "applicationId") Long applicationId) {
-        return SuccessResponse.success(SuccessCode.MODEL_APPLICATION_DETAil_INFO_SUCCESS, designerService.getApplicationDetail(userId, applicationId));
+        return SuccessResponse.success(SuccessCode.MODEL_APPLICATION_DETAil_INFO_SUCCESS, hairModelApplicationRetrieveService.getApplicationDetail(userId, applicationId));


p3
여기서도 userId를 designerId로 수정하면 좋을 것 같습니다!

KWY0218 · 2024-01-31T00:12:13Z

src/main/java/com/moddy/server/service/application/HairModelApplicationRetrieveService.java

+
+        List<PreferRegion> preferRegions = preferRegionJpaRepository.findAllByModelId(modelId);
+


p2
Offer 서비스에서 해당 api를 위해 preferRegion을 참조하는 것 보다
modelRetrieve를 통해서 모델 정보를 가져올 때 preferRegion 정보도 같이 가져오는 것이 어떨까요?

KWY0218 · 2024-01-31T00:44:17Z

@pkl0912
추가로 마일스톤은 이슈에만!!! 지정하면 좋을 것 같습니다!!

…into refactor/#215

hellozo0

good.....파일 변동이 많아서 한번 싹 정리되고 다시 올께요... 굳굳...

hellozo0 · 2024-01-31T16:47:05Z

src/main/java/com/moddy/server/controller/designer/DesignerController.java

+        ApplicationDto applicationDto = hairModelApplicationRetrieveService.getApplicationDetailInfo(applicationId);
+        ApplicationModelInfoDto modelInfoDto = modelRetrieveService.getApplicationModelInfo(applicationId);
+        ApplicationInfoResponse applicationInfoResponse = new ApplicationInfoResponse(
+                applicationId,
+                applicationDto.modelImgUrl(),
+                applicationDto.hairLength(),
+                applicationDto.preferHairStyleList(),
+                applicationDto.recordResponseList(),
+                applicationDto.hairDetail(),
+                hairServiceOfferRetrieveService.getIsSendStatus(applicationId, designerId)
+        );
+
+        ModelInfoResponse modelInfoResponse = new ModelInfoResponse(
+                modelInfoDto.modelId(),
+                modelInfoDto.name(),
+                modelInfoDto.age(),
+                modelInfoDto.gender(),
+                modelInfoDto.regionList(),
+                applicationDto.instgramId()
+        );
+
+        ApplicationDetailInfoResponse applicationDetailInfoResponse = new ApplicationDetailInfoResponse(applicationInfoResponse,modelInfoResponse);
+        return SuccessResponse.success(SuccessCode.MODEL_APPLICATION_DETAil_INFO_SUCCESS, applicationDetailInfoResponse);


p2 :
이 친구들도 DesignerController가 아닌 ApplicationController를 만들어서 그곳에 이동해야할것 같습니다~!

@hellozo0
사실.. 메서드 별 Tag가 되는 줄 모르고 컨트롤러를 분리하면 swagger가 분리되기 때문에 controller 분리를 하지 말기로 말하긴 했습니다..!
그래서 이번 스프린트엔 컨트롤러 분리는 선택으로 가져갈까 합니다..!
어떻게 생각하시나요..?!

KWY0218

수정하시느라 고생하셨습니다!
코맨트 확인해주세요!

KWY0218 · 2024-02-01T00:59:25Z

src/main/java/com/moddy/server/service/offer/HairServiceOfferRetrieveService.java

+    public Boolean getIsSendStatus(final Long applicationId, final Long userId) {
+        Optional<HairServiceOffer> offer = hairServiceOfferJpaRepository.findByHairModelApplicationIdAndDesignerId(applicationId, userId);
+        return offer.isPresent();
+    }


p2
객체 타입인 Boolean 말고 원시 타입인 boolean을 사용해주세요!

KWY0218 · 2024-02-01T01:00:09Z

src/main/java/com/moddy/server/service/application/HairModelApplicationRetrieveService.java

+        List<PreferRegion> preferRegions = preferRegionJpaRepository.findAllByModelId(modelId);
+
+        List<String> regionList = preferRegions.stream().map(preferregion -> {
+            return preferregion.getRegion().getName();
+        }).collect(Collectors.toList());


p2
해당 구문은 사용하지 않는 구문인 것 같습니다!
사용하지 않는 구문은 제거해주세요!

또한, 해당 구문을 지우면 PreferRegionRepository도 사용하지 않게될텐데 해당 변수와 import도 같이 제거해주세요!

…into refactor/#215 # Conflicts: # src/main/java/com/moddy/server/service/application/HairModelApplicationRetrieveService.java # src/main/java/com/moddy/server/service/offer/HairServiceOfferRetrieveService.java

KWY0218

좋습니다 ~ !!!

KWY0218 · 2024-02-06T10:28:28Z

src/main/java/com/moddy/server/controller/designer/DesignerController.java

+    private final ModelRetrieveService modelRetrieveService;
+    private final HairServiceOfferRetrieveService hairServiceOfferRetrieveService;


p2
사용하지 않는 변수는 제거해주세요!

#215 [refactor] 지원서 상세보기 api service 분리

9ec918e

pkl0912 added the refactor label Jan 30, 2024

pkl0912 added this to the 1차 리팩토링 기간 milestone Jan 30, 2024

pkl0912 requested review from hellozo0 and KWY0218 January 30, 2024 16:20

pkl0912 self-assigned this Jan 30, 2024

KWY0218 reviewed Jan 31, 2024

View reviewed changes

KWY0218 removed this from the 1차 리팩토링 기간 milestone Jan 31, 2024

KWY0218 modified the milestone: 1차 리팩토링 기간 Jan 31, 2024

pkl0912 added 3 commits January 31, 2024 17:11

Merge branch 'develop' of https://github.com/TEAM-MODDY/moddy-server …

e0008ea

…into refactor/#215

#215 [refactor] applicationRetrieve service 에서 컨트롤러 변경

8a8fd44

#215 [refactor] 필요없는 함수 제거

c009e4f

hellozo0 reviewed Jan 31, 2024

View reviewed changes

KWY0218 reviewed Feb 1, 2024

View reviewed changes

pkl0912 added 2 commits February 6, 2024 17:13

Merge branch 'develop' of https://github.com/TEAM-MODDY/moddy-server …

5bb0678

…into refactor/#215 # Conflicts: # src/main/java/com/moddy/server/service/application/HairModelApplicationRetrieveService.java # src/main/java/com/moddy/server/service/offer/HairServiceOfferRetrieveService.java

#215 [refactor] 지원서상세보기 controller 분리

70d1dfd

KWY0218 approved these changes Feb 6, 2024

View reviewed changes

#215 [refactor] 사용하지 않는 변수 제거

ff08eb8

pkl0912 merged commit edbd217 into develop Feb 7, 2024
1 check passed

pkl0912 deleted the refactor/#215 branch February 7, 2024 04:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#215 [refactor] 지원서 상세보기 api service 분리 #216

#215 [refactor] 지원서 상세보기 api service 분리 #216

pkl0912 commented Jan 30, 2024 •

edited

Loading

KWY0218 left a comment

KWY0218 Jan 31, 2024

KWY0218 Jan 31, 2024

KWY0218 commented Jan 31, 2024

hellozo0 left a comment

hellozo0 Jan 31, 2024

KWY0218 Feb 1, 2024

KWY0218 left a comment

KWY0218 Feb 1, 2024

KWY0218 Feb 1, 2024

KWY0218 left a comment

KWY0218 Feb 6, 2024


		List<PreferRegion> preferRegions = preferRegionJpaRepository.findAllByModelId(modelId);

		private final ModelRetrieveService modelRetrieveService;
		private final HairServiceOfferRetrieveService hairServiceOfferRetrieveService;

#215 [refactor] 지원서 상세보기 api service 분리 #216

#215 [refactor] 지원서 상세보기 api service 분리 #216

Conversation

pkl0912 commented Jan 30, 2024 • edited Loading

관련 이슈번호

해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)

해결하려는 문제가 무엇인가요?

어떻게 해결했나요?

KWY0218 left a comment

Choose a reason for hiding this comment

KWY0218 Jan 31, 2024

Choose a reason for hiding this comment

KWY0218 Jan 31, 2024

Choose a reason for hiding this comment

KWY0218 commented Jan 31, 2024

hellozo0 left a comment

Choose a reason for hiding this comment

hellozo0 Jan 31, 2024

Choose a reason for hiding this comment

KWY0218 Feb 1, 2024

Choose a reason for hiding this comment

KWY0218 left a comment

Choose a reason for hiding this comment

KWY0218 Feb 1, 2024

Choose a reason for hiding this comment

KWY0218 Feb 1, 2024

Choose a reason for hiding this comment

KWY0218 left a comment

Choose a reason for hiding this comment

KWY0218 Feb 6, 2024

Choose a reason for hiding this comment

pkl0912 commented Jan 30, 2024 •

edited

Loading