Skip to content
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

[클린코드 6기 정민지] 로또 미션 STEP 2 #284

Open
wants to merge 4 commits into
base: jungminji0215
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 12 additions & 7 deletions REQUIREMENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,24 @@

### 로또

- [ ] 로또 1장의 가격은 1000원이다.
- [x] 로또 1장의 가격은 1000원이다.
- [x] 로또 번호는 1~45 사이의 정수여야 한다.
- [x] 로또 번호는 6개여야 한다.
- [x] 로또 번호는 중복되면 안 된다.
- [ ] 보너스 번호는 당첨 번호와 중복될 수 없다.
- [x] 보너스 번호는 당첨 번호와 중복될 수 없다.

### 로또 게임

- [ ] 구입 금액만큼 로또 발행한다.
- [ ] 수익률 계산
- [ ] 당첨은 1등부터 5등까지 있다.
- [x] 구입 금액만큼 로또 발행한다.
- [x] 수익률 계산
- [x] 당첨은 1등부터 5등까지 있다.
- [x] 사용자가 잘못된 값을 입력한 경우 throw문을 사용해 예외를 발생시키고, 에러 메시지를 출력 후 그 부분부터 입력을 다시 받는다.

### 입출력

- [ ] 당첨 번호화 보너스 번호를 입력할 수 있다.
- [ ] 수익룰 출력
- [x] 당첨 번호화 보너스 번호를 입력할 수 있다.
- [x] 수익룰 출력
- [ ] 로또는 오름차순으로 정렬하여 보여준다.
- [x] 당첨 통계를 출력한 뒤에는 재시작/종료 여부를 입력받는다.
- [x] 재시작할 경우 구입 금액 입력부터 게임 다시 시작
- [x] 종료하는 경우 그대로 프로그램 종료
4 changes: 1 addition & 3 deletions src/js/domain/LottoMachine.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ class LottoMachine {

generateLottoNumber() {
return new Lotto(
Array(LottoMachine.LOTTO_LENGTH)
.fill()
.map(() => this.#generateRandomNumbers())
Array(LottoMachine.LOTTO_LENGTH).fill().map(this.#generateRandomNumbers)
);
}

Expand Down
20 changes: 12 additions & 8 deletions src/js/domain/StatisticsLotto.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,11 @@ const PRIZES = {
};

const PRIZE_MAP = [
{ when: ({ hit, bonus }) => hit === 6, match: PRIZES.FIRST },
{ when: ({ hit, bonus }) => hit === 5 && bonus, match: PRIZES.SECOND },
{ when: ({ hit, bonus }) => hit === 5 && !bonus, match: PRIZES.THIRD },
{ when: ({ hit, bonus }) => hit === 4, match: PRIZES.FOURTH },
{ when: ({ hit, bonus }) => hit === 3, match: PRIZES.FIFTH },
{ when: ({ hit, bonus }) => hit === 6, match: "FIRST" },
{ when: ({ hit, bonus }) => hit === 5 && bonus, match: "SECOND" },
{ when: ({ hit, bonus }) => hit === 5 && !bonus, match: "THIRD" },
{ when: ({ hit, bonus }) => hit === 4, match: "FOURTH" },
{ when: ({ hit, bonus }) => hit === 3, match: "FIFTH" },
];

class StatisticsLotto {
Expand All @@ -26,12 +26,10 @@ class StatisticsLotto {
const prize = PRIZE_MAP.find(({ when }) => when({ hit, bonus }));

if (prize) {
prize.match.count++;
this.#prizes[prize.match].count++;
}

this.#calculateReceiveMoney(this.#prizes);

return prize;
}

#calculateReceiveMoney(prizes) {
Expand All @@ -50,6 +48,12 @@ class StatisticsLotto {
getPrizes() {
return this.#prizes;
}

resetCounts() {
Object.values(this.#prizes).forEach((prize) => {
prize.count = 0;
});
}
}

export { StatisticsLotto };
12 changes: 6 additions & 6 deletions src/js/view/output.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,27 +3,27 @@ export function printCount(count) {
}

export function printLottoNumber(lottoNumbers) {
console.log(lottoNumbers);
console.log(lottoNumbers.sort((a, b) => a - b));
}

export function printStatisticsLotto(prizes) {
console.log("\n당첨 통계");
console.log("--------------------");

console.log(
`${prizes.FIFTH.match}개 일치 (5,000원) - ${prizes.FIFTH.count}개`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5000같은 것도 prizes 객체에서 참조해오면 좋을 것 같네요

`${prizes.FIFTH.count}개 일치 (5,000원) - ${prizes.FIFTH.count}개`
);
console.log(
`${prizes.FOURTH.match}개 일치 (50,000원) - ${prizes.FOURTH.count}개`
`${prizes.FOURTH.count}개 일치 (50,000원) - ${prizes.FOURTH.count}개`
);
console.log(
`${prizes.THIRD.match}개 일치 (1,500,000원) - ${prizes.THIRD.count}개`
`${prizes.THIRD.count}개 일치 (1,500,000원) - ${prizes.THIRD.count}개`
);
console.log(
`${prizes.SECOND.match}개 일치, 보너스 볼 일치 (30,000,000원) - ${prizes.SECOND.count}개`
`${prizes.SECOND.count}개 일치, 보너스 볼 일치 (30,000,000원) - ${prizes.SECOND.count}개`
);
console.log(
`${prizes.FIRST.match}개 일치 (2,000,000,000원) - ${prizes.FIRST.count}개`
`${prizes.FIRST.count}개 일치 (2,000,000,000원) - ${prizes.FIRST.count}개`
);
}

Expand Down
47 changes: 36 additions & 11 deletions src/step-console-index.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,26 +10,36 @@ import { WinningLotto } from "./js/domain/WinningLotto.js";
import { StatisticsLotto } from "./js/domain/StatisticsLotto.js";

async function play() {
const purchase = await readLineAsync("> 구입금액을 입력해 주세요. ");
let lottoMachine;
let count;
let purchase;
let lottos = [];

const lottoMachine = new LottoMachine();
const count = lottoMachine.calculateLottoCount(purchase);
printCount(count);
while (true) {
try {
purchase = await readLineAsync("> 구입금액을 입력해 주세요. ");
lottoMachine = new LottoMachine();
count = lottoMachine.calculateLottoCount(purchase);
printCount(count);

const lottos = Array.from({ length: count }, () => {
const lotto = lottoMachine.generateLottoNumber();
const lottoNumbers = lotto.getLottoNumbers();
printLottoNumber(lottoNumbers);
return lottoNumbers;
});
lottos = Array.from({ length: count }, () => {
const lotto = lottoMachine.generateLottoNumber();
const lottoNumbers = lotto.getLottoNumbers();
printLottoNumber(lottoNumbers);
return lottoNumbers;
});

break;
} catch (error) {
console.error("오류가 발생했습니다: ", error);
}
Comment on lines +19 to +35

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

질문, 입력값 변환, 에러 출력등의 코드를 함수로 분리하면 try/catch 구문과 같이 가독성을 해치는 코드들이 추상화되어 가독성이 개선될 것 같습니다. 다음 예시 코드를 보고 비슷하게 구현해봐도 좋을 것 같아요!

const prompt = ({
  query, // 질문
  validate, // 입력값 검증
  transform // 입력값 변환
}) => {
    try {
      // 질문
      // 입력값 검증
      // 입력값 변환 및 반환
    } catch(e) {
      // 에러 출력
    }
  }

// 질문 프롬프트 로직을 추상화한 예시
const lottos = prompt({
 query:  "> 구입금액을 입력해 주세요.",
 validate: LottoMachine.isValidPurchaseAmount,
 transform: (answer) => {
    lottoMachine = new LottoMachine();
    count = lottoMachine.calculateLottoCount(answer);
    printCount(count);
  
    lottos = Array.from({ length: count }, () => {
      const lotto = lottoMachine.generateLottoNumber();
      const lottoNumbers = lotto.getLottoNumbers();
      printLottoNumber(lottoNumbers);
      return lottoNumbers;
    });
 }
})

}
const winningLottoNumbers = await readLineAsync(
"\n> 당첨 번호를 입력해 주세요. "
);

const bonusNumber = await readLineAsync("\n> 보너스 번호를 입력해 주세요. ");

// when
const winningLotto = new WinningLotto(winningLottoNumbers, bonusNumber);

const statisticsLotto = new StatisticsLotto();
Expand All @@ -45,6 +55,21 @@ async function play() {
// 총 수익률 출력
const rateOfReturn = statisticsLotto.calculateRateOfReturn(purchase);
printRateOfReturn(rateOfReturn);

await reply(statisticsLotto);
}

async function reply(statisticsLotto) {
const replyAnswer = await readLineAsync("\n> 다시 시작하시겠습니까? (y/n)\n");

if (isReply(replyAnswer)) {
statisticsLotto.resetCounts();
await play();
}
}

function isReply(replyAnswer) {
return replyAnswer === "y";
}

play();