[아크] 1, 2단계 오목 제출합니다. #3

re4rk · 2023-03-16T06:03:44Z

안녕하세요! 리뷰어님! 오목 리뷰동안 잘 부탁드립니다!

galcyurio

흑, 백의 차례를 객체로 표현한 점이 참신하네요 :)
테스트 코드들의 가독성도 좋았습니다.
몇 가지 피드백을 남겨두었으니 확인 후 다시 요청해주세요!

src/main/kotlin/omok/controller/OmokController.kt

src/main/kotlin/omok/domain/OmokBoard.kt

src/main/kotlin/omok/domain/OmokRule.kt

src/main/kotlin/omok/domain/gameState/BlackTurn.kt

src/main/kotlin/omok/domain/state/StoneState.kt

src/test/kotlin/omok/domain/OmokBoardTest.kt

galcyurio

피드백 잘 반영해주셨네요 👍️
33과 같은 규칙을 별도 객체로 분리한 부분과 오목판을 동적으로 만드는 부분이 인상적이었습니다.
이번 피드백은 다음 단계에서 반영해주시면 됩니다.
다음 단계도 힘내세요!

galcyurio · 2023-03-19T05:37:41Z

src/main/kotlin/omok/controller/OmokController.kt

+    private val inputView: InputView = InputView(),
+    private val outputView: OutputView = OutputView(),


생성자로 주입받는 걸로는 충분하지 않습니다.
InputView, OutputView 구현체에 직접적인 의존성이 있기 때문에 interface로 선언하여 느슨하게 결합해주셔야 합니다.

galcyurio · 2023-03-19T05:44:09Z

src/main/kotlin/omok/view/InputView.kt

+    private fun StoneState.name(): String = when (this) {
+        BlackStoneState -> "흑"
+        WhiteStoneState -> "백"
+        else -> ""
+    }


현재는 간단해서 이 정도로도 충분하지만 여러 데이터를 합쳐야 하는 복잡한 객체인 경우에는 StoneStateModel과 같이 별도의 클래스를 선언해주는게 좋습니다.

galcyurio · 2023-03-19T05:46:00Z

src/main/kotlin/omok/view/OmokMap.kt

+object OmokMap {
+    fun makeMap(omokBoard: OmokBoard): List<List<String>> {
+        val omokMap = MutableList(omokBoard.ySize) { MutableList(omokBoard.xSize) { "" } }
+        omokBoard.keys.forEach { point ->
+            omokMap[point.y.value - 1][point.x.value - 1] = getString(omokBoard[point], point.y.value - 1, point.x.value - 1)
+        }
+        return omokMap
+    }
+
+    private fun getString(stoneState: StoneState, x: Int, y: Int): String =
+        getFix(x, y).format(getStone(stoneState, x, y))
+
+    private fun getFix(x: Int, y: Int): String = when {
+        x == X_MIN_INDEX && y == Y_MIN_INDEX -> " %s─"
+        x == X_MIN_INDEX && y == Y_MAX_INDEX -> "─%s "
+        x == X_MAX_INDEX && y == Y_MIN_INDEX -> " %s─"
+        x == X_MAX_INDEX && y == Y_MAX_INDEX -> "─%s "
+        x == X_MIN_INDEX -> "─%s─"
+        x == X_MAX_INDEX -> "─%s─"
+        y == Y_MIN_INDEX -> " %s─"
+        y == Y_MAX_INDEX -> "─%s "
+        else -> "─%s─"
+    }
+
+    private fun getStone(stoneState: StoneState, x: Int, y: Int): String = when {
+        stoneState == BlackStoneState -> "●"
+        stoneState == WhiteStoneState -> "○"
+        x == X_MAX_INDEX && y == Y_MIN_INDEX -> "┌"
+        x == X_MAX_INDEX && y == Y_MAX_INDEX -> "┐"
+        x == X_MIN_INDEX && y == Y_MIN_INDEX -> "└"
+        x == X_MIN_INDEX && y == Y_MAX_INDEX -> "┘"
+        x == X_MAX_INDEX -> "┬"
+        x == X_MIN_INDEX -> "┴"
+        y == Y_MIN_INDEX -> "├"
+        y == Y_MAX_INDEX -> "┤"
+        else -> "┼"
+    }
+
+    private const val X_MIN_INDEX = 0
+    private const val X_MAX_INDEX = X_MAX_RANGE - X_MIN_RANGE
+    private const val Y_MIN_INDEX = 0
+    private const val Y_MAX_INDEX = Y_MAX_RANGE - Y_MIN_RANGE
+}


오목판 자체를 동적으로 변경할 수 있게 만들어주셨군요 👍️

galcyurio · 2023-03-19T05:48:34Z

src/main/kotlin/omok/domain/omokRule/WhiteWinRule.kt

+package omok.domain.omokRule
+
+object WhiteWinRule : OmokRule(WHITE_STONE, BLACK_STONE) {
+    override fun validate(board: List<List<Int>>, position: Pair<Int, Int>): Boolean =


각 Int가 어떤 역할인지 알 수 있도록 Pair를 반환하고 넘기기보다는 별도의 객체로 선언해주세요.

* docs (README): 기능 구현 목록 작성 * feat (XCoordinate): X의 좌표 구현 * feat (YCoordinate): Y의 좌표 구현 * refactor (Coordinate): 매직 넘버 수정 * docs (README): 기능 구현 목록 수정 * feat (StoneState): 돌의 상태 기능 구현 * feat (XLine): X 좌표를 저장 할 수 있다. * feat (YLine): Y 좌표를 저장 할 수 있다. * feat (OmokBoard): 오목 판을 구현한다. * feat (OmokGame): 플레이어들이 오목을 플레이 할 수 있다. * refactor (OmokPoint): x,y를 하나의 클래스로 구현 * refactor (Test): 테스트 케이스 수정 * feat (InputView): 좌표 입력 받는 기능 구현 * feat (OutputView):현재 오목 상태를 출력한다. * refactor (OmokController): 오목게임을 실행 할 수 있도록 변경 * refactor (OmokBoard): 같은 역할 클래스 합침 * refactor (OmokBoard): 상태 패턴적용 * feat (OmokBoard): 33룰 적용 * refactor (OmokRule): 33룰 적용 * feat (OmokRule): 44룰 적용 * refactor (StoneState): 돌 상태 패턴 적용 * refactor (OmokRule): 승리 조건 테스트 케이스 추가 * refactor (OmokBoard): 오목돌 의존성 변경 * refactor (OmokBoard): 게임 진행상황 상태 패턴 적용 * refactor (OmokBoard): 검은돌 금수 적용 * style (Omok): 코딩 컨벤션 적용 * refactor (OmokRule): 규칙 변경 * refactor (OmokBoard): 추상화된 라인 제거 후 보드로 통합 * refactor (OmokBoard): 예외 메세지 추가 * refactor (XCoordinate): x좌표 숫자로 저장되도록 변경 * refactor (StoneState): 뷰영역 역할 도메인에서 제거 * refactor (OmokBoard): 보드의 크기 보드 객체가 가지고 있도록 변경 * refactor (OmokRule): 규칙 구조 변경 * refactor (OmokRule): 어뎁터 패턴 외부로 추출 * refactor (OmokController): 주입식 의존성으로 변경 * refactor (OmokRule): 룰 규칙 상태 개선 * refactor (OutputView): 화면 출력 적응형으로 개선 * refactor (Constants): 상수 구조 개선 * refactor (OmokBoard): OmokBoard 생성구조 개선 * style (Omok): 코드 스타일 맞추기 * refactor (OmokBoard): 보드 관련 테스트 케이스 개선 * refactor (StoneState): 돌 동일성 케이스 추가 * refactor (OmokRuleTest): 테스트 케이스 가독성 개선 * refactor (InputView): 함수명 변경

re4rk added 27 commits March 14, 2023 14:08

docs (README): 기능 구현 목록 작성

2a25cf5

feat (XCoordinate): X의 좌표 구현

ccc4f01

feat (YCoordinate): Y의 좌표 구현

4ef1399

refactor (Coordinate): 매직 넘버 수정

f61d3ea

docs (README): 기능 구현 목록 수정

e41b95c

feat (StoneState): 돌의 상태 기능 구현

a09a843

feat (XLine): X 좌표를 저장 할 수 있다.

63782b8

feat (YLine): Y 좌표를 저장 할 수 있다.

ab4ba06

feat (OmokBoard): 오목 판을 구현한다.

9a68e32

feat (OmokGame): 플레이어들이 오목을 플레이 할 수 있다.

2184e3e

refactor (OmokPoint): x,y를 하나의 클래스로 구현

2d39170

refactor (Test): 테스트 케이스 수정

ecf6bb0

feat (InputView): 좌표 입력 받는 기능 구현

b0822ba

feat (OutputView):현재 오목 상태를 출력한다.

b304773

refactor (OmokController): 오목게임을 실행 할 수 있도록 변경

8018623

refactor (OmokBoard): 같은 역할 클래스 합침

9059607

refactor (OmokBoard): 상태 패턴적용

521d5b3

feat (OmokBoard): 33룰 적용

4dff34c

refactor (OmokRule): 33룰 적용

00379b0

feat (OmokRule): 44룰 적용

05e9ab6

refactor (StoneState): 돌 상태 패턴 적용

d8ac37f

refactor (OmokRule): 승리 조건 테스트 케이스 추가

2342814

refactor (OmokBoard): 오목돌 의존성 변경

c317b4d

refactor (OmokBoard): 게임 진행상황 상태 패턴 적용

cad3078

refactor (OmokBoard): 검은돌 금수 적용

14bf05d

style (Omok): 코딩 컨벤션 적용

aac675f

refactor (OmokRule): 규칙 변경

9ab18b0

galcyurio requested changes Mar 17, 2023

View reviewed changes

re4rk added 2 commits March 17, 2023 15:05

refactor (OmokBoard): 추상화된 라인 제거 후 보드로 통합

c47e793

refactor (OmokBoard): 예외 메세지 추가

eebaed0

re4rk added 15 commits March 17, 2023 15:24

refactor (XCoordinate): x좌표 숫자로 저장되도록 변경

9d12c85

refactor (StoneState): 뷰영역 역할 도메인에서 제거

424d876

refactor (OmokBoard): 보드의 크기 보드 객체가 가지고 있도록 변경

613146b

refactor (OmokRule): 규칙 구조 변경

0ed6c73

refactor (OmokRule): 어뎁터 패턴 외부로 추출

e45a493

refactor (OmokController): 주입식 의존성으로 변경

a9ab5ba

refactor (OmokRule): 룰 규칙 상태 개선

7495a5c

refactor (OutputView): 화면 출력 적응형으로 개선

9609d53

refactor (Constants): 상수 구조 개선

0b59222

refactor (OmokBoard): OmokBoard 생성구조 개선

2f781eb

style (Omok): 코드 스타일 맞추기

874e6fa

refactor (OmokBoard): 보드 관련 테스트 케이스 개선

6b089d3

refactor (StoneState): 돌 동일성 케이스 추가

2167d55

refactor (OmokRuleTest): 테스트 케이스 가독성 개선

e708c94

refactor (InputView): 함수명 변경

2b12291

galcyurio approved these changes Mar 19, 2023

View reviewed changes

galcyurio merged commit ed41ca6 into woowacourse:re4rk Mar 19, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[아크] 1, 2단계 오목 제출합니다. #3

[아크] 1, 2단계 오목 제출합니다. #3

re4rk commented Mar 16, 2023

galcyurio left a comment

galcyurio left a comment

galcyurio Mar 19, 2023

galcyurio Mar 19, 2023

galcyurio Mar 19, 2023

galcyurio Mar 19, 2023

		private val inputView: InputView = InputView(),
		private val outputView: OutputView = OutputView(),

[아크] 1, 2단계 오목 제출합니다. #3

[아크] 1, 2단계 오목 제출합니다. #3

Conversation

re4rk commented Mar 16, 2023

galcyurio left a comment

Choose a reason for hiding this comment

galcyurio left a comment

Choose a reason for hiding this comment

galcyurio Mar 19, 2023

Choose a reason for hiding this comment

galcyurio Mar 19, 2023

Choose a reason for hiding this comment

galcyurio Mar 19, 2023

Choose a reason for hiding this comment

galcyurio Mar 19, 2023

Choose a reason for hiding this comment