[4기-황준호] View 패키지 생성 및 구현 #739
Conversation
|
변경된 파일은 다음과 같습니다
|
| if (voucherType == 1) { | ||
| outputView.print("할인할 금액을 입력하세요."); | ||
| } | ||
| if (voucherType == 2) { | ||
| outputView.print("할인율을 입력하세요 (1% ~ 100%)"); | ||
| } |
| public void printStartMessage() { | ||
| outputView.print(START_MESSAGE); | ||
| } |
There was a problem hiding this comment.
START_MESSAGE는 printStartMessage에만 쓰이고 있는데, 굳이 변수로 뺴야할 이유가 있을까요?
There was a problem hiding this comment.
그럼 메서드 안에 그냥 놔두는게 나을까요?
매직넘버 상수로 분리하는 느낌으로 분리했습니다!
| if (voucherType == 2) { | ||
| outputView.print("할인율을 입력하세요 (1% ~ 100%)"); | ||
| } | ||
| return StringUtil.convertStringToInt(inputView.read()); |
There was a problem hiding this comment.
| return StringUtil.convertStringToInt(inputView.read()); | |
| return StringUtil.convertStringToInt(read()); |
인창님 죄송한데 도저히 감이 잡히질 않습니다 ㅠㅠ |
| int voucherType = console.readVoucherTypeToCreate(); | ||
| int discountAmount = console.readVoucherDiscountAmount(voucherType); | ||
| int voucherAmount = console.readVoucherAmountToCreate(); |
There was a problem hiding this comment.
저는 값을 입력 받는 것이 console의 역할이라고 생각하고,
이 값들이 어떤 의미를 가지는지는 알 필요 없다고 생각해요.
VoucherType, VoucherDiscountAmount, VoucherAmount의 값은 console이 아닌 해당 객체를 사용하거나 검증할 책임이 있는 객체에서 진행하는게 맞다고 생각합니다.
📌 과제 설명
👩💻 요구 사항과 구현 내용
✅ 피드백 반영사항
✅ PR 포인트
🟣 궁금한 점
지금은 어떻게 관리해야할지 판단이 서지 않아 상수로 분리 시키지 않은 것도 있습니다.