[부나] 3, 4단계 오목 제출합니다. #32
Conversation
app/src/main/java/woowacourse/omok/repository/OmokRepository.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/woowacourse/omok/database/repository/OmokRepository.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/woowacourse/omok/database/repository/OmokRepository.kt
Outdated
Show resolved
Hide resolved
| private fun continuePreviousGame() { | ||
| omokRepo.getAll { | ||
| if (it.count % 2 == 1) toggleTurnHolder(StoneColorModel.BLACK) | ||
| while (it.moveToNext()) { | ||
| val col = it.getInt(it.getColumnIndexOrThrow("x")) | ||
| val row = it.getInt(it.getColumnIndexOrThrow("y")) | ||
| val index = row * Omok.OMOK_BOARD_SIZE + col | ||
| val omokIv: ImageView = boardImageViews()[index] | ||
| val putResult = omok.put { Point(row, col) } | ||
| if (putResult is PutSuccess) { | ||
| drawStoneOnBoard(omokIv, putResult.stoneColor) | ||
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
UI layer에 ui, domain, data 코드가 모두 섞인 스파게티 코드입니다.
아래와 같은 순서로 개선해보세요.
- MainActivity에서 InputView와 OutputView를 구현하도록 만들어보세요.
- OmokController에서 OmokInputView, OmokOutputView가 아닌 InputView, OutputView interface에 의존하도록 만들어보세요.
- MainActivity에서 Controller를 사용하도록 만드세요.
- OmokRepository.kt
|
안녕하세요 토리! 최대한 고민하며 코드를 작성했는데, 보시기에 DM으로 제안해주신 방향과 다르거나 미숙한 부분이 많을 것으로 생각�됩니다. 🥲 |
| import repository.Repository | ||
|
|
||
| class AndroidOmokController( | ||
| private val controller: ConsoleOmokController, |
There was a problem hiding this comment.
같은 layer 끼리 서로 의존성을 가지지 않아야 합니다.
공통되는 로직이 있다면 해당 로직만 별도 객체로 분리해주세요.
There was a problem hiding this comment.
템플릿 메서드를 사용하여 공통되는 로직을 상위 클래스로 옮기고, 변경이 필요한 로직에 대해서는 별도의 서브 클래스에서 구현하도록 변경하였습니다!
There was a problem hiding this comment.
앗.. 지금 리뷰를 다시 읽어 봤는데 공통되는 기능을 하나의 클래스로 분리해서 컴포지션하기를 의도하신 걸까요?!
There was a problem hiding this comment.
네, Composition을 의도했지만 상속을 통한 재사용도 좋습니다!
presentation/src/main/java/woowacourse/omok/presentation/presenter/omok/OmokContract.kt
Show resolved
Hide resolved
| print(ASK_POSITION_MESSAGE) | ||
| val colRow = readln() | ||
| if (colRow.length !in POSITION_INPUT_RANGE) { | ||
| println(INVALID_FORMAT_ERROR_MESSAGE) | ||
| readPosition(onPutStone) | ||
| CoroutineScope(Dispatchers.Main).launch { readPosition() } |
There was a problem hiding this comment.
suspend fun이므로 호출자의 CoroutineScope을 사용하면 되어서 새롭게 CoroutineScope을 만들지 않아도 됩니다.
| import repository.Repository | ||
|
|
||
| class AndroidOmokController( | ||
| private val controller: ConsoleOmokController, |
There was a problem hiding this comment.
네, Composition을 의도했지만 상속을 통한 재사용도 좋습니다!
1, 2단계 커밋이 포함되어 있어서 제외하고 다시 PR을 보냅니다!