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