-
Notifications
You must be signed in to change notification settings - Fork 54
[행성 로또] 유인성 미션 제출합니다. #51
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
Open
inseong01
wants to merge
23
commits into
woowacourse-precourse:main
Choose a base branch
from
inseong01:inseong01
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
- 로또 번호 검증
- 넘길 데이터 구조 확인 - 출력 매서드 2개 테스트
- 기능 - 로또 번호 검증 - 로또 매칭 결과 key 반환 - 로또 관련 Const 작성
- 유틸리티 구현 기능 추가
- 로또 집계 초기 Map key 배열 제거 - [0] -> 0
- 로또 매칭 key 배열 기반 로또 당첨 통계 결과(Map) 반환
- 유틸리티 구현 기능 목록 추가 - 재입력
- action 비동기 실행 오류 발생 시 재실행 및 오류 메시지 출력
- 로또 모델 기능 추가 - 숫자 반환 - 유틸리티 구현 목록 추가 - 무작위 번호 생성 - 구입금액 비례 로또 생성
- 로또 번호 반환 - 오름차순 적용
- 현금 만큼 생성된 로또 배열 반환
- 예제 테스트 (1/2)
- 기능 테스트 성공
- 예외 테스트 실패
- 500j 입력 시 500으로 정상 숫자 반환 (parseInt 원인)
- InputView.askAmount 코드 수정 필요 (미션 제약으로 변경 불가)
- 15:36 예외 테스트 결함 공지, 테스트 중 예외 테스트 항목 제외
- [ERROR] 삭제
- 추가 - 중복 번호 검증 - 수정 - 로또 번호 자리수 제한 문구 수정
- 추가 - 1~30 이내 숫자 검증
- 기능 - 구현하지 않은 목록 정리 - 구현 목록 정리 - 도전 목록 - 도전했던, 도전할 목록 작성
- App 매서드 15 라인 이하 원칙 적용 - 사용자 로또 생성/출력, 로또 결과 출력 두 개의 매서드로 분리
- 도전 목록 -> 도전 과제 수정 - 작성하지 않은 도전 방향 "리팩토링" 작성 추가
- GenerateLottoBy - 로또 반환 코드 간략화 - CountMapKey -> GetLottoMatchResultMapOf - 함수명 수정 - Lotto - 매서드명 수
- 로또 Map key - LOTTO_RANK 상수화 적용
- "[ERROR]" 문구 삭제 - Validator 오류 발생 시 전달되는 문구 차이로 인한 오류
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.
javascript-planetlotto-precourse
공통 예외처리
출력 형식
사용자가 잘못된 값을 입력할 경우 "[ERROR]"로 시작하는 메시지 출력
재입력 방식
오류가 발생한 해당 지점부터 다시 입력
흐름
기능
입력 및 유효성 검사
로또 구입 금액 입력
로또 번호 입력
,으로 구분)보너스 번호 입력
출력
로또 개수 출력
출력 예시
출력 설명
당첨 통계 출력
출력 예시
출력 설명
당첨 결과를 기반으로 당첨 통계 출력
핵심 로직
로또
유틸리티 함수
도전 과제 "리팩토링"
설계
구현
제한 시간 내 구현했는지
테스트 전부 통과하는지
코드 품질
depth는 2를 넘지 않는지
라인이 15줄을 넘지 않는지
코드를 읽었을 때 이해가 되는지
테스트
테스트 코드를 통해 구현했는지
테스트 케이스가 다양한지