feat: get personal Pick Paging mockApi #46

toychip · 2024-07-18T12:19:03Z

🔀 PR 제목의 형식을 잘 작성했나요? e.g. [add] pr template
🧹 불필요한 코드는 제거했나요?

작업 내용

개인, 특정 질문에 대한 픽을 조회하기 위해 QuestionId를 queryParamter로 받고, MemberId와 함께 검색하여 페이징하는 Mock Api를 작성했습니다.

비고 (첨부자료)

{
    "success": true,
    "data": {
        "questionId": "1234564",
        "questionContent": "세상에서 제일 멋쟁이인 사람",
        "questionEmojiImageUrl": "urlurl",
        "totalReceivedPickCount": 10,
        "picks": [
            {
                "pickId": "SAMPLE_PICK_ID",
                "pickerOrdinal": 14,
                "pickerIdOpen": true,
                "pickerId": "SAMPLE_MEMBER_ID",
                "pickerGenderOpen": true,
                "pickerGender": "MALE",
                "pickerPlatformOpen": true,
                "pickerPlatform": "SPRING",
                "pickerSecondInitialNameOpen": true,
                "pickerSecondInitialName": "ㅈ",
                "pickerFullNameOpen": true,
                "pickerFullName": "임준형",
                "latestPickedAt": "-999999999-01-01T00:00:00"
            },
           ----------- SAMPLE_PICK 생략 -----------
            {
                "pickId": "SAMPLE_PICK_ID",
                "pickerOrdinal": 14,
                "pickerIdOpen": true,
                "pickerId": "SAMPLE_MEMBER_ID",
                "pickerGenderOpen": true,
                "pickerGender": "MALE",
                "pickerPlatformOpen": true,
                "pickerPlatform": "SPRING",
                "pickerSecondInitialNameOpen": true,
                "pickerSecondInitialName": "ㅈ",
                "pickerFullNameOpen": true,
                "pickerFullName": "임준형",
                "latestPickedAt": "-999999999-01-01T00:00:00"
            }
        ],
        "totalPage": 1,
        "totalElements": 10,
        "isFirst": true,
        "isLast": true
    },
    "error": null
}

xonmin

몇가지 의견 남겼습니다! 수고하셨습니다

xonmin · 2024-07-21T07:58:20Z

service/src/main/kotlin/com/mashup/dojo/usecase/PickUseCase.kt

+                val platformOpen = pick.isPlatformOpen
+                val secondInitialNameOpen = pick.isSecondInitialNameOpen
+                val fullNameOpen = pick.isFullNameOpen
+                val pickerIdOpen = fullNameOpen && genderOpen && platformOpen && secondInitialNameOpen


이전에 기획 회의 때 얘기했던 것 같은데 fullName 알면 전체 다 오픈된 거랑 동일하다고 했던 것 같아서
fullNameOpen || (genderOpen && platformOpen && secondInitialNameOpen) 로 해야하지 않을까 싶은데
어떻게 생각하시나요!?

넵 저도 fullNameOpen || (genderOpen && platformOpen && secondInitialNameOpen)가 맞다고 생각합니다.
Pick domain에 fullNameOpen이 있어서 픽이 생성, 수정될 때 마다 fullNameOpen 필드를 추가로 Entity에서 가질 것 같아서 가져오는 방식으로 했습니다.
저장할 때 funllNameOpen 필드를 추가로 두어 저장하거나 가져올 때 연산하거나 저희끼리 정하면 좋을 것 같아요~

오.. 저는 별개 질문인데요. 이거 유즈케이스 로직에서 세부적인 정책을 넣으면, 추후 로직이 이곳저곳에서 쓰여야할 때 골칫거리가 될 것 같아요. 서비스 레이어 / 도메인 로직으로 내려버리는건 어떠세요?~

오 그것도 좋은 것 같아요. 저장할 때 service 내에서 연산, 저장하고 usecase에서 꺼내는 방식으로 수정하는 것도 좋을 것 같아요~!

service/src/main/kotlin/com/mashup/dojo/usecase/PickUseCase.kt

yaeoni

고생하셨습니다! 의견 몇가지 남ㄱ두엇어요~

yaeoni · 2024-07-21T12:06:14Z

api/src/main/kotlin/com/mashup/dojo/PickController.kt

+        responses = [
+            ApiResponse(responseCode = "200", description = "내가 받은 픽 중 특정 질문의 페이징")


함수 이름은 getPickDetail 이라 특정 픽의 상세 정보를 반환하는 메소드(마치 pick Id를 받고 해당 pick정보를 반환하는 것 같은,,)로 이해했는데 내부는 페이징(list)이라 헷갈리는 것 같아요 🤔

"특정 질문을 페이징" 한다 라는 정의도 이해가 잘 안가는데 설명 한 번 부탁드려도 되나요?

이것 정말 고민이 많았던 건데, 이번 회의 때 얘기나눠보면 좋을 것 같아요.
처음에는 "단순 게시판 목록, 게시판 조회처럼 /{pickId}로 조회하면 되겠지?" 라고 생각했어요.
하지만 첫번째 사진에서 오른쪽 사진으로 넘어갈 때 둘의 성격이 달랐어요.
이렇게 둘의 성격이 다르기 때문에, 받은 픽을 보여주는 페이지에서 특정 픽의 질문Id와, memberId로 검색해야한다고 생각했어요.
다른 좋은 방법이 있는지 함께 얘기해보면 좋을 것 같아요~~!

왼쪽은 본인이 받은 픽을 보여주는 페이지(질문 중복 가능)
오른쪽은 본인이 받은 픽들 중 특정 질문에 대한 픽 페이징

yaeoni · 2024-07-21T12:07:14Z

api/src/main/kotlin/com/mashup/dojo/PickController.kt

+        @RequestParam(required = false, defaultValue = "0") pageNumber: Int,
+    ): DojoApiResponse<PickPaging> {
+        val pickPaging: PickUseCase.GetPagingPick =
+            pickUseCase.getReceivedPickDetailPaging(PickUseCase.GetPagingPickCommand(MemberId("1"), QuestionId(questionId), pageNumber))


지금은 MemberId("1") 이라는 invalid한 상수값으로 파라미터를 넘겨주고 있는데 요거 추후에 고칠거라면 Todo 라도 붙여줘야 나중에 챙겨질 것 같아요!!

아하 넵넵 반영하겠습니다~!

yaeoni · 2024-07-21T12:09:21Z

service/src/main/kotlin/com/mashup/dojo/service/DefaultPickService.kt

+        /*val pickList: List<Pick> = pickRepository.findAllByMemberId(pickedMemberId)
+             .map { PickEnity.buildDomain() }*/
+        return listOf(DEFAULT_PICK)


요런것도 Todo 붙여놔야 나중에 챙길 수 있을 것 같아용

yaeoni · 2024-07-21T12:09:48Z

service/src/main/kotlin/com/mashup/dojo/service/MemberService.kt

@@ -16,7 +16,7 @@ import java.time.LocalDateTime
 interface MemberService {
    fun getCandidates(currentMemberId: MemberId): List<Candidate>

-    fun findMemberById(memberId: MemberId): Member
+    fun findMemberById(memberId: MemberId): Member?


오 요거 여기서 고쳤네요
제 PR들어가면 컨플릭트 날지도 ㅎ (제 PR에서도 고쳤었음)

ㅋㅋㅋㅋ 확인했습니다. pr 3개가 다 겹치네요

service/src/main/kotlin/com/mashup/dojo/usecase/PickUseCase.kt

yaeoni · 2024-07-21T12:12:17Z

service/src/main/kotlin/com/mashup/dojo/usecase/PickUseCase.kt

+                val platformOpen = pick.isPlatformOpen
+                val secondInitialNameOpen = pick.isSecondInitialNameOpen
+                val fullNameOpen = pick.isFullNameOpen
+                val pickerIdOpen = fullNameOpen && genderOpen && platformOpen && secondInitialNameOpen


오.. 저는 별개 질문인데요. 이거 유즈케이스 로직에서 세부적인 정책을 넣으면, 추후 로직이 이곳저곳에서 쓰여야할 때 골칫거리가 될 것 같아요. 서비스 레이어 / 도메인 로직으로 내려버리는건 어떠세요?~

toychip requested review from wonlog-g, yaeoni and xonmin as code owners July 18, 2024 12:19

toychip self-assigned this Jul 18, 2024

xonmin approved these changes Jul 21, 2024

View reviewed changes

yaeoni approved these changes Jul 21, 2024

View reviewed changes

toychip mentioned this pull request Jul 24, 2024

7월 4주차 스프린트 #51

Closed

3 tasks

toychip added 12 commits August 2, 2024 00:52

fix: Master branch rebase (08020052)

ac4ac78

fix: Master branch rebase (08020102)

b51198b

feat: MemberGender, MemberPlatform UNKNOWN value

4b7041c

fix: Master branch rebase (08020103)

36f7741

fix: Master branch rebase (08020111)

fcd115c

fix: Master branch rebase (08020115)

1194834

fix: Master branch rebase (08020118)

dcf10cf

refactor: rename Dto PagingPick -> PickPaging

970408d

docs: picked-detail API swagger

338fb60

fix: Master branch rebase (08020119)

969903b

refactor: getXxx() -> transformXxx() rename

ad26395

refactor: remove unused method

27bb614

toychip force-pushed the junhyoung/add-pick-detail-mock-api branch from 60deef3 to 27bb614 Compare August 1, 2024 16:20

fix: update page size to match frontend requirements

b2903a9

toychip merged commit c35a0f9 into master Aug 1, 2024
1 check passed

toychip deleted the junhyoung/add-pick-detail-mock-api branch August 1, 2024 16:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: get personal Pick Paging mockApi #46

feat: get personal Pick Paging mockApi #46

toychip commented Jul 18, 2024 •

edited

Loading

xonmin left a comment

xonmin Jul 21, 2024

toychip Jul 21, 2024

yaeoni Jul 21, 2024

toychip Jul 22, 2024

yaeoni left a comment

yaeoni Jul 21, 2024

toychip Jul 22, 2024 •

edited

Loading

yaeoni Jul 21, 2024

toychip Jul 22, 2024

yaeoni Jul 21, 2024

yaeoni Jul 21, 2024

toychip Jul 22, 2024

yaeoni Jul 21, 2024

		responses = [
		ApiResponse(responseCode = "200", description = "내가 받은 픽 중 특정 질문의 페이징")

feat: get personal Pick Paging mockApi #46

feat: get personal Pick Paging mockApi #46

Conversation

toychip commented Jul 18, 2024 • edited Loading

작업 내용

비고 (첨부자료)

xonmin 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

yaeoni left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

toychip Jul 22, 2024 • 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

toychip commented Jul 18, 2024 •

edited

Loading

toychip Jul 22, 2024 •

edited

Loading