[자판기] 유재건 미션 제출합니다. #167
[자판기] 유재건 미션 제출합니다. #167
Conversation
| public int getAmount(){ | ||
| return this.amount; | ||
| } | ||
| public static Coin of(int amount){ |
There was a problem hiding this comment.
넵 시간 재면서 하다보니 리팩터링할 부분이 많은 것 같아요. 코멘트 주신 부분들 생각하면서 수정해볼게요 ~~ 코드리뷰 감사합니당
| } | ||
|
|
||
| private void changeValidation(int change) { | ||
| if (change % coins.stream() |
There was a problem hiding this comment.
if 구문에 stream이 같이 들어가니 읽기가 조금 어렵네요 ㅠㅠ
coins.stream()... 에 대한 결과값을 변수로 한번 분리하는게 좋아보여요
| return initCoin; | ||
| } | ||
|
|
||
| private void coinMaker(int change) { |
There was a problem hiding this comment.
해당 네이밍은 뭔가 클래스명처럼 보이네요..
createCoin과 같이 메소드 이름은 동사 혹은 동사구가 적합합니다.
참고 내용
| } | ||
|
|
||
|
|
||
| public String startChangePrint() { |
There was a problem hiding this comment.
toString 혹은 printChange같은 네이밍은 어떨까요
| changeAmount += tempCoin.getAmount(); | ||
| } | ||
| } | ||
| return; |
| return; | ||
| } | ||
|
|
||
| public String leftChangePrint(){ |
| StringBuilder print = new StringBuilder(); | ||
| print.append(CHANGE_MESSAGE.getMessage()); | ||
| for (Coin tempCoin : leftCoins.keySet()) { | ||
| if(leftCoins.get(tempCoin) >0){ |
There was a problem hiding this comment.
if(leftCoins.get(tempCoin) >0)위 구문이
if(!isExhusted(tempCoin))이렇게 작성되면 보다 읽기가 수월하다고 생각하기 때문에 저는 보통 if 내부에 존재하는 조건을 문장으로풀어쓰기위해 method로 한번 더 분리하곤 합니다.
| this.count = count; | ||
| } | ||
|
|
||
| private void productValidate(int count) { |
| this.products = products; | ||
| } | ||
|
|
||
| public void productExist(String productName) { |
There was a problem hiding this comment.
getProduct라는 네이밍은 어떨까요
내부에서 검증하는 것 까지 외부에서 굳이 알 필요가 없다고 생각되네요
| public int consumeProduct(String productName) { | ||
| for (Product o : products) { | ||
| if (o.getName().equals(productName)) { | ||
| if (o.getCount() == 0) { |
There was a problem hiding this comment.
3depth가 넘어갔네요ㅠㅠ method를 분리하는 방향을 생각해보시는게 어떨까요
| public void saveLeftCoin(int amount){ | ||
| if(amount >= startChange){ | ||
| leftCoins = haveCoin; | ||
| return; | ||
| } |
There was a problem hiding this comment.
leftCoin이라는 메소드 명의 의미를 명확하게 파악하기 어려울 거 같아요!
| public Product(String name, int price, int count) { | ||
| productValidate(count); | ||
| this.name = name; | ||
| this.price = price; | ||
| this.count = count; | ||
| } |
| public int sell(){ | ||
| count --; | ||
| return price; | ||
| } |
No description provided.