code review2 코드 리뷰 - view는 멍청하게!!! 받은 리뷰 중에서 기억에 남는 코드 리뷰 presenter에서는 view에 예외를 넘기고, view에서 예외 종류를 판단하여 그에 따라 적절한 화면을 보여주는 방식입니다. 이러한 방식보다는 presenter에서 직접 view에서 어떤 화면을 보여줘야 할지 구체적으로 지시하는 것이 나을 것 같습니다. view는 가능한 단순하고 멍청하게(?) 만드는 것이 좋습니다. 그렇게 했을 때 presenter를 대상으로 좀더 의미있고 수월하게 테스트 코드를 작성할 수 있습니다. 리뷰를 받기 전의 코드는 다음과 같은 형태로 작성이 되었다. override fun errorMessage(errorCode: Int) { when (errorCode) { 1111 -> { // 1111 에 해당하는 view 처리 } 2222.. 2020. 8. 4. 코드 리뷰 - 간소화가 가능하다면 가능하게!! 받은 리뷰 중에서 기억에 남는 코드 리뷰 1. onActivityResult()를 통해서 값을 전달받을 때에 대한 리뷰 다른 리뷰로도 제안드렸는데, STATUS로 넘기는 값을 String 대신 Int로 처리하면 좀더 편리하게 아래 로직들을 간소화시킬 수 있습니다. 문자열 비교 null 체크 isEmpty 등.. 2020. 7. 30. 이전 1 다음
