[자동차 경주] 박시영 미션 제출합니다. #2

player7571 · 2024-09-07T06:41:05Z

Model

saveCarName() : 입력받은 자동차 이름을 car[] 배열에 저장하고 자동차의 개수만큼 len[]배열의 크기 지정 자동차 이름이 5글자가 넘을경우 예외 발생
saveNumberOfAttempts() : 입력받은 회수 저장 만약 숫자가 아니면 예외 발생
race() : 차의 개수만큼 반복 0부터 9까지 랜덤 숫자 뽑아서 4이상일 경우 len[]++ 나중에 우승자를 찾기 위해 max와 len[]을 비교해서 max 최신화 후 실행 결과를 StringBuilder로 return
findWinner() : max와 len[]이 같으면 우승자인데 여러명이 있을수 있으니 첫 우승자가 나오면 jointWinner를 true로 변경하고 우승자가 또 나오면 , 추가 후 StringBuilder로 return

View

inputCarName() : 경주할 자동차 이름을 입력받음
inputNumberOfAttempts() : 시도할 회수 입력받음
printResult() : 각 차수별 실행결과 출력
printWinnter() : 우승자 출력

Application

View와 Model 선언하고 사용

holyPigeon

고생하셨습니다!!!

holyPigeon · 2024-09-07T07:28:11Z

src/main/java/racingcar/View.java

+    void printResult(Model model){
+        System.out.println("\n실행 결과");
+        while (model.numberOfAttempts --> 0) {
+            System.out.println(model.race());
+        }
+    }


MVC에서는 Controller 에서만 Model과 View가 만나고 View 코드 안에서는 Model과 관련된 부분이 웬만하면 없는 게 맞아서 사소하지만 model 객체가 아닌 model.race()를 직접 넣는 게 더 좋을 듯 합니다!

holyPigeon · 2024-09-07T07:29:48Z

src/main/java/racingcar/Model.java

+        len = new int[st.countTokens()];
+        for(int i=0; i<car.length; i++){
+            String str = st.nextToken();
+            if(str.length()>5) throw new IllegalArgumentException("5글자 이하로 작성");


사소한 거긴 한데 if 문은 한 줄이라도 괄호를 쳐주는 게 좋긴 합니다!

holyPigeon · 2024-09-07T07:35:05Z

src/main/java/racingcar/Model.java

+        try {
+            numberOfAttempts = Integer.parseInt(input);
+        } catch (Exception e){
+            System.out.println("숫자를 입력하세요");
+        }


여기서는 아스키 코드를 이용해서 숫자 범위 안에 드는지 검사할 수 있을 듯! 근데 간단하게 짜려면 이렇게 짜도 되긴 합니당

holyPigeon · 2024-09-07T07:41:00Z

src/main/java/racingcar/Model.java

+            if(rand>=4) len[i]++;
+            sb.append(car[i]).append(" : ")
+            .append("-".repeat(Math.max(0, len[i])))
+            .append("\n");
+            max = Math.max(len[i], max);


이 부분이 대충 경주 결과를 문자열로 나타내는 로직인 건 알겠는데, 정확히 어떤 과정을 통해 결과가 도출되는지 명확하지 않은 듯!

아마 최댓값을 넣는 등의 부분이 메서드 분리가 잘 안된 것도 있고 변수명이 직관적이지 않은 이유도 있는 것 같습니닷

ex. len이 length를 줄인 건 알겠는데 정확히 어떤 길이를 나타내는지? 명확하지 않음

holyPigeon · 2024-09-07T07:42:48Z

src/main/java/racingcar/Model.java

+    StringBuilder findWinner(){
+        StringBuilder sb = new StringBuilder();
+        boolean jointWinner = false;
+        for(int i=0; i<car.length; i++){
+            if(len[i]==max) {
+                if(jointWinner) sb.append(", ");
+                jointWinner = true;
+                sb.append(car[i]);
+            }
+        }
+        return sb;
+    }


이 분은 뭔가 알고리즘 느낌으로 최적화된 코드인 것 같은데, 아마도 유지보수의 측면에서는 우승자를 고르고 바로 StringBuilder에 넣기보다는

전체 우승자 목록을 먼저 반환한다음, 최종적으로 StringBuilder에 넣는 게 좋을 듯합니다...!

devlwh0830 · 2024-09-07T11:10:01Z

src/main/java/racingcar/View.java

+    void printResult(Map<String, Integer> result){
+
+        StringBuilder sb = new StringBuilder();
+        sb.append("\n실행 결과\n\n");
+
+        result.forEach((name, distance)-> sb.append(name).append(" : ")
+                .append("-".repeat(distance))
+                .append("\n"));
+        System.out.println(sb);
+    }
+
+    void printWinner(String winner){
+        System.out.print("최종 우승자 : ");
+        System.out.println(winner);
+    }


출력도 따로 분리 했으면 좋았을거 같아용

devlwh0830 · 2024-09-07T11:11:18Z

src/main/java/racingcar/Application.java

-        View view = new View();
-        Model model = new Model();
-
-        model.saveCarName(view.inputCarName());
-
-        model.saveNumberOfAttempts(view.inputNumberOfAttempts());
-
-        view.printResult(model.race());
-
-        view.printWinner(model.findWinner());
+        Controller controller = new Controller();
+        controller.run();


이거 깔끔하게 변경 잘된거 같아요

devlwh0830 · 2024-09-07T11:15:46Z

src/main/java/racingcar/Model.java

+                winners.add(list.name);
+            }
+        }
+        return winners.toString().replace("[", "").replace("]", "");


이거 좀 신박해서 점수 +1점 😃

Formatting 방식이 너무 좋네요👏🏻

boroboro01 · 2024-09-07T11:35:33Z

src/main/java/racingcar/Model.java

+                int rand = Randoms.pickNumberInRange(0, 9);
+                list.addDistance(rand);
+                result.put(list.name, list.distance);
+                updateMaxValue(maxDistance, list.distance);


최댓값을 비교하여 찾는 로직을 매 시도 마다 수행하는 이유가 궁금합니다! findWinner() 메서드를 실행하기 전에 한 번만 찾는 방법은 어떨까요?

큰 생각없이 최대값을 구하려고 백준문제 풀때처럼 반복문 중간에 집어넣었는데 최대값을 찾는 로직을 아예 함수로 분리하는게 좋았을것같은데 이건 제 실수네요..

boroboro01 · 2024-09-07T11:37:02Z

src/main/java/racingcar/Model.java

+                winners.add(list.name);
+            }
+        }
+        return winners.toString().replace("[", "").replace("]", "");


Formatting 방식이 너무 좋네요👏🏻

boroboro01 · 2024-09-07T11:42:39Z

src/main/java/racingcar/Model.java

+    String findWinner(){
+        ArrayList<String> winners = new ArrayList<>();
+        for (Car list : carList) {
+            if(list.sameDistance(maxDistance)){


44번 라인과 57번 라인에서 향상된 for문을 사용할 때 보통은 (Car �car : carList)와 같이 사용한다고 생각합니다. carList에서 car object 하나를 가져온다는 느낌을 준다고 생각합니다.

(Car list : carList)와 같이 작성한 이유가 있을까요?

foreach문을 자주 사용하지 않아서 몰랐던 사실이었어요.. 하나 배워갑니다!

player7571 and others added 10 commits September 7, 2024 11:47

feat: 경주할 자동차 입력 기능 추가

7bfb034

feat: 시도회수 입력 기능 추가

906e134

feat: 실행 결과 출력 추가

249d53f

feat: 우승자 출력기능 추가

cc79944

fix: 필요없는 매개변수 삭제

b621f47

fix: 중복 코드 삭제

3405b05

add README.md

46f9f20

Merge branch 'InQ-InQ-InQ-InQ-InQ:main' into siyoung

1966802

fix: 라이브러리 변경

df2a5a2

style: 코드 정리

7e93bb7

player7571 changed the title ~~Siyoung~~ [자동차 경주] 박시영 미션 제출합니다. Sep 7, 2024

holyPigeon reviewed Sep 7, 2024

View reviewed changes

player7571 added 4 commits September 7, 2024 17:06

fix: 피드백 바탕으로 변수명 변경, 이동거리및 최대 이동거리 최신화 하는 메서드 분리

858e14a

fix: 우승자를 전부 뽑은 후 한번에 문자열로 반환하도록 변경

06a7da1

fix: car[] distance[] 배열 제거 후 Car 객체 추가 후 코드 수정

c5b3d53

feat: controller 분리

468d82b

devlwh0830 reviewed Sep 7, 2024

View reviewed changes

boroboro01 reviewed Sep 7, 2024

View reviewed changes

[자동차 경주] 박시영 미션 제출합니다. #2

Are you sure you want to change the base?

[자동차 경주] 박시영 미션 제출합니다. #2

Uh oh!

Conversation

player7571 commented Sep 7, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

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

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

player7571 commented Sep 7, 2024 •

edited

Loading