정적 테스트와 코드리뷰 / Code Inspection

 

Code Review

 

코드를 실행하지 않고 사람이 검토해서 코드 상의 잠재적 결함을 찾고 개성하는 과정을 의미.

적은 투자로 큰 효과를 얻을 수 있는 테스트 기법.

코드 리뷰의 효율성 향상을 위해 정적 테스트 도구나 코드 인스펙션 도구를 사용.

표준 프레임워크에서는 '코드 인스펙션 도구'라는 명칭을 사용.

정적 테스트는 정적 분석을 말하며 일반적으로 코드 리뷰를 의미. 

성공적인 코드 리뷰를 위해서는 발견한 것들을 공유하고, 조금씩 자주 수행하는 것이 좋음.

 

오프라인 코드 리뷰 : 여러명이 모여 함께 코드를 리뷰.

온라인 코드 리뷰 : 이메일 등 도구를 사용해 온라인상 피드백.

 

 

주요 체크리스트

• 변수명이 다른 사람이 봐도 이해할 수 있는지
• 로직이 어떤 기능을 하는지 알 수 있는지
• Null Check가 분명한지
• 정보 은닉의 여부
• 무의미한 성능 저하 코드가 있는지
• 메소드가 유추할 수 있는것 외의 동작을 하는지
• 임시 변수가 지나치게 많은지
• 중첩된 if문이 지나치게 많거나 가독성이 떨어지는지
• 상속과 구현이 제대로 이루어졌는지 

  ---

Code Inspection 

정의된 룰을 기반으로 개발자가 작성한 소스 코드를 검사해 오류와 위험 요인을 식별해서 알려줌

 

 

코드 인스펙션 도구

• 소스코드의 구문 오류를 검사하는 기능을 제공
• 실행 시 발생 가능한 오류를 찾아내는 기능 제공
• 실행 시 사용되지 않는 구문을 찾아내는 기능을 제공
• 익스펙션 수행 결과를 Exel, HTML 등의 문서 형식으로 제공
• 룰을 정의하는 API를 사용해 새로운 룰을 추가 정의 가능
• 오픈소스 도구로는 PMD, FindBugs, Checkstyle 등이 존재
• 표준프레임워크에서는 'PMD'로 선정
• 이클립스에 PMD 플러그인이 포함되어 배포됨.

 

 

---

<전자정부 표준프레임워크> 책의 내용을 정리한 것입니다.

p.92-94