코드리뷰란?

코드리뷰

• 커밋을 하기 전에 하는 동작
• 시퀀스

1. 코드의 변경점, 디자인 문서를 미리 팀원들에게 공유 <- 일주일 전쯤 공유 <- 팀원들이 코드를 미리 보고 올것이라 예상하고 진행
2. 대부분 아무도 안 보고 옴 <- 각자 자기의 일이 있기에 바빠서 못 봄 <- 관리자분들은 보고옴
3. 오프라인 코드 리뷰를 시작!
4. 코드랑 디자인을 다 설명해야 됨. -> 디자인 설명 후 이해 확인 후 코드 변경점을 라인 바이 라인으로 설명함

Ex) 함수명, 유닛 테스트, 매개변수가 적절한가, 아키텍처에 맞게 함수가 짜져있는가, 기존 코드에 비해 성능적인 문제가 없는가, 특정 언어와 관련된 리뷰(inline 함수를 정확하게 적었는가, 컴파일러에 맞게 작성되었는가), 알고리즘은 최적의 알고리즘인가

1. 코드 리뷰 내용을 적어주는 사람이 따로 있음 -> 피드백을 받은 부분에 대해 아이템 정리해줌
2. 바로 넣을 수 없는 코드의 변경점이 존재 -> 수정 후 다시 1번부터 시작

 

지라에서 코드 리뷰에 관한 히스토리 정리하기

디자인 문서에도 피드백 내용 반영하기

 

코드 리뷰 끝~~

 

코드 리뷰를 진행하지 않는 시점 : 처음 시작하거나, 기능 구현 하느라 바쁠 때

코드 리뷰가 시간을 너무 잡아먹어 초반에는 개발의 자유성을 넓힌다. -> 후에 시간이 넉넉할 때 코드 리뷰를 진행(제품이 나간 후에는 코드 리뷰를 아주 꼼꼼하게 한다)

 

코드 리뷰 : 코드 콜라보레이터

 

검증을 자주 할 수록 불량이 많이 감소함

 

개발자 스스로 할 수 있는 테스트 : 유닛 테스트, 자주 검증, 코드 리뷰, TDD 등…