🚀 4단계 - 지뢰 찾기(리팩터링) #491

devtaebong · 2024-12-28T13:47:55Z

경록님 안녕하세요!
"철저하게 TDD로 재구현해 본다."라는 요구사항을 반영하여 미션을 진행했습니다. 😃
추가적으로, getter를 사용하지 않고 객체 간 메시지를 전달하는 방식으로 설계를 개선하려고 노력했습니다!

이번 리뷰도 잘 부탁드립니다! 🙇‍♂️
감사합니다. 😊

devtaebong · 2024-12-28T13:51:02Z

docs/step4.md

+- [ ] 하나 이상의 지뢰 셀을 열었으면 게임에서 패배한다.
+- [ ] 모든 빈 셀을 열었으면 게임에서 승리한다.
+- [ ] 입력한 셀에 지뢰가 없다면, 연결된 empty을 모두 Open
+- [ ] close 된 셀은 C를 출력
+- [ ] open 된 셀은 인접한 지뢰 개수를 출력


체크가 안되어있네요🥲
커밋을 다시 하면 리뷰 요청을 다시 드려야하는걸로 알고있는데, 수정해놓고 다음 피드백에 반영하겠습니다!

Rok93

태현님 4단계도 잘 구현해주셨네요 👍💯
크게 피드백 드릴 내용은 없었으나 추가로 몇몇 코멘트 남겨두었습니다. 😃
확인해서 반영 부탁드릴게요.
아마 다음 리뷰 요청에서는 머지가 되지 않을까 싶네요. 😁
마지막까지 화이팅입니다. 💪

Rok93 · 2024-12-29T06:02:30Z

src/main/kotlin/controller/MineSweeperController.kt

@@ -25,12 +24,23 @@ class MineSweeperController {
    private fun gameLoop(mineBoard: MineBoard) {
        val mineSweeperGame = MineSweeperGame(mineBoard)
        while (mineSweeperGame.isContinueGame()) {
-            val coordinate = InputView.askMineCoordinate()
-            mineSweeperGame.openAdjacentCell(coordinate)
+            val requestOpenCoordinate = InputView.askMineCoordinate()


View가 Model 객체를 직접 의존하고있는 것으로 보여요. (InputView의 결과물이 Coordinate)
View에서는 단순히 원시 값을 받아들이고, Model로의 변환은 Controller 계층에서 해주면 M, V 간의 의존성을 제거할 수 있을 것 같습니다.

Rok93 · 2024-12-29T06:06:11Z

src/main/kotlin/domain/Cells.kt

+    fun isAllEmptyCellsOpened(): Boolean {
+        return cells
+            .filter { it.isMineCell().not() }
+            .all { it.status == CellStatus.OPEN }


it.isOpen()과 같이 메시지를 보내볼 수 있을 것 같아요. 😃

이 부분은 Cell.isOpened 메서드는 만드신 것 같은데, 해당 메서드를 호출하도록 변경하시는 부분은 누락된 것 같아요! 👀

Rok93 · 2024-12-29T06:08:29Z

src/main/kotlin/domain/Cells.kt

    }

-    fun countOpenedEmptyCells(): Int {
-        return emptyCells().count { it.status == CellStatus.OPEN }
+    fun getBoardWidth(): Col {


width 라거나 height의 경우 읽기 전용 프로퍼티로 두는 것도 괜찮지 않을까요? 😃

val width: Int get() = cells.maxBy { it.coordinate.col }.coordinate.col val height: Int get() = cells.maxBy { it.coordinate.row }.coordinate.row

Rok93 · 2024-12-29T06:16:00Z

src/main/kotlin/domain/MineBoard.kt

+
+    fun getCell(coordinate: Coordinate): Cell = cells.get(coordinate)
+
+    fun openCell(coordinate: Coordinate) = cells.get(coordinate).open()


openCell, countAdjacentMines 메서드에 대한 테스트도 추가해주시면 좋을 것 같아요. 😃

devtaebong · 2024-12-29T09:06:58Z

경록님 안녕하세요!
꼼꼼한 피드백 다시한번 감사드립니다 🙇🏻‍♂️

Rok93

태현님 마지막 리팩터링 미션 피드백도 잘 반영해주셨습니다. 👍💯
이로써 마지막 미션인 지뢰 찾기 미션도 이만 종료하도록하겠습니다.
미션 진행하시느라 고생 많으셨습니다. 🙏

Rok93 · 2024-12-31T01:04:48Z

src/main/kotlin/domain/Cells.kt

+    fun isAllEmptyCellsOpened(): Boolean {
+        return cells
+            .filter { it.isMineCell().not() }
+            .all { it.status == CellStatus.OPEN }


이 부분은 Cell.isOpened 메서드는 만드신 것 같은데, 해당 메서드를 호출하도록 변경하시는 부분은 누락된 것 같아요! 👀

Rok93 · 2024-12-31T01:07:49Z

src/main/kotlin/domain/Cells.kt

-            }
-            return EmptyCell(coordinate)
+        fun of(mineCellGenerator: MineCellGenerator): Cells {
+            val mineCells = mineCellGenerator.execute()


execute 메서드의 결과를 Set으로 하기보다는 List로 내려주면 어떨까요? 🤔
중복 제거가 걱정된다면 execute 메서드 안에서 distinct를 한번 수행해주면 될 것 같아서요. 😃

devtaebong added 27 commits December 26, 2024 23:02

remove all

01ac987

docs: 요구사항 정리

3101967

test: 사용자로부터 좌표를 입력받고 해당 입력을 검증한다.

9b95ca9

feat: 사용자로부터 좌표를 입력받고 해당 입력을 검증한다.

46491de

test: MineSweeperMetric으로 MineBoard를 생성한다.

f87cd27

rename: MineSweeperMetric 프로퍼티 네이밍 변경

9de06b8

test: 각 셀은 좌표와 타입(빈칸, 지뢰칼)정보를 갖는다.

e6062de

feat: 각 셀은 좌표와 타입(빈칸, 지뢰칼)정보를 갖는다.

2a597f5

refactor: MineBoard 프로퍼티를 일급 컬렉션으로 변경

49f280c

rename: 입력 메서드 네이밍 변경

39393ff

feat: 지뢰찾기 보드의 높이, 너비, 지뢰개수를 입력받는다.

146c588

rename: 메서드 네이밍 변경

b56ecdd

feat: MineCells 생성 스켈레톤 코드

5086117

test: MineCellGenerator 테스트코드 작성

25a0726

feat: RandomMineCellGenerator 구현

81e1e0b

refactor: 원시타입을 래핑한다.

0f85176

refactor: 일급컬렉션을 value class로 변경

556db61

feat: MineBoard.cellsSize 구현

3facef9

feat: 셀은 open 또는 close 상태를 갖으며, 초기화된 시점의 모든 셀의 상태는 closed 이다.

defa6d0

feat: 좌표 open 구현

ea16636

feat: 입력받은 좌표를 open 한다.

7bbae0c

test: 하나 이상의 MineCell이 Open 되어있는 경우 게임을 중단한다.

2e03534

feat: 사용자의 입력에 따라 게임을 속행 또는 중단한다.

fc793f3

feat: MineBoard의 높이와 너비를 조회한다.

4adf646

feat: 좌표를 입력한 경우 인접한 EmptyCell을 함께 open한다.

74f0bb8

feat: 지뢰찾기 게임 진행 상황과 결과를 콘솔에 출력한다.

1b1b4a7

test: getGameResult() 함수 테스트 추가

fb8fb7e

devtaebong commented Dec 28, 2024

View reviewed changes

Rok93 requested changes Dec 29, 2024

View reviewed changes

docs: 문서 현행화

6059f7d

devtaebong added 4 commits December 29, 2024 17:46

fix: Model로 변환을 Controller 계층에서 수행하도록 변경한다.

ef8b8cf

refactor: 셀의 상태 확인(open or closed)을 isOpened() 함수를 사용하도록 변경

1e7a03f

refactor: 보드의 높이와 너비 계산하는 함수를 읽기전용 프로퍼티를 사용하도록 변경

ce79096

test: MineBoard.getCell(), MineBoard.openCell() 테스트 추가

9d20dcb

Rok93 approved these changes Dec 31, 2024

View reviewed changes

devtaebong added 2 commits December 31, 2024 10:32

fix: isOpen() 메서드 사용하도록 수정

5566c6f

fix: execute의 결과를 List로 변경

884fbcf

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 4단계 - 지뢰 찾기(리팩터링) #491

🚀 4단계 - 지뢰 찾기(리팩터링) #491

devtaebong commented Dec 28, 2024

devtaebong Dec 28, 2024

Rok93 left a comment

Rok93 Dec 29, 2024

Rok93 Dec 29, 2024

Rok93 Dec 31, 2024

Rok93 Dec 29, 2024

Rok93 Dec 29, 2024

devtaebong commented Dec 29, 2024

Rok93 left a comment

Rok93 Dec 31, 2024

Rok93 Dec 31, 2024


		fun getCell(coordinate: Coordinate): Cell = cells.get(coordinate)

		fun openCell(coordinate: Coordinate) = cells.get(coordinate).open()

🚀 4단계 - 지뢰 찾기(리팩터링) #491

Are you sure you want to change the base?

🚀 4단계 - 지뢰 찾기(리팩터링) #491

Conversation

devtaebong commented Dec 28, 2024

Choose a reason for hiding this comment

Rok93 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

devtaebong commented Dec 29, 2024

Rok93 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment