개발의 즐거움/Code Review

code Review - xper 의 메일로 받은 내용

[짱가™] 2011. 11. 3. 18:27


코드 리뷰에 대한 고민을 할 필요가 있어서 기록해 놓는다. 
 
- xper 의 메일로 받은 내용임. ( id : miniway 님 ) 

코드 리뷰 툴 :  
code collaborator (http://smartbear.com/products/development-tools/code-review/features/)
(Jira, Confluence 를 만든 Atlassian 의 코드리뷰를 통한 lessons learned)
http://blogs.atlassian.com/developer/2010/03/code_review_in_agile_teams_part_ii.html

* 코드리뷰에 대한 오해
- 버그를 발견하는 것을 보장하진 않는다
- 코드의 결함을 찾는 것이 목적이 아니라, 서로 배우고 가르쳐 주고, 팀의 협업능력을 높여주는 것이어야 한다. 
  
* 코드리뷰가 잘 되려면
- 너무 많은 절차와 규칙을 만들지 마라. 절차를 아주아주아주 간단하게 하라
- 강요하지 마라. 대신 Encourage 하라
- 모든 코드 commit 을 리뷰하도록 한다거나 하는 형태로 Micro - Manage 를 하지마라
- 개개인의 작업 흐름을 끊지마라
- 코드리뷰를 통해 발견한 것들을 널리 공유하라
- 코드리뷰를 늦게 하는것은 안하는 것보다 나쁠수 있다. Iteration 에 포함시켜라
- 한꺼번에 덜하기보다는 조금씩 자주하라
- 툴에 얽메이지 마라, 중요한것은 개발자들이 서로 대화를 하고 코드를 공유하는 것이다
- 너무 많은 리뷰어를 참여시키지 마라. 2-4 명이 적당하다

* 어떻게 잘 되고 있는지 알수 있는가
- 쉽진 않다, 사실 필요없을 수도/불가능 할수도 있다, metric 에 집착할 필요는 없다.
- 장기간의 이득은 측정할 수 없지만 많다
- Simple Metric 들이면 충분할 수 있다. (리뷰에 사용된 시간, 리뷰 comment 등).