코드리뷰의 중요성: 리팩터링 중 재발생한 버그 사례
Source
Evernote/Advanced View Required/코드리뷰.md
Summary
작은 리팩터링 과정에서 코드리뷰를 생략한 결과, 이전에 수정되었던 버그가 재발생한 사례를 통해 코드리뷰의 중요성을 강조합니다. 개발자가 Sign 메서드를 삭제하고 직접 호출하는 과정에서, 원래 수정해야 할 originalFile 인자가 다시 사용되어 에러 없이 실행되지만 잘못된 파일에 작업이 수행되는 치명적인 버그가 발생했습니다. 이 사례는 사소한 변경이라도 코드리뷰를 통해 잠재적 오류를 사전에 발견할 수 있음을 보여줍니다.
Key Points
- 리팩터링 시 기존 버그가 재발생할 수 있으며, 코드리뷰가 이를 방지하는 핵심 수단입니다.
- 에러가 발생하지 않더라도 논리적으로 잘못된 파일 경로 사용과 같은 버그는 테스트만으로는 발견하기 어려울 수 있습니다.
- 작은 코드 변경이라도 코드리뷰를 생략하면 의도치 않은 부작용이 발생할 위험이 있습니다.