[FEAT] 마이페이지 API 구현 #124

jun02160 · 2024-02-10T11:17:12Z

📌 관련 이슈

closed #117

✨ 어떤 이유로 변경된 내용인지

마이페이지의 유저와 매칭 유저(부모-자식) 정보를 반환하는 API 개발

ParentchildRelation ENUM에서 유저의 성별과 자식 여부에 따른 유형을 반환하는 로직 추가 -> _user_type으로 반환
'나'로 표시되는 부분을 구분하기 위한 isMeChild 필드도 함께 반환하도록 했습니다!

Test 결과

매칭 이후

매칭 전

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

API 명세서 추가해뒀으니 확인 바랍니다 :)
상대 미연결인 부분에 대한 반환값은 뷰 나오는 대로 추가하여 반영하겠습니다!

# 제목을 아랫줄에 작성, 제목 끝에 마침표 금지, 무엇을 했는지 명확하게 작성 ################ # 본문(추가 설명)을 아랫줄에 작성 ################ # 꼬릿말(footer)을 아랫줄에 작성 (관련된 이슈 번호 등 추가) ################ # 아이콘 코드 설명 # 🎨 style: 코드의 구조/형태 개선 # 🔥 remove: 코드/파일 삭제 # 🐛 bugfix: 버그 수정 # 🚑 hotfix: 긴급 수정 # ✨ feat: 새로운 기능 구현 # 📝 docs: 문서 추가/수정 # 🎉 init: 프로젝트 시작 # ✅ test: 테스트 추가/수정 # 🔖 release: 릴리즈/버전 태그 # 🔧 chore: 동작에 영향 없는 코드 or 변경 없는 변경사항(주석 추가 등) or 디렉토리 구조 변경 # ♻️ refactor: 코드 리팩토링, 전면 수정 # ⚡️ fix: 간단한 수정 ################

0lynny

아들 최영린입니다~ 설날 작업 넘 수고 많으셨습니다앙

0lynny · 2024-02-10T12:22:30Z

umbba-domain/src/main/java/sopt/org/umbba/domain/domain/parentchild/ParentchildRelation.java

+            if (isChild) return "딸";
+            else return "엄마";


p5;
갠취이긴 하지만 삼항연산자로 뺴도 좋아보여요!

좋은 의견 감사합니다 !!!

0lynny · 2024-02-10T12:24:25Z

umbba-api/src/main/java/sopt/org/umbba/api/service/qna/QnAService.java

@@ -227,7 +264,7 @@ private List<QnA> getQnAListByParentchild(Parentchild parentchild) {
        return qnaList;
    }

-    private QnA getTodayQnAByParentchild(Parentchild parentchild) {
+    protected QnA getTodayQnAByParentchild(Parentchild parentchild) {


q;
protected로 하신 이유가 궁금합니다~!

흠흠 이게 긴 사연이 있는데요!! 일단 잘못 들어간 게 맞구요 ㅎㅎ

바꾸게 된 과정에 대해 설명을 드려보자면,

원래 유저 정보 조회를 QnAService 보다는 다른 서비스 클래스로 분리하는 게 맞다고 생각해서 기능 상으로 도메인이 더 부합한 ParentchildService에서 구현을 하고자 했습니다!

하지만 오늘의 질문, Parentchild 조회 등 동일한 로직이 QnAService에 여럿 쓰이고 있어 해당 로직을 protected로 열어두고 호출하려 했으나,, Service에서 다른 Service를 의존하는 상황 자체가 순환 참조를 발생시킬 우려가 있다고 판단했습니다.

결론! 일단 QnAService에서 구현을 해두고 주로 쓰이는 QnAService의 조회 로직을 분리하기 위해 전면 리팩토링을 진행하기로 했습니다 허헛

그래서 다시 private으로 바꾸겠습니다 .. 감사합니다!!

ddongseop

바쁜 와중에도 수고했슴당 🔥

ddongseop · 2024-02-10T12:31:01Z

umbba-api/src/main/java/sopt/org/umbba/api/config/SecurityConfig.java

            "/kakao/**", "/login", "/reissue",
-            "/qna/**", "onboard/**", "/home", "/dummy",
+            "/qna/**", "onboard/**", "/home", "/dummy", "/user/me",
 //          "/log-out",
            "/test", "/profile", "/health", "/actuator/health",
            "/alarm/qna", "/alarm/drink",


생각해보니까 인증이 필요 없는 API들만 여기 추가되어야하는건데,
일단 테스트를 위해 다 추가해놨던게 그대로 있었네요 ...
다음에 한번에 싹 빼는걸로 합시당

맞아요 ,,ㅋㅋㅋㅋ 좋습니당

ddongseop · 2024-02-10T12:32:42Z

.gitignore

@@ -7,6 +7,8 @@ build/
 !**/src/main/**/build/
 !**/src/test/**/build/

+umbba-test.http


포스트맨이 아니라 IntelliJ의 .http 테스트 활용하신건가요!
신기해서 저도 찾아봤습니당
https://ksh-coding.tistory.com/97

ddongseop · 2024-02-10T12:36:48Z

umbba-api/src/main/java/sopt/org/umbba/api/service/example.txt

@@ -0,0 +1 @@
+.


요건 무슨 용도의 파일인가요!

어엇 잘못 들어간 파일 같네요 삭제하겠습니다 !!

ddongseop · 2024-02-10T12:42:07Z

umbba-domain/src/main/java/sopt/org/umbba/domain/domain/parentchild/ParentchildRelation.java

+    public static String getUserType(ParentchildRelation relation, boolean isChild) {
+        if (relation.childGender.equals("여자")) {
+            if (isChild) return "딸";
+            else return "엄마";
+        } else {
+            if (isChild) return "아들";
+            else return "아빠";
+        }
+    }


지금 예시로 있는 DB가 딱 그런 케이슨데요 (어쩌다보니)
초대하는 측에서 분명 엄마랑 하겠다고 골랐는데, 초대 받는 상대가 자기 성별을 남자로 골라버린 경우죠
이 경우 성별이 남자여도 엄마로 표시하기로 정했던 것 같습니다 (사회적으로는 엄마일...수도.. 있으니까요?)

암튼 그래서 지금 DB에 엄마-아들 관계로 되어 있음 + 둘다 성별은 남자임 이런 상태인데
my_user_type을 아빠가 아닌 엄마로 표시해줄 수 있도록! 로직이 수정되면 좋을것 같습니다!

네넵 그럼 User의 gender 필드 기준이 아닌, relation에 저장된 값과 isChild 기준으로 반환하는 로직으로 수정하겠습니다!!

ddongseop · 2024-02-10T13:07:50Z

umbba-api/src/main/java/sopt/org/umbba/api/service/qna/QnAService.java

+        long qnaCnt = qnaList.stream()
+            .filter(qnA -> qnA.isChildAnswer() && qnA.isParentAnswer())
+            .count();


DB 수정 완료해서 getCount만으로 처리해도 될것같습니다!

ddongseop · 2024-02-11T06:35:05Z

umbba-domain/src/main/java/sopt/org/umbba/domain/domain/parentchild/ParentchildRelation.java

@@ -59,6 +59,15 @@ public static ParentchildRelation relation(String gender, String relationInfo, b
        throw new CustomException(ErrorType.INVALID_PARENT_CHILD_RELATION_INFO);
    }

+    // 아들 | 딸 | 엄마 | 아빠 구분


아주 좋습니당!

jun02160 added 7 commits February 8, 2024 00:22

[FEAT] 마이페이지 API 구현 #117

d2cd9c9

[FEAT] 마이페이지 API 매칭 날짜 필드 수정 #117

72c97db

[CHORE] WhiteList 추가 #117

c46bf1a

[MERGE] 병합 충돌 오류 해결 #117

932178d

[MERGE] 병합 충돌 오류 해결 #117

94d61ce

[FIX] 마이페이지 문답 수 로직 수정 #117

7c05eee

jun02160 added 🔥 Pull Request PR 날림 예준🍒 ✨Feat 새로운 기능 추가 🪄API 서버 API 통신 labels Feb 10, 2024

jun02160 requested review from ddongseop and 0lynny February 10, 2024 11:17

jun02160 self-assigned this Feb 10, 2024

0lynny approved these changes Feb 10, 2024

View reviewed changes

ddongseop approved these changes Feb 10, 2024

View reviewed changes

ddongseop reviewed Feb 10, 2024

View reviewed changes

jun02160 added 2 commits February 11, 2024 01:08

[FIX] 부모-자식 유저 타입 조회 로직 수정 #117

02b85b7

[FEAT] 매칭 이전에 대한 반환값 추가 #117

924d3ee

ddongseop reviewed Feb 11, 2024

View reviewed changes

jun02160 merged commit 2261d87 into develop Feb 11, 2024
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEAT] 마이페이지 API 구현 #124

[FEAT] 마이페이지 API 구현 #124

jun02160 commented Feb 10, 2024 •

edited

Loading

0lynny left a comment

0lynny Feb 10, 2024

jun02160 Feb 10, 2024

0lynny Feb 10, 2024

jun02160 Feb 10, 2024

ddongseop left a comment

ddongseop Feb 10, 2024

jun02160 Feb 10, 2024

ddongseop Feb 10, 2024

ddongseop Feb 10, 2024

jun02160 Feb 10, 2024

ddongseop Feb 10, 2024

jun02160 Feb 10, 2024

ddongseop Feb 10, 2024

ddongseop Feb 11, 2024

[FEAT] 마이페이지 API 구현 #124

[FEAT] 마이페이지 API 구현 #124

Conversation

jun02160 commented Feb 10, 2024 • edited Loading

📌 관련 이슈

✨ 어떤 이유로 변경된 내용인지

마이페이지의 유저와 매칭 유저(부모-자식) 정보를 반환하는 API 개발

Test 결과

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

0lynny left a 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

ddongseop left a 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jun02160 commented Feb 10, 2024 •

edited

Loading