[로또] 이만재 미션 제출합니다.#4
Conversation
| static showLottoNumbers(lotto) { | ||
| Console.print(`[${lotto.join(', ')}]`); | ||
| } |
There was a problem hiding this comment.
로또 당첨번호를 보여줄 때 원본 배열이 아니라 재조합을 통해 출력했는데 원본 배열로 출력할 수 있을 것 같아서 리팩토링 하면서 같이 수정해보겠습니다!
| - [ ] ,가 맨 앞에 오는 경우 | ||
| - [ ] ,가 맨 뒤에 오는 경우 | ||
| - [ ] ,가 중복되서 입력 되는 경우 |
There was a problem hiding this comment.
이 부분을 정규식과 if문으로 검사를 하려고 생각해봤지만 동작이 되는 코드를 작성하기 위해 우선순위를 미뤄두었다가 구현하지는 못했습니다.
조건에 따른 검사를 할 때 정규식을 사용하면 되겠다는 생각이 먼저 드는 편인데 정규식을 사용해 조건을 검사하는 것이 복잡한 정규식일 경우 가독성이 안 좋지 않을까? 하는 생각에 정규식을 사용하는 것이 좋은 코드인지는아직 잘 모르겠습니다.
|
hyesungoh
left a comment
hyesungoh
left a comment
There was a problem hiding this comment.
-
클래스 내부 private 메서드일 수 있을 것들이 많이 보여요
-
각 클래스(모델)이 하는 역할이 명확하지 않은 느낌이 들어요
- 역할과 책임을 기준으로 더 고심해볼 수 있을까요?
-
많은 코드에서 클래스 내부 필드를 조작하고 사용하는데, 이에 따른 단점은 어떤 것이 생길까요?
전체적으로 패턴에 구현체를 끼워 맞춘 코드가 된 느낌입니다.
좋은 코드, 문제를 해결하는 코드가 되기 위해서는 어떻게 해야할까요?
| export const MONEY = Object.freeze({ | ||
| unit: 1000, | ||
| zero: 0, | ||
| blank: '', | ||
| }); |
There was a problem hiding this comment.
Object.freeze를 사용하신 이유에 대해서 기술해 주세요 ~
사용했을 때의 장점, 단점
There was a problem hiding this comment.
현재의 코드가 읽기 쉽고, 수정에 용이한 코드라 생각되시나요?
- 왜 그렇게 생각하시는 지 말씀해주세요
MVC 패턴이 해당 요구사항에 적합했다고 생각하시나요?
- 그렇다면 이유에 대해서 말씀해주세요
| async start() { | ||
| await this.initPurchaseAmount(); | ||
| this.showLottoNumbers(); | ||
| await this.initWinningNumbers(); | ||
| await this.initBonusNumber(); | ||
| this.showResult(); | ||
| } | ||
|
|
||
| async initPurchaseAmount() { | ||
| const inputUserMoney = await InputView.getInputMoney(); | ||
| this.purchase = new Purchase(inputUserMoney); | ||
| OutputView.showLottoCount(this.purchase.getQuantity()); | ||
| } | ||
|
|
||
| showLottoNumbers() { | ||
| this.lottos = LottoGenerator.generate(this.purchase.getQuantity()); | ||
| this.lottos.forEach((lotto) => { | ||
| OutputView.showLottoNumbers(lotto); | ||
| }); | ||
| } |
There was a problem hiding this comment.
start를 기준으로 흐름을 따라가다 보면 뇌에 기억하고 있어야 하는 것들이 계속 늘어나, 흐름을 파악하기 어려울 수 있을 거 같다고 생각되는데요.
두뇌가 한 번에 기억할 수 있는 것을 개선하기 위해서는 어떤 방법이 있을까요?
There was a problem hiding this comment.
이 클래스는 유틸성이 강한 것 같은데, 클래스로 만든 이유가 무엇인가요?
모든 것을 클래스로 만들면 어떤 단점이 생길까요?
| - [ ] ,가 맨 앞에 오는 경우 | ||
| - [ ] ,가 맨 뒤에 오는 경우 | ||
| - [ ] ,가 중복되서 입력 되는 경우 |
4 participants
궁금한 점
구현 실패한 부분