[2주차] 객체지향 코드 연습(seryn6911-hub) #45

seryn6911-hub · 2025-09-29T02:36:36Z

No description provided.

0702Yoon

Controller 26번줄 까지 코드리뷰한 상태입니다.

추석동안 고생하셨어요!

0702Yoon · 2025-10-11T06:29:32Z

src/main/java/lotto/view/InputReader.java

+    default String readLine() {
+        return null;
+    }


이 default의 의미는 무엇인가요?

0702Yoon · 2025-10-12T03:47:55Z

src/main/java/lotto/view/View.java

+    public View() {
+        this.reader = new Scan(); // Scan은 InputReader 구현체
+    }


헌우님이 올리신 appConfig를 읽고 적용해보세요!
인터페이스 활용은 잘하신 것 같아요.

0702Yoon · 2025-10-12T03:49:23Z

src/main/java/lotto/view/View.java

+            // 음수 또는 0 체크
+            if (purchasePrice <= 0) {
+                throw new IllegalArgumentException(ErrorMessage.INVALID_AMOUNT.getMessage());
+            }
+
+            // 1000원 단위 체크
+            if (purchasePrice % 1000 != 0) {
+                throw new IllegalArgumentException(ErrorMessage.INVALID_AMOUNT.getMessage());
+            }


값을 검사하는 객체가 있으면 더 좋을 것 같아요

0702Yoon · 2025-10-12T03:50:25Z

src/main/java/lotto/model/Lotto_purchaser.java

+import java.util.List;
+
+public class Lotto_purchaser {
+    private final int price;


이거는 언제 쓰시는 거죠??

0702Yoon · 2025-10-12T03:51:18Z

src/main/java/lotto/model/Lotto_purchaser.java

+        List<Lotto> lottos = new ArrayList<>();
+        int ticketCount = costToTicketNUM();
+        for (int i = 0; i < ticketCount; i++) {
+            List<Integer> numbers = Randoms.pickUniqueNumbersInRange(1, 45, 6);


매직넘버 적용하면 좋겠네요

0702Yoon · 2025-10-12T03:52:42Z

src/main/java/lotto/model/Lotto_purchaser.java

+            List<Integer> numbers = Randoms.pickUniqueNumbersInRange(1, 45, 6);
+            lottos.add(new Lotto(numbers));
+        }
+        this.lottos = lottos;


이 코드의 의미는 뭔가요??

0702Yoon · 2025-10-12T03:52:57Z

src/main/java/lotto/model/Lotto_purchaser.java

+
+
+
+
+


코드 컨벤션에 대해서 알아보세요!

0702Yoon · 2025-10-12T03:53:56Z

src/main/java/lotto/model/Lotto_purchaser.java

+            lottos.add(new Lotto(numbers));
+        }
+        this.lottos = lottos;
+        List<List<Integer>> lottoList = new ArrayList<>();


List가 아니라 List<List>로 하신 이유가 뭔가요??

baaamk

고생하셨습니다!

MVC 에 대해서 더 공부해보시면 좋을 것 같습니다.

AppConfig 적극 활용해주세요!

baaamk · 2025-10-12T13:25:41Z

src/main/java/lotto/enums/Rank.java

+    SIX,              // 1등
+    FIVE_AND_BONUS,   // 2등
+    FIVE,             // 3등
+    FOUR,             // 4등
+    THREE            // 5등
+                  // 꽝


이렇게 하면 협업하는 입장에서 좋을지 생각해보셔야 할 것 같습니다ㅏ

baaamk · 2025-10-12T13:26:09Z

src/main/java/lotto/Application.java

+        try {
+            View view = new View();
+            Contoller controller = new Contoller(view);
+            controller.startController();
+        } catch (IllegalArgumentException e) {
+            // 예외 발생 시 조용히 종료 - 추가 출력 없음


AppConfig 사용해보세용
"AppConfig 순수자바" 키워드로 검색하신 후 공부하면 좋을 것 같아요

baaamk · 2025-10-12T13:29:08Z

src/main/java/lotto/contorller/Contoller.java

컨트롤러가 너무 담당하는 일이 많고, 협력 객체를 직접 생성하고 있기 때문에 결합도가 너무 높습니다.
컨트롤러는 단순히 view와 model을 연결해 주는 곳이라고 생각하시면 됩니다. 유효성 검사가 컨트롤러에 있을 필요가 있을지 생각해 보시는게 좋을 것 같습니다!
AppConfig를 사용하셔서 담당하는 일을 줄이고, DI(생성자 주입)를 사용하셔보시기 바랍니다!

baaamk · 2025-10-12T13:29:17Z

src/main/java/lotto/contorller/Contoller.java

+            throw e;
+        }
+    }
+}


https://velog.io/@jangws/EOL%EC%9D%84-%EB%84%A3%EC%96%B4%EC%95%BC-%ED%95%98%EB%8A%94-%EC%9D%B4%EC%9C%A0%EC%99%80-git%EC%97%90%EC%84%9C-CRLF-EOL-%EC%B0%A8%EC%9D%B4%EB%A1%9C-%EC%9D%B8%ED%95%9C-%EB%AC%B8%EC%A0%9C-%ED%95%B4%EA%B2%B0-%EB%B0%A9%EB%B2%95

EOL이슈 참고해주세용

baaamk · 2025-10-12T13:29:53Z

src/main/java/lotto/contorller/RankSystem.java

+
+
+    public Map<Rank, Integer> calculateWinningStatistics() {
+        Map<Rank, Integer> result = new EnumMap<>(Rank.class);


EnumMap을 쓰신 이유가 있으실까요?

baaamk · 2025-10-12T13:41:30Z

src/main/java/lotto/model/Lotto.java

+    private void validate(List<Integer> numbers) {
+        if (numbers.size() != 6) {
+            throw new IllegalArgumentException("로또 번호가 많거나 적음");
+        }
+
+        if (numbers.size() != new HashSet<>(numbers).size()) {
+            throw new IllegalArgumentException("로또 번호는 중복될 수 없습니다.");
+        }
+    }


에러 enum을 만드셨으니 이건 수정해주세요!

baaamk · 2025-10-12T13:42:24Z

src/main/java/lotto/model/Lotto_purchaser.java

+import java.util.ArrayList;
+import java.util.List;
+
+public class Lotto_purchaser {


카멜 케이스로 통일해주세요

baaamk · 2025-10-12T13:42:41Z

src/main/java/lotto/model/Lotto_purchaser.java

+
+    // 구매 금액에 따른 로또 티켓 수 반환
+    public int costToTicketNUM() {
+        return costOfPurchasing / 1000;


이것도 매직넘버 필요합니다!

baaamk · 2025-10-12T14:02:57Z

src/main/java/lotto/view/Validator.java

음 검증 전용으로 책임을 분리한건 좋습니다.
하지만 책임 분리가 곧 클래스를 무조건 나눈다는 뜻은 아닙니다.
이 점 생각하셔서 책임을 적절한 곳에 분배해 보세요!!

baaamk · 2025-10-12T14:03:45Z

src/main/java/lotto/view/View.java

+    public void winningStatistics(Map<Rank, Integer> statistics, double lottoYield) {
+        System.out.println("당첨 통계");
+        System.out.println("---");
+
+        System.out.println("3개 일치 (5,000원) - " + statistics.get(Rank.THREE) + "개");
+        System.out.println("4개 일치 (50,000원) - " + statistics.get(Rank.FOUR) + "개");
+        System.out.println("5개 일치 (1,500,000원) - " + statistics.get(Rank.FIVE) + "개");
+        System.out.println("5개 일치, 보너스 볼 일치 (30,000,000원) - "
+                + statistics.get(Rank.FIVE_AND_BONUS) + "개");
+        System.out.println("6개 일치 (2,000,000,000원) - " + statistics.get(Rank.SIX) + "개");
+
+        System.out.printf("총 수익률은 %.1f%%입니다.%n", lottoYield);
+    }
+


view가 모델 내부를 알고있으면 안됩니다.
MVC 설계에서 View는 Model을 알면 안 되고, Model은 View를 알면 안 됩니다!

DTO를 활용해서 해보시는건 어떨까요??

완성본

e64eaf2

seryn6911-hub requested review from Hoya324, KoSeonJe, Seooooo24, baaamk and tiemo0708 as code owners September 29, 2025 02:36

seryn6911-hub added 3 commits September 29, 2025 20:35

완성본

5505e14

map으로 바꿈

e41af09

Lotto_purchaser 수정

96f5e34

0702Yoon removed request for Hoya324 and KoSeonJe October 12, 2025 03:45

0702Yoon reviewed Oct 12, 2025

View reviewed changes

요구 사항 반영

fcc97a1

baaamk reviewed Oct 12, 2025

View reviewed changes



		public Map<Rank, Integer> calculateWinningStatistics() {
		Map<Rank, Integer> result = new EnumMap<>(Rank.class);

[2주차] 객체지향 코드 연습(seryn6911-hub) #45

Are you sure you want to change the base?

[2주차] 객체지향 코드 연습(seryn6911-hub) #45

Uh oh!

Conversation

seryn6911-hub commented Sep 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

0702Yoon 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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

baaamk 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!

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

seryn6911-hub commented Sep 29, 2025 •

edited

Loading