The motive of this thesis is to practice code review in depth. It is generally said that the definition of 'review' is to inspect work product for the purpose of finding defects by manual methods, such as inspection by human eyes, before the stage of 'test,' and that review could be conducted by its developer or by the other party. However, in this thesis, the term 'review' means 'to inspect the work product by the developer, not by the other party, using both traditional manual methods and automation tools.' The targets of this code review is PMCenter, which was one of the MSE 2003-2004 studio projects, and the focus is on finding defects not only in view of development standards, i.e., design rule and naming rule, but also in view of quality attributes of PMCenter, i.e., performance and security.
From review results, a few lessons are teamed. First, defects which had not been found in the test stage of PMCenter development could be detected in this code review. These are hidden defects that affect system quality and that are difficult to find in the test. If the defects found in this code review had been fixed before the test stage of PMCenter development, productivity and quality enhancement of the studio project would have been improved. Second, manual review takes much longer than an automated one. In this code review, general check items were checked by the automation tool, while project-specific ones were checked by the manual method. If project-specific check items also could be checked by the automation tool, code review and verification work after fixing the defects would be conducted very efficiently. Reflecting on this idea, an evolution model of code review is studied, which eventually seeks fully automated review as an optimized code review.
본 논문은 2003 - 2004 소프트웨어 공학 석사과정 (MSE 2003-2004)에서 다룬 코드 리뷰를 보다 심도 있게 실습해 보는데 동기를 두고 있다. 일반적으로 소프트웨어 리뷰는 사람의 눈에 의한 즉 수동적으로 결함을 테스트 단계 이전에 찾아내는 것을 말한다. 다른 사람들이 우리가 작성한 소스코드의 결함을 찾아내기 위하여 그 소스코드를 조사할 수 있을 뿐만 아니라 우리 자신이 직접 조사할 수도 있다. 본 논문에서는 수동적 방법뿐 아니라 자동화 툴을 이용하여 실행될 수 있는 코드 리뷰 방법이 설정되며, 리뷰 대상은 MSE 2003-2004 스튜디오 프로젝트 중의 하나였던 PMCenter (Project Management Center)로 한다. 리뷰의 초점은 개발 표준 즉 디자인 표준과 명명규칙을 따라서 코딩이 되었는지 여부와 PMCenter의 품질요소인 성능(Performance)과 안전성 (Security) 측면에서 결함을 찾아내는 것이다. 본 코드 리뷰의 결과로부터 몇 가지 교훈을 얻을 수 있었다. 첫째, PMCenter의 개발과정 중 테스트 단계에서는 발견되지 않았던 결함들이 본 리뷰 활동으로부터 발견되었다. 그 결함들은 시스템 품질에 영향을 주는 숨은 결함들이라고 말할 수 있으며 PMCenter 개발과정 중의 코드 리뷰 활동을 통하여 발견되었다면 프로젝트의 생산성과 품질을 향상시켰을 것으로 판단된다. 둘째, 사람이 직접 결함을 찾아내는 수동 리뷰는 자동화 툴에 의한 리뷰보다 현저히 많은 시간을 필요로 하였다. 본 실습에서는 어떤 프로젝트에도 적용될 수 있는 일반 항목들은 자동화 툴에 의하여 리뷰가 되었고, 반면에 PMCenter 프로젝트에만 적용될 수 있는 특수 항목들은 수동적으로 리뷰가 이루어졌다. 그 특수 항목들 또한 자동화 툴에 의하여 리뷰가 될 수 있다면 리뷰 활동과 결함 수정 후 검증 작업이 매우 효율적으로 진행될 수 있을 것이다. 이러한 착상에 기반하여 코드 리뷰의 진화 모델이 고찰되었으며 궁극적으로 완전 자동화 된 코드 리뷰 방법이 최적화된 코드 리뷰 모델로서 기술되었다.
