[자동차 경주] 나성 미션 제출합니다. #79

nns503 · 2025-02-09T13:45:30Z

배운 점, 어려웠던 점, 학습 공유

Random 클래스를 테스트에서 사용하기 위하여 Random을 상속받아서 FakeRandom 객체를 사용하였는데 이 과제에서는 nextInt 만을 사용하여 이를 사용해도 충분하였으나 랜덤값에 추가적인 요구사항이 올 것이라 예상되는 상황이라면 Random 클래스를 포함 관계로 가진 RandomGenerate 클래스를 만들어도 좋았을 것 같습니다.
컨트롤러 테스트에서 Input 및 Output을 테스트 하는 부분이 처음보는 내용이었고 어려웠던 것 같습니다. 특히 현재는 하나의 테스트 메서드를 구현하였는데, 여러 테스트를 진행할 경우 초기화 과정에서 어려움이 있었고 결국 다수의 테스트를 만들지 못하여서 이에 대해서 더 학습해야할 것 같습니다.
테스트 코드에서 Fixture를 만들어 재활용하는 과정에서 랜덤값들을 List로 선언하여서 사용하였는데 해당 Fixture에서 파생될 수 있는 값들 (누적 움직임 횟수, 순위 등)을 어떻게 해야할지 어려웠던 것 같습니다. 이번에는 메소드나 변수를 만들다 범위가 너무 커지는 것 같아서 다시 지우고 Number 리스트만을 구현하여서 사용하였는데 이전 방법처럼 연관 메서드나 변수를 추가적으로 만드는 방법이나 혹은 enum이나 다른 클래스로 묶어버리는 방법들 중 뭐가 더 좋을지는 많이 고민해보아야 할 것 같습니다.

구현 시 신경 쓴 부분

최근 모던자바인액션이라는 책을 완독하면서 막연하게 쓰고 있었던 Stream의 동작과 사용법에 대해서 학습하게 되었는데 이를 이번 과제에 녹이기 위해서 고민을 한 것 같습니다.
타인이 보기에 이해하기 쉬운 로직을 짜기 위해서 코드 작성과 추상화, 메소드 추출 등을 진행하였습니다.
너무 많은 추상화는 독이 될 수 있다고 생각하여 충분히 로직을 이해할 수 있다고 생각하면 추상화를 하지 않은 부분도 있는데 제 관점에서 본 것이며 볼 때마다 생각이 바뀌는 부분들이 존재하여( ex) 상수) 해당 부분에서 어떻게 하면 더 좋은 코드가 되었을 지 고민을 많이 했습니다!

리뷰 받고 싶은 내용

코드에서 이해하기 어려운 로직이 어느 부분인지? 혹은 다양한 아이디어에 대해서 듣고 학습하고 싶습니다!

Soundbar91

2주차 미션 고생하셨습니다 ~

최근 모던자바인액션이라는 책을 완독하면서 막연하게 쓰고 있었던 Stream의 동작과 사용법에 대해서 학습하게 되었는데 이를 이번 과제에 녹이기 위해서 고민을 한 것 같습니다.

책 난이도가 조금 있다고 들었는데, 완독까지 하셨다니.. 💯 어렵지 않으셨는 지 궁금하네요 😄 스트림을 사용하신 부분에서 학습하신 내용을 잘 적용하신 것 같습니다 !

코드를 잘 작성해주셔서 크게 이해하기 어려운 부분은 없었던 것 같습니다. List<Map<Key, Value>>를 활용해서, 각 단계의 레이싱 정보를 관리하는 부분이 좋았습니다 👍

다양한 아이디어에 대해서는 테스트 관련해서 하나 코멘트를 달았습니다. 이외에는 궁금한 부분과 책임 분리 관련해서 코멘트를 달았습니다.

3주차도 화이팅 입니다 👍

Soundbar91 · 2025-02-14T02:58:47Z

src/main/java/model/Car.java

+    }
+
+    private void validateCarName(String name) {
+        if(name == null || name.isBlank() || name.length() > 5) {


나성님은 3가지 조건에 대해서 하나의 예외 메시지로 처리하셨네요 !

5자 이하라는 사람마다 생각하는 부분이 다르겠지만, 누구는 0자도 된다라고 생각하고 입력할 수 있을 것 같아요.

5자 이하인 부분과 입력하지 않는 부분 두 가지로 나눠서 예외 메시지를 처리하면 더 좋은 예외 메시지가 될 것 같은데, 이에 대한 나성님의 생각이 궁금합니다 !

저도 이 부분에서 null이거나 isBlank일 경우 "이름을 입력해주세요." 로 나누려다가
이름 입력 -> 6자 이상 입력 -> 5자 이하 입력으로 사용자 입력이 한 번 더 생기게 유도하지 않을까 생각하였습니다!
오히려 0자 일 때 5자 이하의 이름을 작성해주세요. 라고 메시지를 준다면 바로 5자 이하의 이름을 입력을 하는 동작으로 이어지지 않을까라는 생각이 들어서 한번에 처리하게 된 것 같습니다!
하지만 지금 다시 생각해보면 분리하는 것도 좋은 아이디어이며 isBlank와 length() > 5 검사가 다른 검사이기 때문에 분리하는 것도 더 좋은 로직일 수도 있을 것 같습니다!!
0자와 n자 처리에 대한 사항이 생각보다 자주 있는 것 같은데 고민 해봐야할 것 같습니다!

Soundbar91 · 2025-02-14T08:53:26Z

src/main/java/controller/RaceController.java

+        raceOutputView.inputRaceCount();
+        int raceCount = UserInputView.readIntegerInput();
+
+        Racing racing = new Racing(random);


컨트롤러에서 객체 생성하는 대신, 주입 받는 방법에 대해서 나성님은 어떻게 생각하시나요 ??

개인적으로 컨트롤러의 객체 생성 역할이 애매하다고 생각이 들었고, Racing 객체가 변경되면 컨트롤러에 들어가서 수정을 해야하는 소요가 발생할 것 같습니다 !

Racing은 Service 역할을 하는 VO 객체라고 생각하여 단순하게 객체를 생성하였는데 주입하는 방식이 더 좋은 것 같네요!

Soundbar91 · 2025-02-14T09:18:27Z

src/main/java/model/Racing.java

+        List<String> carNames = Arrays.stream(inputCarNames.split(",")).toList();
+        Cars cars = new Cars(rand, carNames);


join이라는 메소드로 옮기고, 매개변수로 Cars를 받는 것에 대해서 어떻게 생각하시나요 ??

Cars를 만드는 팩토리 클래스를 사용한 후 매개변수로 보내는 것도 좋은 방법인 것 같습니다!

Soundbar91 · 2025-02-14T09:20:24Z

src/main/java/controller/RaceController.java

+        raceOutputView.printRacingData(response.moveData());
+        raceOutputView.printWinners(response.winners()
+                .stream().map(Car::getName).toList());


컨트롤러에서 dto의 값을 꺼내서 view로 넘기기 보다는, dto 자체를 넘겨서 view에서 값을 꺼내는 것에 대해서는 어떻게 생각하시나요 ??

컨트롤러에서 dto의 값을 꺼내서 넘겨주면, dto의 의미가 모호해지는 것 같다는 생각이 들었습니다 !

Model -> Controller -> View 를 모두 의존하는 객체를 줄이는 것이 좋다고 생각하였습니다!
DTO 응답으로 필요한 데이터만 보내는 역할을 하는 것도 충분한 DTO의 역할을 한다고 생각합니다.
하지만 이렇게 하면 변환 과정에서 구현 난이도가 더 올라가는 문제점이 있으며 오히려 불필요한 변환 과정이라고도 볼 수 있으므로 View로 DTO를 넘기는 방법도 좋은 것 같습니다~!!

Soundbar91 · 2025-02-14T09:29:37Z

src/test/java/common/FakeRandom.java

+    }
+
+    @Override
+    public int nextInt(int dummyBound) {


dummyBound의 역할이 궁금합니다 👀

FakeRandom은 랜덤값을 생성자로 받게 되는데 그렇게 되면 nextInt에서 사용하는 인자값은 필요 없게 됩니다.
(어떤 값이 들어와도 무의미합니다.)
오버라이드할 때 인자를 사용하지 않는다는 뜻을 주기 위해서 dummyBound라는 네이밍을 사용하였습니다!

Soundbar91 · 2025-02-14T09:34:20Z

src/test/java/model/CarTest.java

+        Car car = createCar(fixedValue);
+        car.move();
+
+        assertThat(car.getPosition()).isEqualTo(2);


자동차의 초기 위치를 1로 설정하신 이유가 있으실까요 ??

뭔가 다른 로직이 있어서 2가 예상 값인가 해서 찾아보다가, 초기 값이 1이더라구요 ㅎㅎ,,

예시에서 처음 상태 출력이 - 로 확인하여서 1을 default 값으로 사용하였습니다!

Soundbar91 · 2025-02-14T09:40:51Z

src/test/java/model/CarTest.java

+    private Car createCar(int fixedValue){
+        return new Car("test1", new FakeRandom(List.of(fixedValue)));


파라미터를 바꾸면 FakeRandom를 더 잘 활용할 수 있을 거 같다는 생각이 들었습니다 ! 이렇게 하면, 뭔가 테스트를 더 다양하게 할 수 있을 거 같아요. 가령, A차, B차, C차가 있으면 승자를 예측할 수 있는 테스트도 할 수 있을 거 같아요

카 테스트 보다는 레이싱 테스트에서 사용할 수 있을 거 같아요 !

Suggested change

private Car createCar(int fixedValue){

return new Car("test1", new FakeRandom(List.of(fixedValue)));

private Car createCar(Integer... fixedValue){

return new Car("test1", new FakeRandom(Arrays.asList(fixedValue)));

그렇게 한다면 Fake 객체를 더 다양하고 잘 사용할 수 있을 것 같아요!!

nns503

리뷰 감사합니다~!!

nns503 · 2025-02-14T11:55:18Z

src/main/java/model/Car.java

+    }
+
+    private void validateCarName(String name) {
+        if(name == null || name.isBlank() || name.length() > 5) {


저도 이 부분에서 null이거나 isBlank일 경우 "이름을 입력해주세요." 로 나누려다가
이름 입력 -> 6자 이상 입력 -> 5자 이하 입력으로 사용자 입력이 한 번 더 생기게 유도하지 않을까 생각하였습니다!
오히려 0자 일 때 5자 이하의 이름을 작성해주세요. 라고 메시지를 준다면 바로 5자 이하의 이름을 입력을 하는 동작으로 이어지지 않을까라는 생각이 들어서 한번에 처리하게 된 것 같습니다!
하지만 지금 다시 생각해보면 분리하는 것도 좋은 아이디어이며 isBlank와 length() > 5 검사가 다른 검사이기 때문에 분리하는 것도 더 좋은 로직일 수도 있을 것 같습니다!!
0자와 n자 처리에 대한 사항이 생각보다 자주 있는 것 같은데 고민 해봐야할 것 같습니다!

nns503 · 2025-02-14T11:56:55Z

src/main/java/controller/RaceController.java

+        raceOutputView.inputRaceCount();
+        int raceCount = UserInputView.readIntegerInput();
+
+        Racing racing = new Racing(random);


Racing은 Service 역할을 하는 VO 객체라고 생각하여 단순하게 객체를 생성하였는데 주입하는 방식이 더 좋은 것 같네요!

nns503 · 2025-02-14T12:07:49Z

src/main/java/controller/RaceController.java

+        raceOutputView.printRacingData(response.moveData());
+        raceOutputView.printWinners(response.winners()
+                .stream().map(Car::getName).toList());


Model -> Controller -> View 를 모두 의존하는 객체를 줄이는 것이 좋다고 생각하였습니다!
DTO 응답으로 필요한 데이터만 보내는 역할을 하는 것도 충분한 DTO의 역할을 한다고 생각합니다.
하지만 이렇게 하면 변환 과정에서 구현 난이도가 더 올라가는 문제점이 있으며 오히려 불필요한 변환 과정이라고도 볼 수 있으므로 View로 DTO를 넘기는 방법도 좋은 것 같습니다~!!

nns503 · 2025-02-14T12:09:37Z

src/main/java/model/Racing.java

+        List<String> carNames = Arrays.stream(inputCarNames.split(",")).toList();
+        Cars cars = new Cars(rand, carNames);


Cars를 만드는 팩토리 클래스를 사용한 후 매개변수로 보내는 것도 좋은 방법인 것 같습니다!

nns503 · 2025-02-14T12:16:31Z

src/test/java/common/FakeRandom.java

+    }
+
+    @Override
+    public int nextInt(int dummyBound) {


FakeRandom은 랜덤값을 생성자로 받게 되는데 그렇게 되면 nextInt에서 사용하는 인자값은 필요 없게 됩니다.
(어떤 값이 들어와도 무의미합니다.)
오버라이드할 때 인자를 사용하지 않는다는 뜻을 주기 위해서 dummyBound라는 네이밍을 사용하였습니다!

nns503 · 2025-02-14T12:17:57Z

src/test/java/model/CarTest.java

+        Car car = createCar(fixedValue);
+        car.move();
+
+        assertThat(car.getPosition()).isEqualTo(2);


예시에서 처음 상태 출력이 - 로 확인하여서 1을 default 값으로 사용하였습니다!

nns503 · 2025-02-14T12:18:51Z

src/test/java/model/CarTest.java

+    private Car createCar(int fixedValue){
+        return new Car("test1", new FakeRandom(List.of(fixedValue)));


그렇게 한다면 Fake 객체를 더 다양하고 잘 사용할 수 있을 것 같아요!!

nns503 added 6 commits February 6, 2025 23:03

feat : Car 객체 생성

ba074c8

test : Car 테스트 코드 작성

a99dd51

feat : 우승 자동차를 구하는 레이싱 구현

9ac93ae

test : 우승 자동차를 구하는 레이싱 테스트 구현

d386542

feat : 게임 실행 로직 구현

c6afd1f

test : 게임 실행 테스트 구현

27b41b2

Soundbar91 reviewed Feb 14, 2025

View reviewed changes

nns503 commented Feb 14, 2025

View reviewed changes

nns503 changed the base branch from main to nns503 February 18, 2025 15:16

boorownie merged commit de68d97 into next-step:nns503 Feb 19, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[자동차 경주] 나성 미션 제출합니다. #79

[자동차 경주] 나성 미션 제출합니다. #79

nns503 commented Feb 9, 2025

Soundbar91 left a comment

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

Soundbar91 Feb 14, 2025

nns503 Feb 14, 2025

nns503 left a comment

nns503 Feb 14, 2025

nns503 Feb 14, 2025

nns503 Feb 14, 2025

nns503 Feb 14, 2025

nns503 Feb 14, 2025

nns503 Feb 14, 2025

nns503 Feb 14, 2025

		List<String> carNames = Arrays.stream(inputCarNames.split(",")).toList();
		Cars cars = new Cars(rand, carNames);

		private Car createCar(int fixedValue){
		return new Car("test1", new FakeRandom(List.of(fixedValue)));

[자동차 경주] 나성 미션 제출합니다. #79

[자동차 경주] 나성 미션 제출합니다. #79

Conversation

nns503 commented Feb 9, 2025

배운 점, 어려웠던 점, 학습 공유

구현 시 신경 쓴 부분

리뷰 받고 싶은 내용

Soundbar91 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

nns503 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment