코드리뷰 3

코드리뷰 - if else 줄이기

받은 리뷰 중에서 기억에 남는 코드 리뷰 이전 코드 val displayColor: Int get() = if (tier != null) { when { tier.isChoice -> R.color.choice_color else -> R.color.default_color } } else { R.color.default_color } 리뷰 코드 val displayColor: Int get() = when (tier?.isChoice) { true -> R.color.white else -> R.color.black } ? 를 통해서 null 체크 하는걸 알고 사용하면서도 가끔씩 약간 낯선(?) 상황을 만나면 나도모르게 몸에 베어있는 코드로 회귀(?)하게 된다.

카테고리 없음 2020.08.05

코드 리뷰 - view는 멍청하게!!!

받은 리뷰 중에서 기억에 남는 코드 리뷰 presenter에서는 view에 예외를 넘기고, view에서 예외 종류를 판단하여 그에 따라 적절한 화면을 보여주는 방식입니다. 이러한 방식보다는 presenter에서 직접 view에서 어떤 화면을 보여줘야 할지 구체적으로 지시하는 것이 나을 것 같습니다. view는 가능한 단순하고 멍청하게(?) 만드는 것이 좋습니다. 그렇게 했을 때 presenter를 대상으로 좀더 의미있고 수월하게 테스트 코드를 작성할 수 있습니다. 리뷰를 받기 전의 코드는 다음과 같은 형태로 작성이 되었다. override fun errorMessage(errorCode: Int) { when (errorCode) { 1111 -> { // 1111 에 해당하는 view 처리 } 2222..

카테고리 없음 2020.08.04
반응형