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

[개선] 2차 전형에서 특별전형으로 불합격시 일반전형으로 전환 #138

Merged
merged 5 commits into from
Sep 26, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Sep 25, 2024

🎫 관련 이슈

close #137


📄 개요

특별전형이 2차전형에서 불합격시 일반전형으로 전환되어 재응시하는 기능을 만들었습니다.


🔨 작업 내용

  • SelectSecondUseCase 에서 불합격 처리가 아닌 재응시하도록 변경했습니다.
  • Score 클래스에서 updateSecondRoundMeisterScoreToRegular(마이스터에서 일반전형으로 점수 변경) 과 updateSecondRoundSocialScoreToRegular(사회통합전형에서 일반전형으로 점수 변경) 메서드를 만들었습니다.
  • CalculateFormScoreService에서 사회통합전형의 심층면접 점수 비율을 일반전형으로 변경하는 메서드를 만들었습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

- changeToRegularSecondRound 메서드를 추가했어요.
- 위 메서드 추가를 위해 changeToRegular 메서드를 changeToRegularFirstRound로 변경했어요.
- 2단계 전형에서 마이스터전형과 사회통합전형이 일반전형으롭 변경될 때, 일반전형의 점수 비율을 반영하는 메서드를 추가했어요.
- 특별전형에서 불합격시 일반전형으로 재응시하는 기능을 추가했어요.
- 특별전형에서 불합격시 일반전형으로 재응시하는 기능을 테스트했어요.
@jyj1289 jyj1289 requested a review from a team as a code owner September 25, 2024 05:52
@jyj1289 jyj1289 requested review from cabbage16 and removed request for a team September 25, 2024 05:52
Copy link

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@jyj1289 jyj1289 self-assigned this Sep 25, 2024
@jyj1289 jyj1289 added the 개선 코드 리팩토링 label Sep 25, 2024
Copy link

sonarcloud bot commented Sep 25, 2024

@cabbage16 cabbage16 merged commit 33b30f6 into develop Sep 26, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
개선 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[개선] 2차 전형에서 특별전형으로 불합격시 일반전형으로 전환
2 participants