🚀 5단계 - 자동차 경주(리팩토링) #6078

baeksangha · 2025-03-21T07:32:12Z

5단계 리팩토링 요구사항

핵심 비지니스 로직을 가지는 객체를 domain 패키지, UI 관련한 객체를 view 패키지에 구현한다.
MVC 패턴 기반으로 리팩토링해 view 패키지의 객체가 domain 패키지 객체에 의존할 수 있지만, domain 패키지의 객체는 view 패키지 객체에 의존하지 않도록 구현한다.

작업 내용

carracing 패키지를 domain, util, view로 구분
Position 객체 추가 (모든 원시값과 문자열을 포장한다.)
Cars 객체 추가 (레이싱에 참여한 차들의 비즈니스 로직 담당)
CarRacing 클래스의 역할 메서드 분리

궁금한 점

4단계 피드백에서 getWinners의 응답 클래스으로 winner 클래스를 생성하는 것을 말씀해주셨는데, 구성을 어떻게 할지 고민됐습니다.
Winner 클래스를 구성하고 Car 객체를 받으면 Winner에서 자동차의 이름을 얻기 위해서 car.getName().getName();을 받는 형식으로 구현하는건가요? Winner 클래스에 원시값과 문자열 형태의 필드가 들어가야 할지, 클래스로 포장된 필드들이 들어가야 할지 고민되어서 질문드립니다!

public class Winner {
    private final Car car;
    public String getCarName() {
        return car.getName().getName();
    }
}

- carracing 패키지를 domain, util, view로 구분 - Position 객체 추가 (모든 원시값과 문자열을 포장한다.) - Cars 객체 추가 (레이싱에 참여한 차들의 비즈니스 로직 담당) - CarRacing 클래스의 역할 메서드 분리

nooose · 2025-03-21T14:23:48Z

4단계 피드백에서 getWinners의 응답 클래스으로 winner 클래스를 생성하는 것을 말씀해주셨는데, 구성을 어떻게 할지 고민됐습니다.
Winner 클래스를 구성하고 Car 객체를 받으면 Winner에서 자동차의 이름을 얻기 위해서 car.getName().getName();을 받는 형식으로 구현하는건가요? Winner 클래스에 원시값과 문자열 형태의 필드가 들어가야 할지, 클래스로 포장된 필드들이 들어가야 할지 고민되어서 질문드립니다!

Winner 객체를 어떻게 바라보냐에 따라 다를 것 같아요
아마 지금 구조라면 view에게 넘겨 우승자를 출력시키기 위한 객체일 가능성이 높아보입니다.
즉, 로직은 존재하지 않는 DTO일 가능성이 높아보여요

자동차의 이름을 뷰에게로만 잘 전달할 수 있다면 CarName 객체 또는 String을 넘겨줘도 큰 상관은 없다고 생각합니다.
상하님은 어떤 방법이 나을 것 같나요?

nooose

안녕하세요 상하님
마지막 단계인 만큼 피드백이 조금 많아졌는데요
개선해 보시면 좋을 것 같습니다
수정 후 다시 요청 부탁드릴게요! 😊

nooose · 2025-03-21T13:50:36Z

src/main/java/edu/nextstep/camp/carracing/domain/Position.java

+public class Position {
+    private int value;


양수를 표현하는 값 객체 좋습니다 👍

nooose · 2025-03-21T13:55:47Z

src/main/java/edu/nextstep/camp/carracing/domain/Position.java

+    public void increment() {
+        this.value++;
+    }


값 객체를 불변으로 만들어보는 것은 어떨까요?

increment() 메소드는 객체의 내부 상태를 변경하는 가변방식으로 동작하고 있습니다
값 객체는 불변성을 가지게 설계하는 것이 좋은데요
기존 값 자체가 변하기보다는 새로운 값을 만들어내는 방식이 더 적절할 것 같아요

예를 들어, 1 + 1 = 2에서 기존의 1(값)이 2로 변하는 것이 아니라, 새로운 2(값)가 생성되는 것처럼요!

var newPosition = position.increment();

이렇게 하면 불변성을 유지할 수 있고, 객체의 예상치 못한 변경을 방지할 수 있습니다! 🚀

https://tecoble.techcourse.co.kr/post/2021-05-16-dto-vs-vo-vs-entity/

관련해서 읽어보시면 도움이 될거예요

포비님 라이브 강의에서 제가 한것처럼 표현하시길래 따라 쓰긴 했는데, 객체의 가변성vs불변성 측면에서 불변성을 가진 객체는 그만큼 인스턴스가 많이 생성되니 객체 생성 비용이나 GC에 부담이 있을 수 있다고 생각했습니다.
공유 주신 문서를 읽어보니 VO 객체는 불변성이 보장되어야 하는 것이 맞는 것 같네요. 읽어보고 코드 수정했습니다. adb180a 감사합니다. 앞으로도 이러한 상황을 맞닥뜨린다면 객체 생성 비용은 무시할 수 있다고 보고(아무래도 클래스에서 객체를 무한히 많이 생성하진 않으니..) 불변성을 보장하도록 객체를 설계하는게 나을까요?

nooose · 2025-03-21T13:57:47Z

src/main/java/edu/nextstep/camp/carracing/domain/Position.java

+    public String repeatSymbol(String symbol) {
+        return symbol.repeat(value);
+    }


MVC 패턴 기반으로 리팩토링해 view 패키지의 객체가 domain 패키지 객체에 의존할 수 있지만, domain 패키지의 객체는 view 패키지 객체에 의존하지 않도록 구현한다.

view 객체를 직접적으로 의존하지는 않지만 출력을 위한 메소드네요
이러한 간접적인 의존도 제거하면 좋겠습니다!

c871377 요런 식으로 수정해 보았습니다.

nooose · 2025-03-21T14:00:28Z

src/main/java/edu/nextstep/camp/carracing/domain/Position.java

@@ -0,0 +1,32 @@
+package edu.nextstep.camp.carracing.domain;
+
+public class Position {


var position1 = new Position(5); var position2 = new Position(5); position1.equals(position2); // false

값은 같아보이는데 왜 다르다고 나올까요?
동등성을 보장시켜 보면 좋겠습니다!

모든 로직에 단위 테스트를 구현한다

위 코멘트 참고해서 테스트도 같이 작성해 보면 좋겠습니다!

bb17bfe
동등성 보장을 위해 equals와 hashcode 메서드 오버라이딩해주고, 테스트 코드도 추가했습니다. 감사합니다!

nooose · 2025-03-21T14:02:38Z

src/main/java/edu/nextstep/camp/carracing/view/ResultView.java

+    }
+
+    public static void printResultMessage() {
+        System.out.println('\n' + RESULT_MESSAGE);


Suggested change

System.out.println('\n' + RESULT_MESSAGE);

System.out.println(System.lineSeparator() + RESULT_MESSAGE);

OS 환경에 따라 개행 문자가 다르므로, 하드코딩된 \n 대신
lineSeparator()를 사용하는 것이 좋겠습니다!

감사합니다. System.lineSeperator()를 사용하는 방식으로 변경했습니다. 몰랐던 사실이네요 ☺️ 83009f8

nooose · 2025-03-21T14:06:33Z

src/main/java/edu/nextstep/camp/carracing/domain/Cars.java

+        for (Car car : this.values) {
+            if (car.isMaxPosition(winnerPosition)) {
+                result.add(car.getName().getName());
+            }
+        }


indent(인덴트, 들여쓰기) depth를 2를 넘지 않도록 구현한다. 1까지만 허용한다.
예를 들어 while문 안에 if문이 있으면 들여쓰기는 2이다.
힌트: indent(인덴트, 들여쓰기) depth를 줄이는 좋은 방법은 함수(또는 메소드)를 분리하면 된다.

메소드를 추출하면서 depth를 줄여보면 좋겠습니다.
요구사항이니 리팩토링 해보시죠!

nooose · 2025-03-21T14:06:50Z

src/main/java/edu/nextstep/camp/carracing/view/ResultView.java

+    }
+
+    public static void printWinners(Cars winners) {
+        System.out.println(String.join(", ", winners.getWinnerNames()) + "가 최종 우승했습니다.");


join 사용으로 깔끔하네요! 💯

nooose · 2025-03-21T14:11:47Z

src/main/java/edu/nextstep/camp/carracing/domain/CarRacing.java

+    public static void main(String[] args) {
+        List<String> carNames = inputCarNames();
+        int tryCount = getTryCount();
+
+        Cars cars = Cars.fromNames(carNames);
+
+        ResultView.printResultMessage();
+        for (int i = 0; i < tryCount; i++) {
+            cars.moveCars();
+            cars.printCarStatus();
+        }
+        ResultView.printWinners(cars.getWinners());
+    }


애플리케이션 실행의 진입점인 main 메소드가 도메인 영역에 위치하는 것은 어색해 보여요
main은 애플리케이션의 최초 실행이 되는 영역입니다

실제 로직을 수행하는 도메인 모델과는 분리되는 것이 자연스러워 보이는데,
그림에서도 도메인 영역보다는 컨트롤러 계층과 더 가까운 느낌을 줍니다.

해당 클래스를 다른 패키지 계층으로 이동시켜보면 좋을 것 같습니다 🚀

8b55e9a
domain 객체 상위로 이동하고, 클래스명도 CarRacingApplication으로 변경했습니다. 감사합니다.

nooose · 2025-03-21T14:12:36Z

src/main/java/edu/nextstep/camp/carracing/domain/Cars.java

+    public static Cars fromNames(List<String> carNames) {
+        List<Car> cars = new ArrayList<>();


정적 팩토리 메소드로 간결하게 표현이 가능하네요! 👍 👍

nooose · 2025-03-21T14:14:11Z

src/main/java/edu/nextstep/camp/carracing/domain/Cars.java

+    public void moveCars() {
+        for (Car car : this.values) {
+            car.move(generateRandomNumber());
+        }
+    }


Car의 움직임은 테스트하기 쉽게 만들어졌지만
Cars의 moveCars는 랜덤을 내부적으로 생성하고 있어 테스트할 때 제어하기가 힘들 것 같아요

Cars도 도메인 모델이니 테스트하기 쉬운 구조로 재구성하면 어떨까요?

요거 어떻게 하면 테스트하기 쉬운 구조로 재구성할 수 있을까요? 좀 고민해보았는데 구조가 안떠오릅니다 🙆‍♂️

- Cars 클래스가 ResultView를 의존하지 않도록 구조 변경 - getMaxPosition을 private으로 변경 - getWinnerNames 메서드 삭제 - 2depth 코드를 stream으로 변경

refactor:

2d671a8

- carracing 패키지를 domain, util, view로 구분 - Position 객체 추가 (모든 원시값과 문자열을 포장한다.) - Cars 객체 추가 (레이싱에 참여한 차들의 비즈니스 로직 담당) - CarRacing 클래스의 역할 메서드 분리

nooose requested changes Mar 21, 2025

View reviewed changes

baeksangha added 6 commits March 22, 2025 08:49

refactor: Position의 repeatSymbol 제거

c871377

refactor: Position 클래스의 값이 같으면 동일하도록 equals와 hashcode 오버라이드

bb17bfe

refactor: 개행문자 -> System.lineSeperator로 변경

83009f8

refactor: Position 객체의 불변성을 보장하도록 increment 로직 변경

adb180a

refactor: 상위 depth로 CarRacingApplication 클래스 이동

8b55e9a

refactor:

2908754

- Cars 클래스가 ResultView를 의존하지 않도록 구조 변경 - getMaxPosition을 private으로 변경 - getWinnerNames 메서드 삭제 - 2depth 코드를 stream으로 변경

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 5단계 - 자동차 경주(리팩토링) #6078

🚀 5단계 - 자동차 경주(리팩토링) #6078

baeksangha commented Mar 21, 2025

nooose commented Mar 21, 2025

nooose left a comment

nooose Mar 21, 2025

nooose Mar 21, 2025

baeksangha Mar 23, 2025 •

edited

Loading

nooose Mar 21, 2025

baeksangha Mar 21, 2025

nooose Mar 21, 2025

nooose Mar 21, 2025

baeksangha Mar 22, 2025

nooose Mar 21, 2025

baeksangha Mar 22, 2025 •

edited

Loading

nooose Mar 21, 2025

nooose Mar 21, 2025

nooose Mar 21, 2025

baeksangha Mar 23, 2025

nooose Mar 21, 2025

nooose Mar 21, 2025

baeksangha Mar 23, 2025

		@@ -0,0 +1,32 @@
		package edu.nextstep.camp.carracing.domain;

		public class Position {

	System.out.println('\n' + RESULT_MESSAGE);
	System.out.println(System.lineSeparator() + RESULT_MESSAGE);

		public static Cars fromNames(List<String> carNames) {
		List<Car> cars = new ArrayList<>();

🚀 5단계 - 자동차 경주(리팩토링) #6078

Are you sure you want to change the base?

🚀 5단계 - 자동차 경주(리팩토링) #6078

Conversation

baeksangha commented Mar 21, 2025

5단계 리팩토링 요구사항

작업 내용

궁금한 점

nooose commented Mar 21, 2025

nooose left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

baeksangha Mar 23, 2025 • edited Loading

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

baeksangha Mar 22, 2025 • edited Loading

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

baeksangha Mar 23, 2025 •

edited

Loading

baeksangha Mar 22, 2025 •

edited

Loading