프로그래밍/C++

코드 리뷰

이재만박사 2017. 7. 11. 16:17

* 코드리뷰에 대한 오해


- 버그를 발견하는 것을 보장하진 않는다


- 코드의 결함을 찾는 것이 목적이 아니라, 서로 배우고 가르쳐 주고, 팀의 협업능력을 높여주는 것이어야 한다. (그렇지 않으면 팀이 깨어진다)



* 코드리뷰가 잘 되려면


- 너무 많은 절차와 규칙을 만들지 마라. 절차를 아주아주아주 간단하게 하라


- 강요하지 마라. 대신 Encourage 하라


- 모든 코드 commit 을 리뷰하도록 한다거나 하는 형태로 Micro - Manage 를 하지마라


- 개개인의 작업 흐름을 끊지마라


- 코드리뷰를 통해 발견한 것들을 널리 공유하라


- 코드리뷰를 늦게 하는것은 안하는 것보다 나쁠수 있다. Iteration 에 포함시켜라


- 한꺼번에 덜하기보다는 조금씩 자주하라


- 툴에 얽메이지 마라, 중요한것은 개발자들이 서로 대화를 하고 코드를 공유하는 것이다


- 너무 많은 리뷰어를 참여시키지 마라. 2-4 명이 적당하다




* 어떻게 잘 되고 있는지 알수 있는가


- 쉽진 않다, 사실 필요없을 수도/불가능 할수도 있다, metric 에 집착할 필요는 없다.


- 장기간의 이득은 측정할 수 없지만 많다


- Simple Metric 들이면 충분할 수 있다. (리뷰에 사용된 시간, 리뷰 comment 등).