eddie.kk #24
Open
eddie.kk #24
Conversation
This file contains 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
added 17 commits
December 20, 2022 13:23
- 컴퓨터 도메인 생성
- Set을 활용하여 중복 제거
- view 패키지로 분리하여 구현
- 비교 결과에 따라 낫싱, 스트라이크, 볼 정보를 생성하는 기능
- 스트라이크와 볼 카운트를 알고있어야 할 책임 존재 - 비교 결과에 따라 스트라이크/볼 카운트 증가 - 스트라이크와 볼 카운트에 따라 결과 description 생성
- 스트라이크/볼 증가 테스트 - 게임 결과 판별 테스트
- 수가 같은지 다른지 판별 - 수가 같은 자리에 있는지, 포함하고 있는지 판별 - 게임 종료(3 strike) 야부 판별
- 사용자의 수와 컴퓨터의 수를 비교하는 로직 테스트
- 내부적으로만 사용되는 메서드인 만큼 private으로 수정
- view로 부터 받은 데이터를 기반으로 domain에 작업을 위임하는 구조로 구현하고자 함
- 낫싱, 스트라이크 개수, 볼 개수 정보를 반환하도록 수정 - 개별 메소드 단위의 테스트도 추가
- 사용자 숫자 입력값 유효성 검증은 현 로직에서 이미 잘못된 입력값이 들어오더라도 모두 낫싱으로 판별되도록 되어있음
- Runnable을 활용하여 출력 메세지를 변경할 수 있도록 구현
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.