-
Notifications
You must be signed in to change notification settings - Fork 192
[로또] 최권진 미션 제출합니다. #161
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
base: main
Are you sure you want to change the base?
[로또] 최권진 미션 제출합니다. #161
Conversation
- 구매금액에 맞는 로또를 발행하는 메서드 구현 - 로또를 오름차순으로 정렬하는 메서드 구현
- constructor에서 로또 번호 오름차순 정렬 - generateLottos: 여러 개 로또 생성 - getNumbers: 로또 번호 반환(private라 해당 메서드 필요) - compareWithWinningNumbers: 당첨 번호와 로또 비교
- App에서 로또 생성, 번호 정렬, 비교 로직 제거 - Lotto.generateLottos 및 Lotto.compareWithWinningNumbers 활용
- 구매 금액이 숫자가 아닌 경우 - 구매 금액이 1,000원 미만인 경우 - 구매 금액이 1,000원 단위가 아닌 경우
- 번호가 숫자가 아닌 경우 - 번호가 1 ~ 45 범위가 아닌 경우 - 번호가 6개가 아닌 경우 - 번호가 중복된 경우
- 번호가 숫자가 아닌 경우 - 번호가 1 ~ 45 범위가 아닌 경우 - 번호가 당첨번호와 중복되는 경우
- 통계 관련 로직 분리 - App 클래스 내 불필요한 메서드 및 변수 제거 - LottoResult와 App간 연동 로직 수정
66b6ab0 to
79ff6f6
Compare
manNomi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드가 원체 깔끔해서 제가 피드백을 많이 못드린것 같네요 ..
그러다 보니 조금 과한 부분에 대해서 피드백을 드리게 되어서 이것들은 고민하고 반영하셔도 좋을것 같습니다 3주차 고생하셨습니다!
| - App에 있는 로또 관련 로직 로또 클래스에 옮기기 | ||
| - App에 있는 입출력 관련 로직 view로 분리하기 | ||
|
|
||
| - 에러 처리 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
애러처리 너무 상세한데요???
이거 복사해서 제 리드미에 몰래 넣고 싶은걸요
| BONUS_NUMBER_RANGE: '[ERROR] 보너스 번호는 1 ~ 45 범위로 입력해 주세요.', | ||
| BONUS_NUMBER_DUPLICATE: | ||
| '[ERROR] 보너스 번호는 당첨 번호와 중복될 수 없습니다.', | ||
| }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좀 과하긴 하지만 object.freez 사용과
[ERROR] -> ERROR_PREFIX :"[ERROR]" 또는
export default class CustomError extends Error{}
같은 방법모두 고려해보시면 좋을것 같아요!
기능 요구 사항
간단한 로또 발매기를 구현한다.
Error를 발생시키고 해당 메시지를 출력한 다음 해당 지점부터 다시 입력을 받는다.입출력 요구 사항
입력
출력
실행 결과 예시
구입금액을 입력해 주세요. 8000 8개를 구매했습니다. [8, 21, 23, 41, 42, 43] [3, 5, 11, 16, 32, 38] [7, 11, 16, 35, 36, 44] [1, 8, 11, 31, 41, 42] [13, 14, 16, 38, 42, 45] [7, 11, 30, 40, 42, 43] [2, 13, 22, 32, 38, 45] [1, 3, 5, 14, 22, 45] 당첨 번호를 입력해 주세요. 1,2,3,4,5,6 보너스 번호를 입력해 주세요. 7 당첨 통계 --- 3개 일치 (5,000원) - 1개 4개 일치 (50,000원) - 0개 5개 일치 (1,500,000원) - 0개 5개 일치, 보너스 볼 일치 (30,000,000원) - 0개 6개 일치 (2,000,000,000원) - 0개 총 수익률은 62.5%입니다.추가 요구사항
구현할 기능 목록
로또 구입 금액 입력받기
로또 구입 갯수 출력하기
로또 구입 내역 출력하기
당첨 번호 입력 받기
보너스 번호 입력 받기
당첨 통계 출력하기
리팩토링
에러 처리