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

[Feat] #113-진행자를 이겼을 때 isWin으로 설정 및 리스트 정렬 수정 #114

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

HyunJaeyeon
Copy link
Member

👀 연관 이슈

resolved: #113

🛏️ 요약

이번 PR에서 작업한 내용을 간략히 설명해주세요

  • 진행자를 이겼을 때 isWin으로 설정 및 리스트 정렬 수정

🎁 작업 내용

1. / 3하기 전 원점수로 비교하다보니 동점이 진짜 동점이 아니다?

  • 진행자: 278, 참가자 277 -> 같은 98점으로 뜨지만
  • 원점수로 비교했을 때 진행자 이긴걸로 나옴
let isWin = !isHost && (player.score / 3) >= (rankedPlayers.first(where: { $0.isHost })?.score ?? 0) / 3

###2. 진짜 동점이여도 진행자가 위에 뜨는 에러 발생

  • 내림차순 정렬은 같지만, 점수가 같으면 진행자가 아닌 참가자가 우선으로 오도록 설정
     if $0.score / 3 != $1.score / 3 {
                    // 점수를 기준으로 내림차순 정렬
                    return $0.score / 3 > $1.score / 3
                } else {
                    // 점수가 같으면 진행자가 아닌 참가자가 우선
                    return !$0.isHost && $1.isHost
                }

📱 작업 스크린샷 (선택)

  • 동점이 안나와서 프리뷰로 확인함 ㅎ(>,ㅎ
image

🎾 공유사항 (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요



격려의 한 마디 (선택)

외마디 비명 (선택)

- 진행자보다 위에 위치하게
@HyunJaeyeon HyunJaeyeon self-assigned this Dec 4, 2024
@HyunJaeyeon HyunJaeyeon merged commit 14ff728 into showcase Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant