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

 

받은 리뷰 중에서 기억에 남는 코드 리뷰

 

presenter에서는 view에 예외를 넘기고, view에서 예외 종류를 판단하여 그에 따라 적절한 화면을 보여주는 방식입니다.

이러한 방식보다는 presenter에서 직접 view에서 어떤 화면을 보여줘야 할지 구체적으로 지시하는 것이 나을 것 같습니다.
view는 가능한 단순하고 멍청하게(?) 만드는 것이 좋습니다.

그렇게 했을 때 presenter를 대상으로 좀더 의미있고 수월하게 테스트 코드를 작성할 수 있습니다.

 

 

리뷰를 받기 전의 코드는 다음과 같은 형태로 작성이 되었다.

override fun errorMessage(errorCode: Int) {
    when (errorCode) {
        1111 -> {
            // 1111 에 해당하는 view 처리
        }
        2222 -> {
            // 2222 에 해당하는 view 처리
        }
        3333 -> {
            // 3333 에 해당하는 view 처리
        }
        else -> {
            // 나머지 view 처리
        }
    }
}