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.

Sign up for GitHub

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

Jump to bottom

[Feat/186] 매칭 우선순위 API 구현 #189

Merged
merged 28 commits into from
Feb 4, 2025
Merged

[Feat/186] 매칭 우선순위 API 구현 #189

merged 28 commits into from
Feb 4, 2025

Conversation

rimi3226
Copy link
Contributor

@rimi3226 rimi3226 commented Jan 26, 2025 •
edited
Loading

🚀 개요

매칭 우선순위 API 구현

⏳ 작업 상세 내용

  • 조건에 따라 priority 값 측정하는 matchingPrioirtyEvaluateService 구현
  • 게임 모드, 매칭 타입에 따라 우선순위 계산하는 matchingPriorityCalculateService 구현
  • Mike ENUM 상태 두 가지로 변경
  • MemberService에서 GameStyle 관련 수정하는 Service를 전부 MemberGameStyleService로 옮김. 원래는 FacadeService에서 직접 각 단계를 불러와서 처리했으나, 재사용성을 고려해 MemberService의 updateGameStyle 메소드에서 MemberGameStyleService를 불러와서 실행시킴
  • Matching 우선순위 리스트 처리하기 전, 서로 차단한 사용자의 경우 제외
  • 5분동안 대기중인 매칭 기록 리스트 불러올 때 index 처리하도록 Repository 내용 설정

📝 더 꼼꼼히 봐야할 부분

  • 빠른대전 티어 기획 정리되면 수정하겠습니다!
  • Matching 로직 중에 차단당한 사용자를 확인하는 메소드를 사용하는 부분 괜찮은지 확인 부탁드립니다.
  • MatchingRepository에서 인덱스로 조회하는거 괜찮은지 확인 부탁드립니다.

🔍 반드시 참고해야하는 변경 사항

  • 없을 경우 체크

rimi3226 added 11 commits January 23, 2025 02:34
@rimi3226
✨ [Feat] 매칭 API 클래스 구조 설계 및 계산 클래스 구현
e8d4ae1
@rimi3226
✨ [Feat] 각 게임모드마다 달라지는 계산을 구현, MatchingPriorityCalculateService
c56a307
@rimi3226
✨ [Feat] Mike ENUM 값 변경 및, 매칭 우선순위 계산 로직 작성
374883e
@rimi3226
✨ [Feat] 빠른 대전 제외, MatchingPriorityCalculateService 매칭 타입, 게임 모드에 따른 …
00b4ee8 
…우선순위 값 계산 구현
@rimi3226
✨ [Feat] 매칭 우선순위 API DTO 생성
bb8e081
@rimi3226
✨ [Feat] 매칭 우선순위 API DTO 생성
b4f2e94
@rimi3226
✨ [Feat] GameStyle 변경하는 로직 MemberFacadeService가 아닌 MemberService 내에서 …
14b3e7f 
…진행하도록 수정
@rimi3226
✨ [Feat] setGameStyle Service 수정
d6f06e7
@rimi3226
✨ [Feat] 매칭 정보로 Member 업데이트
e2996c9
@rimi3226
✨ [Feat] 매칭 우선순위 계산 Facade Service 구현
3e9ca64
@rimi3226
✨ [Feat] 제한 조건 추가
3c2b9db
@rimi3226 rimi3226 self-assigned this Jan 26, 2025
@rimi3226 rimi3226 linked an issue Jan 26, 2025 that may be closed by this pull request
✨ [Feat] 매칭 우선순위 계산 API 구현 #186
Closed
2 tasks
Eunjin3395
Eunjin3395 requested changes Jan 26, 2025
View reviewed changes
Copy link
Member

@Eunjin3395 Eunjin3395 left a comment

Choose a reason for hiding this comment

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

일단 api 요청이 에러가 나서 ㅠㅠ 수정 후에 다시 더 테스트를 해봐야 할 것 같습니다......

rimi3226 added 13 commits January 26, 2025 23:54
@rimi3226
🐛 [fix] 우선순위 API 구현 안되는거 해결
87d435b
@rimi3226
🐛 [fix] PENDING->Pending, MemberService, MemberGameStyleService 분리
d2c828c
@rimi3226
✨ [Feat] 겜구매칭 DB 조회할 때, 조건 제외하고 조회하도록 수정
52ce6d9
@rimi3226
✨ [Feat] 주석 추가
2ed159f
@rimi3226
✨ [Feat] 주석 추가
6cf005d
@rimi3226
✅ [Test] MatchingScoreCalculatorTest 테스트 구현
ea3641b
@rimi3226
✅ [Test] MatchingStrategyProcessorTest 테스트 구현
bca80d4
@rimi3226
✅ [Test] 필드명 수정
2b95339
@rimi3226
✅ [Test] MatchingService 매칭 리스트 조회 테스트 구현
ebba37a
@rimi3226
✅ [Test] MatchingService 매칭 게임모드, 매칭타입에 따른 우선순위 계산 테스트 구현
e4749ff
@rimi3226
✅ [Test] MatchingServiceTest 대기 중인 매칭 리스트 조회 구현
2caa5aa
@rimi3226
✅ [Test] MatchingServiceTest 매칭 기록 생성 테스트 구현
721f2c7
@rimi3226
✨ [Feat] 특정 사용자의 가장 최근 matchingRecord 조회
a069368
Eunjin3395
Eunjin3395 approved these changes Feb 4, 2025
View reviewed changes
Copy link
Member

@Eunjin3395 Eunjin3395 left a comment

Choose a reason for hiding this comment

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

테스트 코드 제외 리뷰 완료

Eunjin3395
Eunjin3395 approved these changes Feb 4, 2025
View reviewed changes
Copy link
Member

@Eunjin3395 Eunjin3395 left a comment

Choose a reason for hiding this comment

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

👍

@rimi3226 rimi3226 merged commit 28893c4 into develop Feb 4, 2025
1 check passed
@rimi3226 rimi3226 deleted the feat/186 branch February 4, 2025 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Eunjin3395 Eunjin3395 Eunjin3395 approved these changes

@coldmans coldmans Awaiting requested review from coldmans coldmans is a code owner

Assignees

@rimi3226 rimi3226

Labels
✨ feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 매칭 우선순위 계산 API 구현
2 participants
@rimi3226 @Eunjin3395