[김용진_BackEnd] 1주차 과제 제출합니다. by Rlayd36 · Pull Request #44 · BCSDLab-Edu/java-racingcar

Rlayd36 · 2025-03-24T14:26:11Z

No description provided.

Application.java : 기능 요구 사항_차량 이름 입력받기 carInput(), carSplit() 매서드 추가

…intEachMove() 추가

dradnats1012

고생하셨습니다!
변수명이나 개행, 주석같은 부분은 계속 코드를 치시다보면 익숙해질겁니다
이번주에 객체지향 관련 학습을 하시면서 Car 클래스, Racing 클래스 등 다양한 역할을 가진 클래스로 구분하는 연습을 해보시면 좋을 것 같아요 :) 👍

dradnats1012 · 2025-03-27T08:39:06Z

src/main/java/racingcar/Application.java

+// import java.util.LinkedList;
+// import java.util.Queue;


사용하지 않는 코드들은 삭제해주세요!

dradnats1012 · 2025-03-27T08:40:10Z

src/main/java/racingcar/Application.java

 public class Application {
    public static void main(String[] args) {
        // TODO: 프로그램 구현
+        String carsStr = carInput();


carStr이라는 변수명은 자료형이 들어가 권장되지 않는 변수명입니다!

dradnats1012 · 2025-03-27T08:40:30Z

src/main/java/racingcar/Application.java

+        // 여기까지 입력
+
+        Game(carsList, tryNum);
+


이런 개행은 지워주면 깔끔할 것 같아요

dradnats1012 · 2025-03-27T08:41:51Z

src/main/java/racingcar/Application.java

+
+    }
+
+    public static String carInput() { // 사용자 입력 (자동차 목록)


static으로 메서드를 선언하셨는데 이렇게 되면 어떻게 동작할까요?
객체지향에서 static 사용은 객체지향에서 어떻게 다가올까요?

dradnats1012 · 2025-03-27T08:42:43Z

src/main/java/racingcar/Application.java

+    public static String[] carSplit(String carsStr) { // 사용자 한테 입려받은 문자열 "," 기준으로 찢기
+        String[] carsList = carsStr.split(",");
+        return carsList;
+    }


바로 return 해줘도 될 것 같아요

dradnats1012 · 2025-03-27T08:43:03Z

src/main/java/racingcar/Application.java

+    }
+
+    public static void errCheck_carsList(String[] carsList) throws IllegalArgumentException{ // 입력한 차 이름이 5글자 이하인지 체크
+        for (String s : carsList) {


s 보다는 carName 이라는 변수명은 어떨까요?

dradnats1012 · 2025-03-27T08:43:41Z

src/main/java/racingcar/Application.java

+        return tryNum;
+    }
+
+    public static void errCheck_carsList(String[] carsList) throws IllegalArgumentException{ // 입력한 차 이름이 5글자 이하인지 체크


메서드명에 _를 추가하셨는데 다른 메서드들과 다른 이유가 있을까요?
validate 라는 네이밍은 어떨까요?

dradnats1012 · 2025-03-27T08:45:23Z

src/main/java/racingcar/Application.java

+    public static int[] move(String[] carsList, int[] moveCnt) {
+        for (int i = 0; i < carsList.length; i++) {
+           moveCnt = randomMove(moveCnt, i);
+        }
+        return moveCnt;
+    }


이렇게 되면 각 자동차들의 움직임에 대한 관리가 어려울 것 같은데 어떻게 생각하시나요?
이번주 객체지향을 공부하시면서 Car 클래스로 분리할 방법에 대해 고민해보면 좋을 것 같아요!

dradnats1012 · 2025-03-27T08:45:51Z

src/main/java/racingcar/Application.java

+    }
+
+    public static String[] checkWinner(String[] carsList, int[] moveCnt) {
+        /*


이 부분도 더 이상 사용하지 않으면 삭제해주세요!

dradnats1012 · 2025-03-27T08:46:33Z

src/main/java/racingcar/Application.java

+        return moveCnt;
+    }
+
+    public static void printEachMove(String[] carsList, int[] moveCnt){


코드의 depth가 3을 넘어가는것 같아요

알려주신 부분들 확인해서 리팩토링에 반영해보겠습니다.!

Choon0414

고생하셨습니다!
기능 단위로 메서드를 분리하려고 많이 노력하신 것 같아요.

처음이라 많이 어려우셨을 텐데, 앞으로 천천히 교육을 들으면서 코드를 고쳐나가면 실력이 금방 느실 것 같습니다.

Choon0414 · 2025-03-30T08:09:43Z

src/main/java/racingcar/Application.java

+
+    }
+
+    public static String carInput() { // 사용자 입력 (자동차 목록)


메서드 명명 규칙에 대해 찾아보면 좋을 것 같습니다!
짧고 간결한 것도 좋지만, 이름을 통해 역할을 잘 표현하는 것도 중요합니다.

Choon0414 · 2025-03-30T08:11:16Z

src/main/java/racingcar/Application.java

+        for (int i = 0; i < moveCnt.length; i++) {
+            if(moveCnt[i] == maxValue) {
+                winner[cnt] = carsList[i];
+                cnt++;


두 분기점에서 공통으로 수행하는 부분은 if문 바깥에 구현해주는 건 어떤가요?

Choon0414 · 2025-03-30T08:13:17Z

src/main/java/racingcar/Application.java

+            printEachMove(carsList, moveCnt);
+        }
+
+        String[] winner = checkWinner(carsList, moveCnt);


우승자 계산 부분을 메서드로 따로 분리해보는 건 어떨까요?

리뷰해주신 메서드 명, if문 구현, 매서드 분리 리팩토링 해보겠습니다.!

…e 변수명 변경

…urn 하도록 수정

… for-each 문 변수명 변경

errCheck_carsList() -> validateCarsList() errCheck_tryNum() -> validateTryNum()

skdud0629 · 2025-04-03T14:01:42Z

분발해라

Rlayd36 added 9 commits March 24, 2025 18:46

docs/README.md 작성

a2de675

docs/README.md : 수정

836319e

Application.java : 기능 요구 사항_차량 이름 입력받기 carInput(), carSplit() 매서드 추가

Application.java : tryNumInput() 추가

c0e0378

tryNumInt() 추가에 대한 README.md 수정 추가 commit

05a224b

Application.java : 예외처리 errCheck_carsLists(), errCheck_tryNum() 추가

f797d65

Application.java : 자동차 랜덤 전진 및 출력 구현 Game(), move(), randomMove(), pr…

64ef695

…intEachMove() 추가

Application.java : 우승자 출력 checkWinner() 추가

5b0521f

Application.java : 우승자 출력 시 null 출력되는 현상 수정, checkWinner() 큐 대신 배열 사용

f69d33c

README.md 수정

64a2adf

dradnats1012 reviewed Mar 27, 2025

View reviewed changes

Choon0414 reviewed Mar 30, 2025

View reviewed changes

Rlayd36 added 11 commits April 1, 2025 01:47

docs: 2주차 코드 리팩토링 관련 README.md 수정

0a129d5

refactor: main/java/racingcar/Application 클래스 사용하지 않는 코드 삭제

14c3f4d

refactor: main/java/racingcar/Application 클래스 carsStr -> inputCarsNam…

7a9a404

…e 변수명 변경

refactor: main/java/racingcar/Application 필요없는 개행 삭제

ddb116e

refactor: main/java/racingcar/Application 클래스 carSplit() 메서드에서 바로 ret…

a7aaa56

…urn 하도록 수정

refactor: main/java/racingcar/Application 클래스 errCheck_carsList() 매서드…

18f0ae7

… for-each 문 변수명 변경

refactor: main/java/racingcar/Application 클래스 예외처리 매서드 명 수정

56e4746

errCheck_carsList() -> validateCarsList() errCheck_tryNum() -> validateTryNum()

refactor: Car 클래스 생성

432caf0

refactor: Car 클래스 생성 위치 변경

0196b5c

refactor: carsInfoInput() 메서드 Car 클래스 이용해서 입력받도록 수정

20e0647

refactor: Car 클래스 활용해서 리팩토링

bfae2c6


		}

		public static String carInput() { // 사용자 입력 (자동차 목록)

Conversation

Rlayd36 commented Mar 24, 2025

Uh oh!

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

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

skdud0629 commented Apr 3, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants