[로또 저장소] 임현우 미션 제출합니다. #393
Open
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
- 사용자 입력을 위한 메시지(USER_PROMPT_MESSAGES) 추가 - 확인 메시지(CONFIRMATION_MESSAGES) 추가 - 에러 메시지(ERROR_MESSAGES) 추가 - 등수에 따른 상금 정보(PRIZE_AMOUNTS) 추가
- printMessage와 promptUserInput 함수로 콘솔 관련 기능 모듈화
- 구입 금액이 숫자가 아닌 경우에 대한 에러 메시지 추가 - INVALID_PURCHASE_AMOUNT_NOT_NUMBER - 구입 금액이 티켓 단위로 나누어 떨어지지 않는 경우의 에러 메시지 네이밍 수정 - INVALID_PURCHASE_AMOUNT_NOT_TICKET_UNIT
- 입력받은 금액이 숫자인지 확인하는 로직 추가 - isPromptAmountNumber - 구매 금액이 로또 티켓 가격 단위로 나누어지는지 확인하는 로직 추가 - isValidTicketUnit - 전체 유효성 검증을 위한 함수 추가 - validateAmount
- 사용자에게 구매 금액 입력을 요청하는 함수 추가 - promptUserAmount - 구매 금액을 입력받고 유효성을 확인하고 반환하는 함수 추가 - getPurchaseAmount - 유효하지 않은 입력의 경우 에러 메시지를 출력하고 재입력 요청
- 로또 구입 금액 입력 및 1,000원 단위 검증 기능을 완료
- 공백 입력 시 발생하는 에러 메시지 추가 - INVALID_EMPTY_INPUT - 공통 유효성 검사 파일 common.js 추가 - 공백 제거 후 빈 값 체크 함수 추가 - trimInputAndCheckEmpty - purchase-amount.js에서 입력 값을 공백 제거 후 사용하도록 개선
- 올바른 금액 입력 시 정상적으로 처리되는지 확인하는 테스트 추가 - 잘못된 입력 값에 대해 에러 메시지가 발생하는지 확인하는 테스트 추가
- trim이 작동하지 않던 문제 해결: 입력 값을 문자열로 변환 후 처리 - 문자열로 변환한 후 trim 적용하여 공백 제거 및 유효성 검사 개선 - 숫자 여부 검증 후 최종적으로 Number로 변환
- constants.js 파일로 상수 통합 (기존 messages.js 파일 변경) - 로또 티켓 가격 상수를 constants.js 파일로 이동 (LOTTO_TICKET_PRICE) - 공통 상수 관리로 코드 일관성과 재사용성 향상
- 로또 번호의 개수, 중복 여부에 대한 에러 메시지 추가 - INVALID_LOTTO_NUMBER_COUNT - INVALID_DUPLICATE_NUMBERS
- 로또 번호 발행 로직 추가 - 중복되지 않는 6개의 숫자 생성 및 정렬 - 로또 리스트 생성 기능 추가 - 구매 금액에 따라 로또 개수 계산 - Lotto 클래스 수정 - 로또 번호 개수 및 중복 유효성 검사 추가가 - 로또 번호 반환 메서드 추가 (getNumbers)
- 구매한 로또 개수 출력 로직 추가 - PURCHASE_COMPLETE_MESSAGE 사용하여 로또 개수 출력 - 각 로또 번호 출력 기능 추가 - printLotto 함수를 통해 오름차순 정렬된 번호를 콘솔에 출력 - printLottoList 함수에서 모든 로또 번호를 순회하며 개별 출력 처리
- 발행된 로또의 개수가 입력된 금액에 비례하여 올바르게 계산되는지 테스트 - 각 로또 번호가 1 ~ 45 사이의 중복되지 않는 6개의 숫자로 구성되어 있는지 테스트 - 각 로또 번호가 오름차순으로 정렬되어 있는지 테스트
- MAX_PURCHASE_AMOUNT 상수 및 관련 에러 메시지 추가 - validateAmount 함수에 최대 구입 금액 검증 로직 추가
- 구입 금액이 10만 원을 초과할 경우 예외 처리 검증 테스트 추가
- 로또 번호의 최소 및 최대 값을 정의하는 LOTTO_NUMBER_RANGE 추가 - 당첨 번호의 형식에 맞지 않는 경우의 에러 메시지 추가 (INVALID_FORMAT) - 로또 번호 범위 검증 관련 에러 메시지를 포괄적으로 변경 (INVALID_LOTTO_NUMBER) - 보너스 번호 중복에 대한 에러 메시지 개선 (INVALID_BONUS_NUMBER) - 당첨 번호의 포맷을 검증하는 WINNING_NUMBER_FORMAT_REGEX 상수 추가
- 로또 번호의 오름차순 정렬 기능을 별도의 유틸 파일로 분리 - `sort-numbers.js` 파일 생성 - 다른 곳에서도 재사용할 수 있도록 모듈화
- 당첨 번호 유효성 검사 모듈화 (`validateWinningNumbers`) - 입력 형식 검증 (`isValidFormat`): 콤마로 구분된 숫자 6개인지 확인 - 숫자 범위 검증 (`validateNumberRange`): 로또 번호는 1~45 사이의 숫자여야 함 - 중복 숫자 검증 (`validateUniqueNumbers`): 중복된 번호가 없어야 함 - 당첨 번호 입력 및 처리 함수 추가 (`getWinningNumbers`) - 사용자 입력을 받아 유효성 검사 후 오름차순으로 정렬 - 잘못된 입력 시 에러 메시지 출력 후 재입력 요청
- validateNumberCount 함수 삭제하여 숫자 개수 유효성 검증을 통합 처리 - 파일명 'winning-number.js'에서 'winning-numbers.js'로 변경하여 파일명 통일
- 올바른 형식의 당첨 번호 입력 시 정상 처리되는지 확인하는 테스트 추가 - 잘못된 형식의 당첨 번호 입력 시 각 에러 메시지 발생 여부를 확인하는 테스트 추가
- 숫자 입력 유효성 검증을 위한 에러 메시지 상수 추가
- 보너스 번호 입력을 처리하는 함수 생성 (get-bouns-number.js) - 보너스 번호 유효성 검사 모듈 추가 (bouns-number.js) - 보너스 번호가 당첨 번호와 중복되지 않도록 검증하는 로직 포함
- '01'과 같은 입력값을 숫자 '1'로 처리하기 위해 입력 문자열을 map(Number)로 숫자 변환
- 반환 값의 형식 변경에 따른 테스트 케이스 수정 - 올바른 형식의 당첨 번호 입력 시 예상 결과를 문자열 배열에서 숫자 배열로 변경
- 숫자가 아닌 값을 입력했을 때 에러 메시지가 발생하는지 확인하는 테스트 추가 - 보너스 번호가 당첨 번호와 중복될 때 에러 메시지가 발생하는지 확인하는 테스트 추가 - 보너스 번호가 범위를 벗어난 경우 에러 메시지가 발생하는지 확인하는 테스트 추가
- validateNumber 함수에서 숫자가 아닌 값을 올바르게 처리하도록 로직 수정 - 기존 테스트에서 문자열 'a' 입력 시 올바른 에러 메시지가 발생하지 않는 문제 해결
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.
아... 시간이 없어서 미완성이지만 4주차를 하고 싶어 제출합니다.