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

[개선] 1, 2차 합격자 결정 시 동점자 처리 #132

Closed
cabbage16 opened this issue Sep 8, 2024 · 0 comments · Fixed by #139
Closed

[개선] 1, 2차 합격자 결정 시 동점자 처리 #132

cabbage16 opened this issue Sep 8, 2024 · 0 comments · Fixed by #139
Assignees
@cabbage16
Labels
개선 코드 리팩토링

Comments

@cabbage16
Copy link
Member

cabbage16 commented Sep 8, 2024 •
edited
Loading

🙌 개선 방향

1, 2차 합격자를 자동으로 결정하는 도중 동점자가 발생할 시, 학교 입학전형 요강의 동점자 처리기준에 맞추어서 처리해야합니다.

  • 1차 전형에서 동점자가 발생한 경우
    • 동점자들 모두 1차합격 처리합니다.
  • 2차 전형에서 동점자가 발생한 경우
    • 입학 전형 요강의 동점자 처리기준에 따라 1순위부터 6순위까지의 기준으로 선택합니다.

💻 코드

No response

🏄‍♂️ 참고 자료

순위 기준 순위 기준 순위 기준
1순위 교과성적 3순위 NCS 인·적성검사 5순위 출석성적
2순위 심층면접 4순위 3학년 1학기 성적 6순위 봉사활동
@cabbage16 cabbage16 added the 개선 코드 리팩토링 label Sep 8, 2024
@cabbage16 cabbage16 self-assigned this Sep 8, 2024
cabbage16 added a commit that referenced this issue Sep 25, 2024
@cabbage16
docs(#132): 오탈자 수정
5a8afd9
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
refactor(#132): 사용되지 않는 import문 삭제, 의미에 맞지 않는 변수명 변경
4f59a91
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
feat(#132): 2차 전형 자동 합격 시 동점자 처리
0f9a897 
- FormRepositoryImpl에서 원서를 받아올 때, order by에 여러개의 기준을 둬서 1순위부터 6순위까지의 동점자 처리 기준 역할을 하도록 했습니다.
- 동점자 처리 기준중 3학년 1학기 성적이 있어서 Score에 3학년 1학기 성적을 추가했습니다.
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
perf(#132): 1차 전형 자동 합격 시 동점자 처리
7c6a88b 
- 입학요강에 따라 커트라인 점수를 저장해서, 이미 1차 정원이 다 찼더라도 동점자의 경우 추가로 합격을 시키도록 만들었어요.
- 전형별로 합격을 시키는 과정에서 똑같은 for 반복문이 너무 많이 사용되어서 함수형 인터페이스를 사용해서 메서드화 하여서 중복 코드를 줄였어요.
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
Merge branch 'develop' of https://github.com/Bamdoliro/marubase into …
91a50b2 
…perf/#132
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
perf(#132): 2차 전형 합격 처리 로직 중복 코드 처리
dc370f9 
- 전형별로 합격을 시키는 과정에서 똑같은 for 반복문이 너무 많이 사용되어서 함수형 인터페이스를 사용해서 메서드화 하여서 중복 코드를 줄였어요.
- 특별전형 인원 미달 시 일반전형 지원자로 충원하는 로직을 SelectFirstPassUseCase처럼 수정했어요.
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
test(#132): SelectSecondPassUseCaseTest AfterEach메서드 생성
f7a309a 
- SelectSecondPassUseCaseTest에서 테스트가 여러개가 되어서 하나의 테스트가 끝날때 마다 Repository를 초기화하는 메서드를 작성했어요.
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
refactor(#132): MissingTotalScoreException 에러 코드 수정
e474597 
- MissingTotalScoreException의 에러 코드가 409가 더 의미에 맞다고 생각하여 변경했어요.
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
fix(#132): 검정고시 원서 3학년 1학기 성적 산출 시 오류 수정
6cfe868 
- 검정고시는 특정 학기의 성적을 구할 수 없으므로 null로 하여서 오류를 방지했어요.
- 입학요강에 따라 검정고시 원서의 경우 동점자 처리 중 3학년 1학기 성적에서 일반 원서보다 우선순위가 낮아요. 따라서 FormRepositoryImpl에서 order by절에 3학년 1학기 성적에 nullsLast 옵션을 추가하여 명시적으로 수정했어요.
@cabbage16 cabbage16 linked a pull request Sep 26, 2024 that will close this issue
[개선] 1, 2차 합격자 결정 시 동점자 처리 #139
Merged
4 tasks
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
fix(#132): 검정고시 지원자의 과목 점수가 반올림 되지 않는 오류 해결
cb7ad89
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
test(#132): CalculateFormScoreServiceTest 3학년 1학기 성적 검증 추가
feef4dc
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
perf(#132): 공지사항에서 여러개의 파일 업로드 가능
6a65695 
- 공지사항에 최대 3개의 파일을 업로드할 수 있어요.
cabbage16 added a commit that referenced this issue Sep 26, 2024
@cabbage16
test(#132): 테스트 수정
6e63a41 
- 공지사항에 여러 파일을 업로드할 수 있도록 변경되면서 그에따라 테스트를 수정했어요.
cabbage16 added a commit that referenced this issue Sep 28, 2024
@cabbage16
test(#132): 공지사항 파일 업로드 response가 비어있는 오류 수정
f210f36 
- NoticeControllerTest에서 공지사항 파일 업로드 때, response가 비어있는 오류를 해결했어요.
cabbage16 added a commit that referenced this issue Sep 28, 2024
@cabbage16
refactor(#132): stream 부분 코드 컨벤션 준수
b1f4dd3
cabbage16 added a commit that referenced this issue Oct 4, 2024
@cabbage16
perf(#132): spring actuator 라이브러리로 health check api 대체
d35d487 
- health check api 를 위한 컨트롤러인 HealthCheckController 를 삭제했어요.
- 대신 spring actuator 라이브러리를 추가하여 컨트롤러 없이 health check api 를 만들었어요.
jyj1289 added a commit that referenced this issue Oct 7, 2024
@jyj1289
Merge pull request #139 from Bamdoliro/perf/#132
d207356 
[개선] 1, 2차 합격자 결정 시 동점자 처리
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cabbage16 cabbage16

Labels
개선 코드 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[개선] 1, 2차 합격자 결정 시 동점자 처리 Bamdoliro/marubase
1 participant
@cabbage16