코드 품질 개선 - 세션 57: 백문이 불여일견
·2025.12.12 11:00
코드 리뷰 시 조건 필터의 중복이나 누락을 방지하기 위해 호출부와 호출 대상의 코드를 함께 검토해야 한다.
코드 리뷰 중 특정 조건에 따른 조기 반환(early return) 코드가 추가될 때, 해당 조건이 다른 곳에서도 동일하게 적용되고 있는지 확인하는 것이 중요하다.
예를 들어, 메시지 전송 함수에 isBlank() 체크를 추가한다면, 이 함수를 호출하는 곳이나 내부에서 사용하는 파라미터 변환 함수에 이미 유사한 필터가 있는지 검토해야 한다. 조건이 서로 겹치거나 누락되면 논리적 오류가 발생할 수 있다.
이러한 중복을 방지하기 위해 Kotlin과 같은 정적 타입 언어에서는 에러 상태를 나타내는 별도의 객체 대신 null을 사용하여 에러 타입을 명확히 구분하는 것이 효과적이다.
더 나은 리뷰를 위해 IDE를 활용하여 변경된 코드의 **호출부(caller)**와 호출 대상(callee), 그리고 유사한 역할을 하는 클래스들을 함께 살펴보는 '망원경(Telescope)' 관점의 리뷰가 권장된다.
GitHub의 Pull Request를 로컬 환경에서 직접 확인하려면 git fetch 명령어를 통해 해당 브랜치를 체크아웃하여 리뷰의 정확도를 높일 수 있다.
이 요약은 원문 이해를 돕기 위한 큐레이션입니다. 저작권은 원저작자에게 있으며, 정확한 내용과 맥락은 원문을 확인하세요.
요약 오류, 출처 표기 문제, 삭제 요청은 문의 · 건의로 알려주세요.