[페어매칭관리 애플리케이션] 송인봉 미션 제출합니다. #108
Closed
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
- TODO:: switch-case 제거
* refactor: Delimiter 클래스 수정 및 사용 * refactor: 출력 문자열을 상수 클래스로 분리 * fix: 페어 매칭 시, 홀짝 구분이 제대로 이뤄지지 않는 이슈 해결 * feat: 파일 입출력 예외 처리 클래스 생성 * feat: 각 계층 별 DI 설정 및 DI 컨테이너 생성
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
기능 미구현
페어 매칭에서 "이미 페어를 맺은 크루와는 매칭이 될 수 없다"는 조건에 대한 기능을 구현하지 못했습니다.
소감
프로그램이 동작하도록 완성시키기는 했으나 많은 아쉬움이 남습니다. 기능을 완벽히 완성시키지 못했다는 것도 아쉽지만, 다급한 마음에 15줄 이상의 메소드 선언, 2 이하의 depth 등을 지키지 못해 스스로를 자책하고 있습니다. 그간 3주간의 프리코스에서 좋은 모습을 보여드리고자 오랜 시간을 들여 코드를 작성하여 과제를 제출했지만, 막상 최종 테스트에서 이런 모습을 보여 드려 죄송할 따름입니다.