Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/#140 내 파트너 조회 API 예외 처리 추가 #141

Merged
merged 5 commits into from
Aug 10, 2024

Conversation

suhyun0918
Copy link
Collaborator

🤨 Motivation

🔑 Key Changes

매칭 신청자가 1회차에 매칭을 신청했고, 다음 참가자들을 모집하기 위해 시스템은 매칭 회차를 업데이트 합니다.
그 후, 사용자가 내 파트너를 조회합니다.
이 때, 시스템은 업데이트 된 매칭 회차로 매칭 파트너를 조회합니다. (currentMatchingRound = 2, memberId = 1)
(당연히 쿼리 결과가 없습니다. 😓)

그래서, 조회 결과 리스트가 (isEmpty() == true && member.hasPartner() == true) 인 경우
직전 매칭 회차로 파트너를 조회합니다. (previousMatchingRound = 1, memberId = 1)
이렇게, 매칭 회차가 업데이트 된 경우에도 나의 매칭 결과를 조회 할 수 있도록 구현했습니다.

🙏 To Reviewers

'왜 (isEmpty() == true && member.hasPartner() == true) 인 경우를 확인하고 가져오는거냐?'
'바로 currentMatchingRound - 1 을 해서 previousMatchingRound를 구한다음 조회하면 되는거 아니냐?'
고 생각 하실 수 있습니다.

그렇게 구현하게 되면 실제 환경에서 매칭 결과를 바로 확인해볼 수 없게됩니다.
다음 매칭 회차가 +1 될때까지 기다려야 결과를 볼 수 있는 것이죠....
테스트 시, 결과를 바로 API로 조회해볼 수 있다는 장점이 있습니다!

나중엔 그냥 해당 api 요청할때 파라미터로 매칭 회차를 직접 넣어서 하는 방식이 더 나을수도 있겠네요...
하지만 이 api가 현재 매칭회차의 파트너 조회이므로,
이런식으로 구체적인 정보를 주지 않고도 결과를 주려다 보니 위 방식으로 구현하게 되었습니다.


오류를 찾아주신 @mjj111 명준님 감사합니다 😊

앞으론 다양한 예외 케이스에 대해서 생각을 깊게 하도록 하겠습니다 🥲

@suhyun0918 suhyun0918 added feat 기능 개발 이슈 fix 오류 관련 태그 style 코드 스타일 관련 태그 test 테스트 관련 태그 labels Aug 3, 2024
@suhyun0918 suhyun0918 requested review from mjj111 and coke98 August 3, 2024 09:34
@suhyun0918 suhyun0918 self-assigned this Aug 3, 2024
@suhyun0918
Copy link
Collaborator Author

변경된 파일이 많아보이지만, 대부분 import문 최적화로 인해 변경점이 생긴것입니다!

commit 단위로 보시면 보기 편하십니다

실제 변경된 코드는 몇 줄 안되므로, 피드백 할 부분이 없으시면 그냥 ✅ 만 해주세용

Copy link
Member

@mjj111 mjj111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'왜 (isEmpty() == true && member.hasPartner() == true) 인 경우를 확인하고 가져오는거냐?'

처음에 Key Changes 읽고 흠?! 했지만, 말씀하신 대로 해당 방법의 쿼리로 하게 되면 실제 환경에서 매칭 결과를 바로 확인해볼 수 없게되네요.
좋은 수정 방법으로 생각합니다 👍
고생하셨습니다 🙇‍♂️

@mjj111 mjj111 merged commit d31a3c7 into develop Aug 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 개발 이슈 fix 오류 관련 태그 style 코드 스타일 관련 태그 test 테스트 관련 태그
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants