[2주차] 객체지향 코드 연습(hughryu1125) #44

hughryu1125 · 2025-09-28T16:10:10Z

현재까지 작성한 부분까지 올립니다..ㅠㅠ

input 과정에서 빈칸을 사용자가 입력한 것으로 받아들이는 오류가 있습니다.
과제 요구 사항들을 많이 반영하지 못했습니다. (else 미사용, enum 사용, bonus number에 대한 잘못된 이해 등)

최대한 29일까지 완성해보겠습니다.

프로그래밍 요구 사항들을 다 반영시키지 못했습니다...

magic number 전체적 구조 변화

0702Yoon

고생하셨습니다.
흥미로운 부분도 있었습니다.

전체적인 리뷰는 아니기에 제가 말한 것과 스스로 돌아보면서 고쳐보세요!

0702Yoon · 2025-10-10T11:01:36Z

src/main/java/controller/FlowManager.java

+    Input input;
+    Output output;
+    Validator validator;
+    Calculator calculator;


접근제어자, final에 대해서 알고 계시나요??

다시 공부해보겠습니다..ㅠㅠㅠ

넵! 객체지향의 캡슐화를 지킬 수 있는 방법이여서 학습하시길!

0702Yoon · 2025-10-10T11:05:59Z

src/main/java/domain/service/Validator.java

+public class Validator {
+
+    public void validateAmountOfMoney(int money){
+        if(money % 1000 != 0 || money < 0) throw new IllegalArgumentException();


매직 넘버 + 에외 이유도 적으면 더 좋을 것 같아요.

0702Yoon · 2025-10-10T11:06:37Z

src/main/java/view/Output.java

+        System.out.println("구입하실 금액을 입력하세요: ");
+    }
+
+    public void printAskNumbers(){


메서도 네이밍이 모호한 것 같아요.

printAskWinningNumber()로 하면 괜찮을까요..?

0702Yoon · 2025-10-10T11:10:37Z

src/main/java/view/Input.java

+            inputNumbers.add(Integer.parseInt(input.trim()));
+        }
+
+        return inputNumbers; // validation하지 않은 raw값을 보냄.


이거는 검사하는 것을 아직 구현하지 않아서 주석 달으신거에요? 아니면 그냥 이 메서드를 설명하신건가요?

validator를 만들기 전에 작성한 코드라 그 주석이 있는 것 같습니다..

0702Yoon · 2025-10-10T11:11:45Z

src/main/java/controller/FlowManager.java

+        List<Integer> winningNumbers = input.readLottoNumbers();
+
+        output.printAskNumbers();
+        int bonusNumber = input.readBonusNumber();


보너스 번호도 검사를 일부러 안하신건가요?

이 부분은 까먹었습니다ㅠㅠ

0702Yoon · 2025-10-10T11:13:02Z

src/main/java/domain/model/BunchOfLotties.java

+
+public class BunchOfLotties {
+    int lottiesCount;
+    RandomNumberCreator randomCreator = new RandomNumberCreator(); // 이걸 instance로 진짜 해야돼?


필드 주입을 하신 이유가 무엇인가요?

외부에서 전체적인 로직을 알 필요 없다고 생각해서 숨기려고 했습니다..!
randomCreator도 FlowManager에서 가지고 있다가 함수 파라미터로 넣어주는게 적절할까요?

오호 의존성 주입에 대한 개념을 아시나요??
의도 자체는 굉장히 좋은 것 같아요.
소프트웨어에서는 BunchOfLotties가 RandomCreator를 사용할 때 직접 고르는 것을 선호하지 않거든요!
현우멘토님이 보내신 AppConfig를 한번 보시면 좋을 것 같아요

넵!! 더 공부해보겠습니다..!!

0702Yoon · 2025-10-10T11:17:13Z

src/main/java/controller/FlowManager.java

+        int winnerCount = lotties.getWinnerCount(winningLotto);
+        int secondWinnerCount = lotties.get2ndWinnerCount(winningLotto);
+        int thirdWinnerCount = lotties.get3rdWinnerCount(winningLotto);
+        int fourthWinnerCount = lotties.get4thWinnerCount(winningLotto);
+        int fifthWinnerCount = lotties.get5thWinnerCount(winningLotto);


갑자기 잘 포장했다가 배를 가르셨어요.
그 이유가 뭐죠?

winnerCount를 비롯해 각각의 변수들이 이후 두 군데(결과 출력, 이윤 계산)에서 쓰여서 변수로 직접 선언했습니다. 매번 get함수로 불러오는 것은 적절하지 않은 것 같았습니다.

그리고 한 함수로 처리하지 않고 각 함수를 5개씩이나 만들게 됐던 건..
함수 하나로 하려다보니 list를 써야했고, list를 쓰면 아무래도 index부분이나, 가독성이 좋지 않을 것 같아서 모두 각기 다른 함수로 만들었습니다

이제 보니, list를 쓰는게 더 깔끔하고 가독성 부분도 괜찮았을 것 같습니다...

그러면 혹시 lotties가 결과, 이윤을 반환하는 건 별로였을까요??

lotties가 이윤을 반환하기 위해선 Calculator class를 가지고 있어야 하는데,
Calculator의 경우, 이윤을 계산하는 함수 뿐 아니라 다 함수도 가지고 있어서
lotties가 Calculator를 가지고 있는 것이 부적절하다고 느꼈습니다.

Calculator를 기능별로 더 분리해서 만들고, lotties가 Calculator를 가지는 구조가 적절할까요..??

0702Yoon · 2025-10-10T11:19:48Z

src/main/java/domain/model/BunchOfLotties.java

+    public int get3rdWinnerCount(WinningLotto winningLotto) {
+        List<Integer> winningNumbers = winningLotto.getNumbers();
+        int count = 0;
+        for(int i = 0; i < lottoList.size(); i++){
+            if (lottoList.get(i).getSameNumberCount(winningNumbers) == LOTTOINFO.MAX_LOTTO_COUNT-1
+                    && !lottoList.get(i).hasBonusNumber(winningLotto.getBonusNumber())){
+                count++;
+            }
+        }
+
+        return count;
+    }
+


get하실 때마다 특정 순위의 개수를 반환하는 것 같아요.
근데 그러면 전체 로또를 6번 돌아야 하기 때문에 조금 비효율적인 것 같아요.

이걸 한번에 검사해서 list로 반환하는게 좋을까요..?

네네
만약 로또가 10만개일 경우에는
1등부터 보너스 당첨자까지 확인할려면 70만개를 도는 거라서 다른 구현이 더 좋을 것 같아요!

baaamk

고생하셨습니다.

MVC에 대한 공부가 조금 더 필요하실 것 같습니다.
책임을 너무 분배한것이 아닌가?? 라는 생각도 드네요.

그럼에도 고민을 많이 하신게 정말 많이 보여서 기쁩니다!!
고생하셨어요!

AppConfig 잘 활용해서 리팩토링 해보세요!

baaamk · 2025-10-12T14:05:34Z

src/main/java/domain/service/LOTTOINFO.java

클래스 명 카멜 케이스 유의해주세요

baaamk · 2025-10-12T14:05:38Z

src/main/java/domain/service/PRIZE.java

클래스 명 카멜 케이스 유의해주세요

baaamk · 2025-10-12T14:05:51Z

src/main/java/domain/service/WINNERCOUNT.java

클래스 명 카멜 케이스 유의해주세요

baaamk · 2025-10-12T14:06:30Z

src/main/java/User_checkMatchingNumbers

클래스 명 카멜 케이스 유의해주세요

baaamk · 2025-10-12T14:06:47Z

src/main/java/FlowController

패키지 구조 유의해주세요

baaamk · 2025-10-12T14:25:39Z

src/main/java/view/Output.java

+        System.out.println(count + "개를 구매했습니다.");
+        System.out.print("[ ");
+        for(int i = 0; i < count; i++) {
+            if(i == count-1) System.out.print(numbers.get(i));


저는 중괄호 생략을 별로 좋아하지 않는 편입니다..
다음에 협업하실때는 이런것 맞추는게 좋아보여용

baaamk · 2025-10-12T14:26:19Z

src/main/java/view/Output.java

+    public void printMatchingResult(List<Integer> winnerCounts){
+        System.out.println("3개 일치 (5,000원) -- " + winnerCounts.get(WINNERCOUNT.WINNER) + "개");
+        System.out.println("4개 일치 (50,000원) -- " + winnerCounts.get(WINNERCOUNT.SECONDWINNER) + "개");
+        System.out.println("5개 일치 (1,500,000원) -- " + winnerCounts.get(WINNERCOUNT.THIRDWINNER) + "개");
+        System.out.println("5개 일치 + 보너스 숫자 일치 (30,000,000d원)" + winnerCounts.get(WINNERCOUNT.FOURTHWINNER) + "개");
+        System.out.println("6개 일치 (2,000,000,000원) -- " + winnerCounts.get(WINNERCOUNT.FIFTHWINNER) + "개");
+    }


DTO 활용 고민해보시면 좋을 것 같습니다~

baaamk · 2025-10-12T14:27:11Z

src/main/java/view/Output.java

+        System.out.println(" ]");
+    }
+
+    public void printLotties(List<Lotto> lottoList){


MVC 설계에서 View는 Model을 알면 안 되고, Model은 View를 알면 안 된다.
이 사실을 유의하면서 코드 짜보시면 좋을 것 같아요!

baaamk · 2025-10-12T14:27:35Z

src/main/java/view/Output.java

+    }
+
+
+}


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-12T14:29:06Z

src/main/java/view/Output.java

+    }
+
+    private void printLotto(List<Integer> numbers){
+        int count = numbers.size(); // 과연 이게 좋을까?? 이미 로또의 사이즈가 확정적으로 정해져있음.. 그치만 혹시 모를 상황을 생각해서 일단 이렇게 함...


이걸 생각하셨다면 input에서 최소한의 검증은 해주시는게 좋습니다.
숫자가 6개가 들어왔는지와 같은 input 검증이요!

hughryu1125 added 4 commits September 29, 2025 00:32

기능만 구현한 초기 버전입니다....

3d16740

프로그래밍 요구 사항들을 다 반영시키지 못했습니다...

기능만 구현한 초기 버전입니다....

ddd00ad

프로그래밍 요구 사항들을 다 반영시키지 못했습니다...

기능만 구현한 초기 버전입니다....

383a43a

프로그래밍 요구 사항들을 다 반영시키지 못했습니다...

Merge remote-tracking branch 'origin/main'

6c65acd

hughryu1125 requested review from Hoya324, KoSeonJe, Seooooo24, baaamk and tiemo0708 as code owners September 28, 2025 16:10

yunjin1213 requested review from 0702Yoon, enohs and yunjin1213 and removed request for Hoya324, KoSeonJe, Seooooo24 and tiemo0708 September 29, 2025 02:15

hughryu1125 added 3 commits October 1, 2025 02:10

패키지 분류

224d0ae

magic number 전체적 구조 변화

패키지 분류

27c4b3d

magic number 전체적 구조 변화

user delete ver

9a15c4e

0702Yoon reviewed Oct 10, 2025

View reviewed changes

hughryu1125 added 2 commits October 11, 2025 21:13

user delete ver

c2c52bd

수정 버전입니다

0f3e905

baaamk reviewed Oct 12, 2025

View reviewed changes

		}


		} No newline at end of file

[2주차] 객체지향 코드 연습(hughryu1125) #44

Are you sure you want to change the base?

[2주차] 객체지향 코드 연습(hughryu1125) #44

Uh oh!

Conversation

hughryu1125 commented Sep 28, 2025

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!

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!

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