🚀 1단계 - 지뢰 찾기(그리기)

[mins99]

안녕하세요 리뷰어님!
드디어 마지막 미션 지뢰찾기가 시작되었네요~! 하지만 마지막이라고 안주하지 않고 완주 해보겠습니다 🏃‍♀️
이미 알고 있는 게임이라 그런지 블랙잭에 비해서는 수월하게 구현했던것 같습니다
커밋을 보시면 아시겠지만 엄청나게 단순하게 구현했다가.. 조금 더 객체를 나눌 수 있을것 같아서(+ 객체지향 생활 체조 원칙을 지키고자) 나누어보았습니다!

참고사항

패키지 구조

• board(지뢰찾기 게임 판)와 cell(칸 하나)로 패키지를 나누고 그 아래에 관련된 객체들을 두는 구조
• cell의 경우 가능한 케이스가 빈칸 또는 지뢰 로 되어있다 보니 인터페이스(Opening)를 사용하여 유연하게 구성

고민거리

일급컬렉션 사용시 view 영역에 데이터를 가져올 때
fun minefield(): Array<Array<Opening>> = minefield.minefield 와 같이 한 번 더 꺼내게 되는데,
혹시 객체지향적으로 더 좋은 방법이 있다면 알려주시면 감사하겠습니다! 🙇

열심히 해보겠습니다! 잘 부탁드립니다~!

[laco-dev]

안녕하세요. 지뢰찾기 미션동안 잘 부탁드립니다. 🙇🏻

그동안 해온 미션들에서 학습한 개념들이 지뢰찾기에서도 이어서 적용되야 합니다.
랜덤 테스트, 추상화 등 배웠던 것들을 기억하면서 다시 고민 해 보면 좋을 만한 내용들을 남겼습니다.

[laco-dev]

일반적으로 생각했을 때, getPosition() 이라는 함수가 무엇을 반환할 것이라고 예측할까요?
과연 무작위 값을 반환할 것으로 기대할 수 있을까요?

저는 그 이전에 행, 열 이라는 값 객체가 Position ? 이라는 위치라는 것이 무엇을 의미하는 것인지 부터 이해가 잘 되지 않을 것 같아요.

어떠한 x, y 좌표에 지뢰를 심는다는 역할로 만들어 보는건 어떨까요?

// position 위치에 지뢰를 심는다.
fun mine(position: Position) { ... }

[laco-dev]

이어서, 지뢰를 심는다는 동작이 현재는 무작위 값에 의존이 되어 있어요.
그동안 미션에서 해왔던 것 처럼 무작위 값은 테스트를 매우 힘들게 하고 객체가 자율적으로 동작을 수행하는 것을 신뢰할 수 없도록 만듭니다.

앞서 위치라는 개념을 사용해본다면, 아래와 같은 순서를 참고 해 보시면 도움이 될 것 같습니다.

1. 5 x 5 크기의 보드를 준비한다 // Minefield
2. 지뢰의 위치를 판단한다. // MineSweeperGame
3. 지뢰의 위치는 랜덤으로 받을 수도 있고, 명시적으로 받을 수도 있다 (interface)
4. 주어진 지뢰의 위치만큼 지뢰를 심는다. ( ??? )

[laco-dev]

cell의 경우 가능한 케이스가 빈칸 또는 지뢰 로 되어있다 보니 인터페이스(Opening)를 사용하여 유연하게 구성

빈칸, 또는 지뢰라는 의미가 Opening 이라는 타입으로 잘 이루어져 있을까요?
Opening 이라는 것은 어떠한 상태? 처럼 느껴지지 않을까요?
isOpening() ...

지뢰찾기의 기본 게임 구성으로는 보드 판에는 N개의 셀(Cell)이 존재한다 을 표현하고는 합니다.
그렇다면 이를 대입해 봤을 때 보드 판에는 N개의 Opening이 존재한다 로 비교 해 보실 수 있습니다.

[laco-dev]

"*" 을 출력하는 것은 UI의 관심사 이지 않을까요?
가령 지뢰를 별이 아닌 그래픽 모양의 지뢰로 출력한다고 하면, 도메인 모델의 value는 아무런 역할을 하지 못하게 될거에요.

[laco-dev]

일급컬렉션 사용시 view 영역에 데이터를 가져올 때
fun minefield(): Array<Array> = minefield.minefield 와 같이 한 번 더 꺼내게 되는데,
혹시 객체지향적으로 더 좋은 방법이 있다면 알려주시면 감사하겠습니다! 🙇

결과(모든 셀을 출력)에만 집중하는 대신 필요한 기능을 작게 접근 해 볼 수록 좋습니다.
가령 행과 열의 크기를 알고 있을 때 우리가 원하는 것은, "해당 위치에 있는 셀을 알려줘" 입니다.

fun getCellByPosition(position: Position): Cell { ... }

val position = Position(x, y)
val cell: Cell = getCellByPosition(position)

[laco-dev]

Array<Array<Opening>> 타입이라면, 어떠한 위치의 셀에 접근하기 위해 다소 불필요한 순회가 많이 필요할 수 있습니다.

특정 위치를 보다 효율적으로 접근하기 위한 자료구조를 고려 해 볼 수 있지 않을까요?