자판기 미션 리뷰 부탁드립니다!! by cwjjjjjj · Pull Request #193 · woowacourse/java-vendingmachine-precourse

cwjjjjjj · 2024-11-26T17:23:33Z

우선 늦어서 죄송합니다 ㅎㅎ ㅠㅠㅜ
5시간 동안 구현해보았는데 시간이 부족해서 미완성한 것들이 있습니다 -> 잔돈 거슬러주기, 오류 처리
코드리뷰하고 이 부분도 시간날 때 이어서 작성해보겠습니다!

Jihyun3478

주요 로직을 어떻게 코드로 풀어낼지에 대한 고민이 깊게 보였던 것 같아요! 테스트코드도 작성해주시면 더 좋을 것 같아요 고생하셨습니다!!:)☺️

Jihyun3478 · 2024-11-27T13:46:57Z

src/main/java/vendingmachine/Application.java

+        CoinController coinController = new CoinController();
+        ProductController productController = new ProductController();
+        MoneyController moneyController = new MoneyController();
+        OrderController orderController = new OrderController();


Service나 Domain 계층에 책임을 위임하지 않고, Controller에 구현하신 우진님만의 의도가 궁금합니다!

Jihyun3478 · 2024-11-27T13:47:54Z

src/main/java/vendingmachine/Application.java

+        MoneyController moneyController = new MoneyController();
+        OrderController orderController = new OrderController();
+
+        int[] coins = coinController.initMoney();


Collection이 아닌 배열을 사용하신 이유가 궁금합니다!

Jihyun3478 · 2024-11-27T13:55:47Z

src/main/java/vendingmachine/controller/ProductController.java

+    public static boolean validateEmptyStock(Products products) {
+        //재고가 하나라도 있으면 진행 가능
+        return products.getProducts().stream().anyMatch(product -> product.getStock() > 0);
+    }


재고 확인 검증을 Domain이 아닌 Controller에서 하신 이유가 궁금합니다!

Jihyun3478 · 2024-11-27T14:03:36Z

src/main/java/vendingmachine/model/Products.java

+    public static Product getProductByName(String name) {
+        return products.stream().filter(product -> product.getName().equals(name)).findFirst()
+            .orElse(null);
+    }


상품 이름으로 Product를 반환하는 메서드라면, findAny()를 사용하는게 더 좋지 않을까요?? 우진님의 의견은 어떠신지 궁금해요!

Jihyun3478 · 2024-11-27T14:04:37Z

src/main/java/vendingmachine/view/InputView.java

+    }
+
+    public static String input() {
+        return readLine();


항상 Console.readLint()을 사용해왔는데, 해당 라인처럼 더 간결하게 표현할 수 있어서 좋은 것 같아요!👍

Jihyun3478 · 2024-11-27T14:06:00Z

src/main/java/vendingmachine/controller/CoinController.java

+        //보유 금액만큼 동전 생성 후 출력
+        printCoins(coins);
+        return coins;
+    }


주석을 남겨두신 이유가 궁금해요!🧐

Jihyun3478 · 2024-11-27T14:09:10Z

src/main/java/vendingmachine/controller/MoneyController.java

+
+    public int receiveMoney() {
+        printPurchasingMoneyMessage();
+        return inputInteger();


Console과 같은 라이브러리뿐만 아니라, InputView와 같은 클래스까지 모두 static import 하신 의도가 궁금합니다!
InputView를 명시해주는 편이 해당 클래스 내에서 역할 및 관계를 명확히 이해할 수 있다고 생각하는데 우진님의 의견은 어떠신가요??

Jihyun3478 · 2024-11-27T14:10:46Z

src/main/java/vendingmachine/controller/OrderController.java

+
+    public void giveChange() {
+
+    }


이 부분은 아직 작성되지 않은건가요?!

songsunkook

시간이 부족한 부분이 많이 아쉽네요 🥲
고생하셨습니다!

songsunkook · 2024-11-29T01:12:50Z

README.md

다음번에는 시간을 재고 해보는 것도 좋을 것 같아요!

songsunkook · 2024-11-29T01:13:29Z

src/main/java/vendingmachine/Application.java

+        CoinController coinController = new CoinController();
+        ProductController productController = new ProductController();
+        MoneyController moneyController = new MoneyController();
+        OrderController orderController = new OrderController();


컨트롤러가 많네요..
구현과 동시에 리팩토링을 추구하신 건가요??

songsunkook · 2024-11-29T01:16:40Z

src/main/java/vendingmachine/controller/MoneyController.java

+public class MoneyController {
+
+    public int receiveMoney() {
+        printPurchasingMoneyMessage();
+        return inputInteger();
+    }
+}


컨트롤러 분리가 조금 과한 감이 있는 것 같아요. 코드가 많이 길어지면 그 때 가서 분리해도 좋을 것 같습니다!

songsunkook · 2024-11-29T01:17:18Z

src/main/java/vendingmachine/controller/CoinController.java

+            //동전 합이 total이 될 때까지 loop
+            //0부터 size까지를 왔다갔다할 인덱스
+            int coinPrice = coinType[index].getAmount();
+            //현재 index에 해당하는 동전의 액면가
+            int possibleCoin = total / coinPrice;
+            //현재 index의 동전의 최대 개수
+            List<Integer> numberRange = makeNumberRange(possibleCoin);
+            //pickNumberInList의 매개변수 타입에 맞게 0부터 possibleCoin까지의 모든 수를 넣은 List
+            if (possibleCoin > 0) {
+                //가능한 코인 개수가 있으면
+                int pickedNumber = pickNumberInList(numberRange);
+                //랜덤 돌려서
+                coins[index] += pickedNumber;
+                //코인 개수에 추가
+                total -= coinPrice * pickedNumber;
+                //while문 조건에 반영하기 위해 동전개수 * 액면가를 total에서 제해줌


주석은 설계 과정에서 스스로 쉽게 알아보기 위해 작성한 건가요?!

songsunkook · 2024-11-29T01:26:59Z

src/main/java/vendingmachine/view/OutputView.java

+    private static final String HAVING_MONEY_MESSAGE = "자판기가 보유하고 있는 금액을 입력해 주세요.";
+    private static final String HAVING_COIN_MESSAGE = "\n자판기가 보유한 동전";
+    private static final String ORDER_MESSAGE = "\n상품명과 가격, 수량을 입력해 주세요.";
+    private static final String PURCHASING_MONEY_MESSAGE = "\n투입 금액을 입력해 주세요.";
+    private static final String PURCHASING_MONEY = "\n투입 금액: %d원";
+    private static final String PURCHASING_ITEM_MASSAGE = "구매할 상품명을 입력해 주세요.";
+    private static final String printForm = "%d원 -  %d개";


운영체제에 의존적이지 않은 줄바꿈 방법을 사용하면 더 좋을 것 같아요!

songsunkook · 2024-11-29T01:27:28Z

src/main/java/vendingmachine/view/OutputView.java

+        for (int i = 0; i < coins.length; i++) {
+            System.out.println(String.format(printForm, Coin.values()[i].getAmount(), coins[i]));
+        }


stream을 활용할 수 있을 것 같아요!

cwjjjjjj added 27 commits November 26, 2024 05:55

feat: 리드미 수정

b3834d0

feat: getter 추가

d6032eb

feat: CoinController 추가

58ddf68

feat: InputView, OutputView 추가

4b44d91

feat: Coins 추가

d30e905

feat: 보유 금액만큼 동전 생성 로직 구현

f23db6c

feat: Coins 삭제

3e26a3d

remove: Coins 삭제

e2e0cb8

Merge remote-tracking branch 'origin/main'

7e5c14b

feat: 보유 코인 출력 메서드 추가

9fa0c91

feat: main에서 controller 인스턴스 초기화 및 보유 동전 설정

70d42b8

feat: 프로그램 흐름 조정하는 코드 추가

e9ab128

feat: return 타입 수정

d477652

feat: int 변환 메소드 추가

9c838df

feat: 출력 메소드 추가

0465174

feat: 투입 금액 입력 클래스

a09f743

feat: 재고 상품의 데이터 담는 클래스

60ffbb1

feat: 재고 상품 집합 관리하는 클래스

027e16b

feat: 재고 상품 집합 controller

2e708f5

feat: 주문 관련 controller

bb6a233

feat: 일정 조건 하에서 주문 계속 반복되는 로직 구현

89b32e2

feat: 투입금액과 안내문구 출력 메서드

565f565

feat: 재고 여부 확인, 가격 중 최소값 확인 메서드

81e10ce

Merge remote-tracking branch 'origin/main'

02e8e9d

feat: static 키워드 추가

02adde9

feat: 리드미 수정

bc44def

Merge remote-tracking branch 'origin/main'

cb8df9b

Jihyun3478 reviewed Nov 27, 2024

View reviewed changes

songsunkook reviewed Nov 29, 2024

View reviewed changes

Conversation

cwjjjjjj commented Nov 26, 2024

Uh oh!

Jihyun3478 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

songsunkook left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Jihyun3478 left a comment •

edited

Loading