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

Aimbe · 2024-11-23T04:57:35Z

안녕하세요 수현님!
이번 스텝은 view/domain클래스의 책임분리와 이전단계에서 피드백 주셨던
리뷰 바탕으로 복습했습니다.
하면서 객사오 사서 보고 있는데, ��확실히 이전에 봤을때랑 다르게 다가오는거 같아요. 마지막 단계도 피드백 잘 부탁드려요!

wisemuji

안녕하세요 정호님! 객사오를 다시 읽고 계시는군요! 도움이 되었다니 기쁩니다 🙂

이전에 비해서 메세지를 전달하는 과정이 더 "능동적인" 객체의 책임으로 이어진 것 같아요.
마지막 단계인 만큼 도전적인 코멘트를 몇 가지 달았습니다.

궁금한 점이나 논의하고 싶은 내용이 있으면 언제든 알려주세요!

wisemuji · 2024-11-23T07:16:32Z

src/main/kotlin/misson/car/PlayGame.kt

@@ -20,9 +22,9 @@ class PlayGame(private val racingCars: RacingCars, private val attempts: Int) {
    fun play() {
        repeat(attempts) {
            racingCars.moveAll { (0..9).random() }
-            printRoundResult(racingCars.getPositionsWithName())
+            printRoundResult(racingCars)


마지막 단계인 만큼 좀 더 개선해볼 수 있는 여러 아이디어를 제시하고 싶어요 🙂

ResultView 호출을 최대한 줄여보면 어떨까요?

우리가 흔히 웹 프로그램에서 서버와 클라이언트 간에 소통하는걸 생각해보면
서버와 클라이언트가 서로 요청과 응답을 주고받는 과정은 모두 비용입니다. 이러한 비용을 줄이는 것이 웹 프로그래밍에서 중요하게 여겨지기도 합니다.

또 지금은 단순한 콘솔 프로그램이지만, 나중에 뷰가 안드로이드 앱으로 바뀐다던지, 웹페이지 등으로 바뀐다고 하면 이러한 호출이 모두 비용이 됩니다.

wisemuji · 2024-11-23T07:17:20Z

src/main/kotlin/misson/car/domain/CarPositionFormatter.kt

+class CarPositionFormatter {
+    companion object {
+        fun formatCarPosition(car: RacingCar): String {
+            return "자동차 ${car.name} : ${formatPosition(car.position)}"


뷰 요구 사항에 의존하고 있는 이 로직이 domain 패키지에 있는 것이 적절할까요?

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

아무래도 domain에 있긴 안맞고 view에 있기에도 애매해서 util 패키지 생성 후 이동시켰습니다! ㅎㅎ

wisemuji · 2024-11-23T07:19:01Z

src/main/kotlin/misson/car/domain/RacingCar.kt

+package misson.car.domain
+
+// 코틀린 문법에서 position을 private으로 하지 않아도 내부적으로는 getter로 접근
+class RacingCar(val name: String = "붕붕이", var position: Int = 0) {


현재는 position이 private하지 않기 때문에 외부에서 position 값을 마음대로 수정할 수 있어요.

val car = RacingCar() car.position = 123

어떻게 개선해볼 수 있을까요?
https://kotlinlang.org/docs/properties.html#getters-and-setters

"붕붕이"는 너무 귀엽지만(ㅋㅋㅋ) 테스트 편의성을 위해 만들어진 값이므로 프로덕션 코드에 추가되기에는 적합하지 않은 것 같아요.

wisemuji · 2024-11-23T07:22:43Z

src/test/kotlin/misson/RacingCarTest.kt

-        assertThrows<IllegalArgumentException> {
-            RacingCar("1234567")
-        }
+        assertThat(assertThrows<IllegalArgumentException> { RacingCar("1234567") }.message)


RacingCar의 position 값을 초기화한다는 것을 명시적으로 드러내기 위해 named arguments를 활용하시면 어떨까요?

RacingCar(position = "1234567")

이 외에도 named arguments를 활용할 곳이 있나요? 프로젝트 전반적으로 적용될 수 있는 변경사항인 것 같아요!

wisemuji · 2024-11-23T07:24:20Z

src/test/kotlin/misson/RacingCarsTest.kt

+        "3, 0",
+        "2, 0",
+    )
+    fun `Param 값에 따라 자동차 그룹이 이동한다`(


테스트명을 통해 어떤 것을 테스트하고자 하는 것이 제대로 드러나지 않는 것 같아요. 테스트명을 작성하실 때에는

검증하고자 하는 로직을 명확하게 정하고 (검증이 필요한 로직이 두 개 이상이면 필요에 따라 또 다른 함수로 분할한다)

그것을 함수명으로 드러내보면 어떨까요?

이상적으로는 기능 요구 사항으로 정리하신 문서의 시나리오가 그대로 테스트 시나리오로도 이어지면 좋습니다 🙂

wisemuji · 2024-11-23T07:25:36Z

src/test/kotlin/misson/car/CarPositionFormatterTest.kt

+        "3, '---'",
+        "5, '-----'",
+    )
+    fun `position마다 -를 붙인다`(


다음 요구 사항을 다시 한 번 고려해보면 어떨까요~?

모든 로직에 단위 테스트를 구현한다. 단, UI(System.out, System.in) 로직은 제외
핵심 로직을 구현하는 코드와 UI를 담당하는 로직을 구분한다.
UI 로직을 InputView, ResultView와 같은 클래스를 추가해 분리한다.

Aimbe added 11 commits November 23, 2024 12:28

docs: 요구사항 정리

02c22f8

refactor: validate requires이용

6733ca7

refactor: value클래스 이용

466aabf

refactor: test 에러 메세지 검증 추가

8fa1eef

refactor: position을 찍는 객체 분리

a2bfca2

refactor(ResultView): 출력에 대한 책임 ResultView 변경

4043a93

test(review): parameterized Test 적용

9eae42f

fix: 객체 책임 분리에 따른 코드 수정

4ba9fb2

test: formatter 클래스 테스트

a33ed7e

chore: domain클래스 패키지 분리

38cfee7

docs: 구현 사항 체크

a9a9aaf

wisemuji requested changes Nov 23, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

Aimbe commented Nov 23, 2024

wisemuji left a comment

wisemuji Nov 23, 2024

wisemuji Nov 23, 2024

Aimbe Nov 25, 2024

wisemuji Nov 23, 2024

wisemuji Nov 23, 2024

wisemuji Nov 23, 2024

wisemuji Nov 23, 2024

wisemuji Nov 23, 2024

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

Are you sure you want to change the base?

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

Conversation

Aimbe commented Nov 23, 2024

wisemuji 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