[STEP 4] 💣지뢰찾기💥 리팩터링 #492

y2gcoder · 2024-12-28T19:56:05Z

안녕하세요 남준님!

오랜만에 인사드리게 되었습니다😅

크리스마스는 잘 보내셨는지요?🎄

마지막 단계다 보니 그동안 배웠던 걸 적용해보고자 노력했습니다!

항상 양질의 피드백 주시는 남준님께 감사드립니다👍

포기하지 않고 마무리까지 열심히 해보겠습니다!

감사합니다

양영근 올림

…nes 로 변경

… 존재한다

…생성할 수 있음

…표시할 수 있다

…dDslMaker 추가

… 셀을 열 수 있도록 처리

… Area) 추가

…jacent() 추가

- open() 메서드 간소화: findAllShouldOpen()으로 책임 이전 - findAllShouldOpen(): BFS를 통해 열어야 할 Set<Location> 계산 - applyOpen(): Set<Location>을 인덱스 기반 Set 으로 변경 && 가변 리스트로 변경해서 계산 후 불변 리스트로 반환함으로써 성능 향상

…r 와 구현체에 위임

namjackson · 2025-01-12T11:49:06Z

아무래도 리팩터링 과정이라서, 크게 코멘트 드릴부분이 없을거같아요 :)
이대로 마무리 하시려면 그냥 리뷰요청 주시거나, 머지 하도록할게요 :)
코드 정리하셔도 되고, 추가로 궁금한점있으시면 코멘트 달아주셔도 좋아요 !

namjackson

안녕하세요 영근님! 새해복 많이 받으세요!
개인적으로 연초에 일정이 바쁘다 보니까 피드백이 늦었네요 :)
지뢰찾기 마지막 단계 고생많으셨습니다!
코멘트 참고해주세요!

namjackson · 2025-01-12T10:51:00Z

src/main/kotlin/tdd/minesweeper/domain/Cell.kt

@@ -0,0 +1,13 @@
+package tdd.minesweeper.domain


개인적으로 가독성을 위해 domain 패키지 내에서도
기능 등으로 묶어서 패키지를 관리해봐도 좋을거같아요 :)
cell관련 객체를 모아보는건 어떨까요?

namjackson · 2025-01-12T11:00:50Z

src/main/kotlin/tdd/minesweeper/domain/Cell.kt

+    val adjacentMines: AdjacentMines?
+
+    fun isOpen(): Boolean
+
+    fun hasMine(): Boolean
+
+    fun open(): Cell
+
+    fun isExpandableToAdjacent(): Boolean


Cell을 인터페이스 로 분리하셧네요!
MineCell에서는 adjacentMines는 null 일수밖에 없고,
open, isExpandableToAdjacent 또한 ClosedCell에서만 필요한 함수이기도해요 :)
객체지향 원칙중 인터페이스 분리 원칙에 어긋날수도 있다는 생각이 들어요,
더 작은 단위로 인터페이스를 분리해볼수 있을거같아요!
(하지만 언제나, trade-off 비용이 드는건 어쩔수 없을거 같아요!
오히려 조금더 복잡해질수 있다는 단점이 있어요!
복잡하지 않고, 인터페이스를 분리할수 있는 지점을 영근님의 기준으로 잡아보면 어떨까요:) )

namjackson · 2025-01-12T11:11:54Z

src/main/kotlin/tdd/minesweeper/domain/dsl/BoardBuilder.kt

+import tdd.minesweeper.domain.strategy.DefaultBoardCellsCreator
+
+@BoardDslMaker
+class BoardBuilder(private val boardCellsCreator: BoardCellsCreator = DefaultBoardCellsCreator()) :


Dsl 활용 👍

namjackson · 2025-01-12T11:24:29Z

src/main/kotlin/tdd/minesweeper/domain/strategy/DefaultShouldOpenLocationFinder.kt

+class DefaultShouldOpenLocationFinder : ShouldOpenLocationFinder {
+    override fun findAllShouldOpen(
+        board: Board,


Board의 open에서 연관된 Cell 관련 부분을 분리해 주셨네요 👍
Board에서도 ShouldOpenLocationFinder를 의존하고있고,
ShouldOpenLocationFinder에서도 Board를 의존하고 있어요 :)
Board와 ShouldOpenLocationFinder는 결합도가 높다고 볼수도 있을거 같아요 :)
양방향 의존성은 가독성 측면이나, 유지보수 측면에서 어려울수 있어요

board가 아닌 Cells를 받아서 처리하도록 수정할수 있을거 같아요

namjackson · 2025-01-12T11:31:16Z

src/main/kotlin/tdd/minesweeper/domain/strategy/DefaultBoardCellsCreator.kt

+        val allLocations = createAllLocations(area)
+
+        // 지뢰 위치 구하기
+        val mineLocations = calculateMineLocations(allLocations, inputManualMineLocations, countOfMines)


calculateMineLocations는 분리해볼수 있지 않을까요?
DefaultBoardCellsCreator의 관심사는 MineLocations 일거같아요
MineLocations이 inputManual인지, 랜덤 location인지는 몰라도
Cells을 만드는데 영향이 없지는 않을까요?

namjackson · 2025-01-12T11:40:07Z

src/main/kotlin/tdd/minesweeper/domain/strategy/DefaultBoardCellsCreator.kt

+        val closedCells = createMinePlantedCells(allLocations, mineLocations)
+
+        // 지뢰 인접 위치 표시
+        val updatedCells = markOfAdjacentMines(closedCells, allLocations, mineLocations)


이미 closedCells에는 allLocations, mineLocations에 대한 정보가 있지 않나요?
지뢰 인접 위치 표시 기능에 3개의 리스트가 필요할까요? :)

namjackson · 2025-01-12T11:47:33Z

src/test/kotlin/tdd/minesweeper/domain/AdjacentMinesTest.kt

+import io.kotest.core.spec.style.BehaviorSpec
+import io.kotest.matchers.shouldBe
+
+class AdjacentMinesTest : BehaviorSpec({


새롭게 TDD로 다시한번 작성하신부분은 어떠셨나요?

저는 TDD를 처음 도입하다보면 아무래도 처음 작성한 테스트케이스에대한 테스트 코드를 먼저 작성하고,
이에 만족하기위해 구현하다보니, 설계가 어긋나거나 꼬여서, 어려웠던 기억이 나네요,

새로 한번 작성하게되니 기존에 어느정도 도메인과 설계에 대한 기반이 쌓여서
조금더 자연스럽게 TDD를 할수 있엇던거 같아요 :)

y2gcoder · 2025-01-17T07:38:55Z

아무래도 리팩터링 과정이라서, 크게 코멘트 드릴부분이 없을거같아요 :) 이대로 마무리 하시려면 그냥 리뷰요청 주시거나, 머지 하도록할게요 :) 코드 정리하셔도 되고, 추가로 궁금한점있으시면 코멘트 달아주셔도 좋아요 !

@namjackson

남재님 새해 복 많이 받으십쇼 :)
확인이 늦어서 죄송합니다!

역시 기대를 저버리지 않고 양질의 코멘트를 남겨주셨군요... 감사합니다😊

마지막 피드백은 머지해주시면 포크한 제 레포에서 개인적으로 반영해보겠슴다!

벌써 코스가 끝난 지도 한 달이 다 되어 가는데 남재님을 계속 붙잡아두는 것 같아 죄송합니다🥹

그동안 지저분한 제 코드 봐주셔서 너무 감사했습니다!

남재님께 배운 것 까먹지 않고 계속 사용해서 좋은 코드를 작성하기 위해 노력하겠습니다 👍

y2gcoder added 30 commits December 24, 2024 23:11

feature(minesweeper): 닫힌 셀은 열린 상태가 아님

67d6034

feature(minesweeper): 지뢰 셀은 열린 상태가 맞음

87f9a81

feature(minesweeper): 숫자 셀은 열린 상태가 맞음

fcf55c6

feature(minesweeper): 지뢰 셀은 열면 지뢰 셀이다

e795376

feature(minesweeper): 숫자 셀은 열면 숫자 셀이다

785ab43

feature(minesweeper): 지뢰가 없는 닫힌 셀을 열면 숫자 셀이 된다

15419fa

feature(minesweeper): 지뢰가 있는 닫힌 셀을 열면 지뢰 셀이 된다

cbef00b

feature(minesweeper): 지뢰는 인접 지뢰 수를 0 ~ 8까지 가질 수 있다

7e95717

refactor(minesweeper): 지뢰 셀의 이름을 MineCell 로 변경 && 인접 지뢰 수도 AdjacentMi…

18a37e0

…nes 로 변경

feature(minesweeper): 지뢰가 없을 때 닫힌 셀을 열면 똑같은 인접한 지뢰 수를 가진 숫자 셀이 된다

fae4162

feature(minesweeper): AdjacentMines 값 객체 생성 및 적용

167164e

feature(minesweeper): 각 셀들은 지뢰가 있는지 알 수 있다

4c4d663

feature(minesweeper): 보드는 셀 목록을 받아 생성할 수 있다

5f25ce3

feature(minesweeper): 보드는 영역과 셀 목록을 받아 생성할 수 있다

873606a

feature(minesweeper): 높이, 너비, 지뢰 개수를 받아 보드를 만들 수 있다

53bf394

feature(minesweeper): 높이, 너비, 지뢰 개수를 받아 보드를 만들면 지뢰 개수만큼의 지뢰를 가진 닫힌 셀이…

1cbe37d

… 존재한다

feature(minesweeper): 인접 지뢰 수는 1씩 증가할 수 있음 && 인접 지뢰 수끼리는 서로 비교할 수 있음

3595790

feature(minesweeper): 닫힌 셀은 인접 지뢰 수를 업데이트할 수 있음

bf4da51

feature(minesweeper): BoardBuilder 는 너비, 높이, 지뢰 개수와 수동 지뢰 위치를 받아 보드를 …

346b425

…생성할 수 있음

feature(minesweeper): BoardBuilder 에서 보드판을 생성할 때 지뢰가 아닌 셀들에 인접 지뢰 수를 …

6a01bd9

…표시할 수 있다

refactor(minesweeper): BoardBuilder를 dsl 패키지로 이동

b88f181

feature(minesweeper): Board를 DSL로 생성하기 위해 BoardCreator.board(), @Boar…

7db19b6

…dDslMaker 추가

feature(minesweeper): 보드 셀 생성 책임을 BoardCellsCreator 로 위임

066477a

feature(minesweeper): 위치에 대한 인접 8방향 위치를 구하는 책임을 지닌 AdjacentDirection 추가

99b27e6

refactor(minesweeper): DefaultBoardCellsCreator 리팩토링

3d23580

refactor(minesweeper): Builder 를 dsl 패키지로 이동

35a1b1f

refactor(minesweeper): BoardCellsCreator 와 구현체를 strategy 패키지로 이동

85882d5

feature(minesweeper): 보드는 보드 내에 없는 위치의 셀을 열려고 하면 예외를 던짐

7d0f692

feature(minesweeper): BoardBuilder, BoardCreator에 openAt()을 추가하여 수동으로…

17d508f

… 셀을 열 수 있도록 처리

test(minesweeper): 보드의 이미 열린 셀을 열면 보드의 변화가 없음

1c667cf

y2gcoder added 17 commits December 27, 2024 11:28

feature(minesweeper): 보드에서 지뢰를 가진 닫힌 셀을 열면 해당 셀만 연다

e89e17a

feature(minesweeper): 보드에서 이웃한 8방향 중에 지뢰가 있는 셀을 열면 해당 셀만 열린다

b0abb91

feature(minesweeper): Location 에 인덱스로 바꾸는 것과 인덱스를 Location으로 바꾸는 메서드 추가

92b9dfa

test(minesweeper): 이웃한 8방향 중에 지뢰가 없는 셀을 열면 이웃한 지뢰가 없는 셀을 모두 연다

8b6285c

feature(minesweeper): 위치가 보드 영역 내에 있는 유효한 위치인지 체크할 수 있는 isValid(area:…

f1d4677

… Area) 추가

feature(minesweeper): 셀이 인접 셀까지 확장해서 열 수 있는 셀인지 확인하는 isExpandableToAd…

34f8cd4

…jacent() 추가

feature(minesweeper): 열어야 할 셀 위치 목록을 계산하는 책임을 ShouldOpenLocationFinde…

b87b41b

…r 와 구현체에 위임

feature(minesweeper): Game은 높이, 너비, 지뢰 개수를 받아 생성한 보드를 가지고 만들어진다

a1aed5b

feature(minesweeper): 보드는 닫힌 셀 수를 구할 수 있다

ed5a968

feature(minesweeper): 보드는 닫힌 셀 수를 구할 수 있다

c7cd98a

feature(minesweeper): 보드의 닫힌 셀 수와 총 지뢰 수가 같으면 게임은 승리한 것이다

ce91b02

feature(minesweeper): 게임의 패배, 계속 조건 추가

855aa14

feature(minesweeper): 게임 플레이를 위한 ui, 메인 클래스 추가

7cd9ec2

refactor(minesweeper): ConsoleInputView 에 위치 입력 후 한 칸 띄우도록 println() 추가

22e236f

feature(minesweeper): OutputView 에 에러 메시지 출력용 printError 추가

905cf24

feature(minesweeper): 입력값 유효성 검사 실패시 재시도 처리

13b4815

namjackson requested changes Jan 12, 2025

View reviewed changes

y2gcoder requested a review from namjackson January 17, 2025 07:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[STEP 4] 💣지뢰찾기💥 리팩터링 #492

[STEP 4] 💣지뢰찾기💥 리팩터링 #492

y2gcoder commented Dec 28, 2024

namjackson commented Jan 12, 2025

namjackson left a comment

namjackson Jan 12, 2025

namjackson Jan 12, 2025

namjackson Jan 12, 2025

namjackson Jan 12, 2025

namjackson Jan 12, 2025

namjackson Jan 12, 2025

namjackson Jan 12, 2025

y2gcoder commented Jan 17, 2025 •

edited

Loading

[STEP 4] 💣지뢰찾기💥 리팩터링 #492

Are you sure you want to change the base?

[STEP 4] 💣지뢰찾기💥 리팩터링 #492

Conversation

y2gcoder commented Dec 28, 2024

namjackson commented Jan 12, 2025

namjackson left a comment

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

namjackson Jan 12, 2025

Choose a reason for hiding this comment

y2gcoder commented Jan 17, 2025 • edited Loading

y2gcoder commented Jan 17, 2025 •

edited

Loading