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

코드 리뷰에 대한 고민을 할 필요가 있어서 기록해 놓는다. 
 

- xper 의 메일로 받은 내용임. ( id : miniway 님 ) 

코드 리뷰 툴 :  
code collaborator (http://smartbear.com/products/development-tools/code-review/features/)

참고 블로그 : http://javawork.egloos.com/2858102

(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 등).