Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
HYH0804
added a commit
that referenced
this pull request
May 27, 2026
) ## #️⃣ 연관된 이슈 <!-- Ex) - #이슈번호 --> <!-- 연관된 이슈 번호를 링크 형태로 작성하세요 --> - #250 ## 📝 작업 내용 <!-- 이번 PR/이슈에서 실제 수행한 작업 내용을 작성하세요 --> ### ♻️ Refactor <!-- 기존 코드 리팩토링 내용 작성 --> | 내용 | 파일 | |------|------| | 불필요한 import 제거 | `UserService.java` | | 투표 비율 계산 기준을 preVoteOption → postVoteOption으로 수정 | `BattleVoteServiceImpl.java` | | totalCount도 사후 투표 완료된 건만 카운트하도록 수정 | `BattleVoteServiceImpl.java` | | countByBattleAndPostVoteOption, countByBattleAndPostVoteOptionIsNotNull 메서드 추가 | `BattleVoteServiceImpl.java` | ## 📌 공유 사항 <!-- 팀원에게 공유할 내용이나 참고 사항 작성 --> <!-- 노션 환경 설정 파일 확인 부탁드립니다! --> > ▎ 1. 기존 코드에서 투표 비율(%)이 preVoteOption 기준으로 계산되어 사전 투표만 한 유저도 카운트에 포함되고 있었습니다. ▎ 2. 사후 투표(postVoteOption)가 완료된 건만 기준으로 비율을 계산하도록 수정했습니다. ## ✅ 체크리스트 <!-- PR 제출 전에 체크해야 할 사항들 --> - [x] Reviewer에 팀원들을 선택했나요? - [x] Assignees에 본인을 선택했나요? - [x] 컨벤션에 맞는 Type을 선택했나요? - [x] Development에 이슈를 연동했나요? - [x] Merge 하려는 브랜치가 올바르게 설정되어 있나요? - [x] 컨벤션을 지키고 있나요? - [x] 로컬에서 실행했을 때 에러가 발생하지 않나요? - [x] 팀원들에게 PR 링크 공유를 했나요? ## 📸 스크린샷 <!-- Swagger, Postman, JUnit 테스트 화면 첨부 --> <!-- 기능 동작 화면이나 테스트 결과 캡처를 첨부하면 좋습니다 --> ## 💬 리뷰 요구사항 <!-- 리뷰어에게 요청하는 구체적인 사항 작성 --> <!-- 종료 의도 판단을 Java 키워드 → AI 2차 검증 구조로 설계했는데 해당 구조가 유지보수 및 확장 측면에서 적절한지 의견 부탁드립니다. --> > 1. ## #️⃣ 연관된 이슈 <!-- Ex) - #이슈번호 --> <!-- 연관된 이슈 번호를 링크 형태로 작성하세요 --> - # ## 📝 작업 내용 <!-- 이번 PR/이슈에서 실제 수행한 작업 내용을 작성하세요 --> ### ✨ Feat <!-- 새로운 기능 구현 내용 작성 --> | 내용 | 파일 | |------|------| | 로그인 기능 구현 | `AuthController.java`, `AuthService.java` | | | | ### ♻️ Refactor <!-- 기존 코드 리팩토링 내용 작성 --> | 내용 | 파일 | |------|------| | 불필요한 import 제거 | `UserService.java` | | | | ### 🐛 Fix <!-- 버그 수정 내용 작성 --> | 내용 | 파일 | |------|------| | 널 포인트 예외 처리 | `PaymentService.java` | | | | ## 📌 공유 사항 <!-- 팀원에게 공유할 내용이나 참고 사항 작성 --> <!-- 노션 환경 설정 파일 확인 부탁드립니다! --> > 1. ## ✅ 체크리스트 <!-- PR 제출 전에 체크해야 할 사항들 --> - [ ] Reviewer에 팀원들을 선택했나요? - [ ] Assignees에 본인을 선택했나요? - [ ] 컨벤션에 맞는 Type을 선택했나요? - [ ] Development에 이슈를 연동했나요? - [ ] Merge 하려는 브랜치가 올바르게 설정되어 있나요? - [ ] 컨벤션을 지키고 있나요? - [ ] 로컬에서 실행했을 때 에러가 발생하지 않나요? - [ ] 팀원들에게 PR 링크 공유를 했나요? ## 📸 스크린샷 <!-- Swagger, Postman, JUnit 테스트 화면 첨부 --> <!-- 기능 동작 화면이나 테스트 결과 캡처를 첨부하면 좋습니다 --> ## 💬 리뷰 요구사항 <!-- 리뷰어에게 요청하는 구체적인 사항 작성 --> <!-- 종료 의도 판단을 Java 키워드 → AI 2차 검증 구조로 설계했는데 해당 구조가 유지보수 및 확장 측면에서 적절한지 의견 부탁드립니다. --> > 1. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
#️⃣ 연관된 이슈
📝 작업 내용
♻️ Refactor
UserService.javaBattleVoteServiceImpl.javaBattleVoteServiceImpl.javaBattleVoteServiceImpl.java📌 공유 사항
✅ 체크리스트
📸 스크린샷
💬 리뷰 요구사항