[PC-1300] 다중매칭 메인 화면 구현 #171
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…into feature/PC-1300
3개의 조회 api 연동
- 레포에서 조회 시 예외를 처리하지 못함. (3개 중 한개만 에러나도 3개가 전부 예외처리)
- 메인화면 진입 마다 matchInfo를 api를 통해 갱신하고 있기 때문에 필요
remainMatchingUpdateTimeInSec 파라미터 제거
comst19
reviewed
Nov 25, 2025
| .collect { remainTimeInSec -> | ||
| setState { copy(remainWaitingTimeInSec = remainTimeInSec) } | ||
|
|
||
| if (remainTimeInSec == 0L) { |
Collaborator
There was a problem hiding this comment.
TimerImpl 로직을 보면 0은 emit되지 않고 바로 TIMEOUT_FLAG(-1)이 방출되는 것으로 보입니다.
그래서 0을 기준으로 동작하는 내부 로직이 있다면 수행되지 않을 것 같은데...
혹시 이 부분 한 번만 확인 가능하실까요?
Collaborator
Author
There was a problem hiding this comment.
0L 로 돌리니 -1에서 진행 안되는 것 확인했어요. 수정했습니다~
4a14e83
Collaborator
There was a problem hiding this comment.
이렇게 했을 때 타이밍 이슈는 괜찮았나요? -1초가 잠깐 보이는 문제가 생길 수도 있을 거 같아요
feature/matching/src/main/java/com/puzzle/matching/graph/main/MatchingViewModel.kt
Show resolved
Hide resolved
feature/matching/src/main/java/com/puzzle/matching/graph/main/MatchingViewModel.kt
Outdated
Show resolved
Hide resolved
| val dialogType: PurchaseDialogType = PurchaseDialogType.InsufficientPuzzle | ||
| ) : UiState { | ||
| val formattedRemainWaitingTime: String = formatTimeToHourMinuteSecond(remainWaitingTimeInSec) | ||
| val formattedRemainingTimeMap = |
Collaborator
There was a problem hiding this comment.
state에서 한 카드만 갱신되더라도 전체 카드의 포맷팅이 함께 일어나는 것 같아, 혹시 이렇게 두신 이유가 있으신지 궁금합니다.
(ex: 카드가 3개이고 각각 초마다 갱신된다면, 결과적으로 1초에 9번 정도 포맷팅이 반복될 수 있을 것 같습니다.)
다른 방법으로는 raw 값을 넘기고 MatchingUserCard 내부에서 포맷팅하는 방법이 있을 거 같아요
core/data/src/main/java/com/puzzle/data/repository/MatchingRepositoryImpl.kt
Outdated
Show resolved
Hide resolved
feature/matching/src/main/java/com/puzzle/matching/graph/main/MatchingViewModel.kt
Outdated
Show resolved
Hide resolved
tgyuuAn
approved these changes
Nov 25, 2025
| .collect { remainTimeInSec -> | ||
| setState { copy(remainWaitingTimeInSec = remainTimeInSec) } | ||
|
|
||
| if (remainTimeInSec == 0L) { |
tgyuuAn
added
머지 해도될듯염🌟
comst19
approved these changes
Nov 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1. ⭐️ 변경된 내용
2. 🖼️ 스크린샷(선택)
3. 💡 알게된 부분
4. 📌 이 부분은 꼭 봐주세요!
참고
현재 다중매칭 Api가 제대로 나오지않은 상태라 주석처리 해놓은 함수가많고 코드가 중구난방일 확률이 매우높습니다..
위에 요청드린 부분 위주로 봐주시면 감사하겠습니다.