대부류 - [Dev u Ryu]

받은 리뷰 중에서 기억에 남는 코드 리뷰 이전 코드 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 체크 하는걸 알고 사용하면서도 가끔씩 약간 낯선(?) 상황을 만나면 나도모르게 몸에 베어있는 코드로 회귀(?)하게 된다.

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

받은 리뷰 중에서 기억에 남는 코드 리뷰 1. onActivityResult()를 통해서 값을 전달받을 때에 대한 리뷰 다른 리뷰로도 제안드렸는데, STATUS로 넘기는 값을 String 대신 Int로 처리하면 좀더 편리하게 아래 로직들을 간소화시킬 수 있습니다. 문자열 비교 null 체크 isEmpty 등.. ‌