Merge pull request #28 from Sookmyung-Software-Hackathon/hwi_feat/post/…

…#4 change profile controller pk -> id #4
Sookmyung-Software-Hackathon · Aug 28, 2022 · 67e8db5 · 67e8db5
2 parents f6c4e72 + b7ff110
commit 67e8db5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 11 deletions.
diff --git a/src/main/java/com/team20/t4/member/MemberController.java b/src/main/java/com/team20/t4/member/MemberController.java
@@ -11,6 +11,7 @@
 
 import javax.validation.Valid;
 import java.util.List;
+import java.util.Map;
 
 @RequiredArgsConstructor
 @RestController
@@ -38,9 +39,9 @@ public MemberInfoResponseDto getInfo(){
         return memberService.getMemberInfo();
     }
 
-    @GetMapping("/member/{memberPk}/info")
-    public MemberInfoAndReviewListResponseDto getInfoAnotherMember(@PathVariable Long memberPk){
-        return memberService.getMemberInfo(memberPk);
+    @GetMapping("/member/info/other")
+    public MemberInfoAndReviewListResponseDto getInfoAnotherMember(@RequestBody Map<String, String> memberId){
+        return memberService.getMemberInfo(memberId.get("memberId"));
     }
 
     @GetMapping("/member/plans")

diff --git a/src/main/java/com/team20/t4/member/MemberService.java b/src/main/java/com/team20/t4/member/MemberService.java
@@ -136,21 +136,21 @@ public MemberInfoResponseDto getMemberInfo(){
     }
 
     @Transactional
-    public MemberInfoAndReviewListResponseDto getMemberInfo(Long memberPk){
-        Member memberByMemberPk = getMemberByMemberPk(memberPk);
-        List<ReviewResponseDtoByTarget> reviewListOfOther = getReviewListOfOther(memberPk);
-        MemberInfoResponseDto memberInfoResponseDto = MemberInfoResponseDto.toDtoWithProfileImage(memberByMemberPk, getImgUrl(memberByMemberPk));
+    public MemberInfoAndReviewListResponseDto getMemberInfo(String memberId){
+        Member memberByMemberId = getMemberByMemberId(memberId);
+        List<ReviewResponseDtoByTarget> reviewListOfOther = getReviewListOfOther(memberId);
+        MemberInfoResponseDto memberInfoResponseDto = MemberInfoResponseDto.toDtoWithProfileImage(memberByMemberId, getImgUrl(memberByMemberId));
         return new MemberInfoAndReviewListResponseDto(reviewListOfOther, memberInfoResponseDto);
     }
 
-    public List<ReviewResponseDtoByTarget> getReviewListOfOther(Long memberPk) throws RequestException {
-        Member target = getMemberByMemberPk(memberPk);
+    public List<ReviewResponseDtoByTarget> getReviewListOfOther(String memberId) throws RequestException {
+        Member target = getMemberByMemberId(memberId);
         List<Review> reviewList = reviewRepository.findAllByTarget(target);
         return getReviewResponseDtoByTargetList(reviewList);
     }
 
-    private Member getMemberByMemberPk(Long memberPk) {
-        return memberRepository.findById(memberPk)
+    private Member getMemberByMemberId(String memberId) {
+        return memberRepository.findByMemberId(memberId)
                 .orElseThrow(() -> new RequestException(RequestErrorCode.NOT_FOUND, "회원가입하지 않은 아이디입니다."));
     }