[장지훈_BackEnd] 1주차 과제 제출합니다. by Superpinoiman · Pull Request #2 · BCSDLab-Edu/Java_Racingcar_2026-1

Superpinoiman · 2026-03-29T05:23:34Z

테스트 모두 확인하고 올립니다.

dh2906

고생하셨습니다!
메소드 명을 더 명확하게, 이게 무슨 역할을 담당하는지를 고민해주시면 좋을 것 같아요.

dh2906 · 2026-03-29T12:42:05Z

src/main/java/racingcar/Application.java

+    static class Car {
+        String name;
+        int score = 0;
+        boolean winner = true;
+
+        Car(String name) {
+            this.name = name;
+        }
+    }


Application.java에 Car 클래스가 종속되는 것보단 파일을 추가로 만들어서 분리하는 것은 어떻게 생각하시나요?

클래스를 사용하는 의도에 맞게 유지,보수가 편해지도록 따로 분리하여 코딩하는 습관이 있어야 할 것 같다고 생각하게 되었습니다. 참고하겠습니다.

dh2906 · 2026-03-29T12:42:57Z

src/main/java/racingcar/Application.java

+        String name;
+        int score = 0;
+        boolean winner = true;


접근 제어자를 지정하지 않으면 무슨 일이 생길까요??

자동차 경주를 시작하지도 않았는데 기본값으로 winner를 true로 둔 이유가 있으신가요?

접근제어자를 지정하지 않으면 같은 패키지 안에서만 접근이 가능해집니다. 어떤 의도로 클래스를 만들었는지 알아보기 어려워질 수 있겠네요. 참고하겠습니다. winner는 실행 마지막에 score 비교 후 낮은 객체의 winner를 False로만 바꾸면 간단할 것 같다고 생각해서 미리 true로 설정했는데 이것도 경주시작도 안했는데 winner라면 말이 안되네요. 참고하겠습니다.

dh2906 · 2026-03-29T12:45:10Z

src/main/java/racingcar/Application.java

+        }
+    }
+
+    private static boolean nameAgain(ArrayList<Car> Cars){


private static 메소드는 무슨 특징을 지닐까요?

이렇게 지정한 이유가 있으신가요?

해당 클래스 안에서만 호출이 가능하고 객체 없이 호출이 가능한 특징이 있습니다. 그리고 여기 클래스 안에서만 사용하고 따로 객체 없이 기능을 하는 메소드라는 특징이 있습니다. 따로 이유는 없이 유틸 메소드를 만들려고 static으로 지정했습니다.

dh2906 · 2026-03-29T12:46:09Z

src/main/java/racingcar/Application.java

+    }
+
+    private static boolean nameAgain(ArrayList<Car> Cars){
+        Set<String> nameSet = new HashSet<>();


Set 자료구조의 특징은 무엇일까요?

순서가 없어 중복을 허용하지 않습니다.

dh2906 · 2026-03-29T12:48:14Z

src/main/java/racingcar/Application.java

+            } else {
+                Cars.add(new Car(names[i]));
+            }


별 차이는 없지만, if 문에서 조건이 걸린다면 어차피 예외를 발생해서 프로그램을 종료하므로 else 문은 명시하지 않아도 될 것 같아요!

더욱 깔끔하게 되네요. 참고하겠습니다.

dh2906 · 2026-03-29T12:49:51Z

src/main/java/racingcar/Application.java

+        for (int i = 0; i < Cars.size(); i++){
+            random = Randoms.pickNumberInRange(0, 9);
+            if(random >= 4){
+                Cars.get(i).score++;


이런 식으로 객체의 멤버 변수를 직접 접근하는 방식은 지양해주세요!
전용 메소드를 하나 만들어서 이를 호출하는 방식을 채택해주세요

이것 또한 클래스를 사용한 의도랑 맞지 않네요. 메소드를 만들도록 참고하겠습니다.

dh2906 · 2026-03-29T12:51:19Z

src/main/java/racingcar/Application.java

+        }
    }
-}
+}


깃허브에 코드를 올릴때는 맨 마지막 라인에 공백이 들어와야 해요!

자료 참고바랍니당

참고하겠습니다. 피드백 공부 많이 됐습니다 감사합니다.

Superpinoiman added 11 commits March 28, 2026 23:57

feat: 시도_횟수_0_이하_예외_처리

bb90cf8

feat: 시도_횟수_숫자_아님_예외_처리

4d4980a

feat: 자동차_이름_5자_초과_예외_처리

0fdd0c4

feat: 자동차_이름_공백_포함_예외_처리

f5cc4a7

feat: 자동차_이름_빈값_예외_처리

9287cb9

feat: 구현해야할 목록

91902f8

feat: 구현해야할 목록

0a232c0

feat: 자동차 이름과 시도횟수 입력

bc89b37

feet: 자동차 이름과 시도횟수 입력

ec06bea

feet: 레이싱 진행 출력

53803f4

feet: 우승자 출력

1072cf3

dh2906 reviewed Mar 29, 2026

View reviewed changes

+                      }
                   }
-              }
+              }

                
                    No newline at end of file

Conversation

Superpinoiman commented Mar 29, 2026

Uh oh!

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

2 participants