리뷰의 기술

건강한 소프트웨어를 위한 설계 리뷰 바로잡기

현실 문제 : 리뷰에 들인 노력만큼의 효과를 거둔 현장이 극히 드물다.

목표 : 효율적인 리뷰를 실천해보기

[문제]

1. 생각나는 대로 말하기

- 중요한 문제에 대한 지적이 준다.

2. 건수 채우기

- 건수 할당량만 채우면 불성실해지기 쉽다.

3. 문서 작성자 헐뜯기

- 비난으로 확대되면 도움이 되지 않는다.

[리뷰의 목적]

수정 공수 줄이기 : 문제를 미리 발견하여 비용을 절감하는 것이 리뷰를 시행하는 가장 큰 목적이다.

[리뷰 결과]

리뷰 결과는 리뷰어의 마인드에 따라 크게 좌우된다.

- 문서 작성 시에 실수는 으레 따라다니게 마련이다. 좋은 시스템을 만들기 위해 서로 협력해서 조기에 중요한 문제를 검출하자. 같은 긍정적인 마인드를 의식적으로 유지할 필요가 있다.

[리뷰 핵심]

문제 검출, 문제 지적

[시간을 낭비하고 중요한 문제를 놓치는 네 가지 안티패턴]

1. 인간관계 끌어들이기

- 이 사람은 잘하니까 제대로 했겠지, 이 사람은 문제일거야 같은 마인드를 버리자.

- 위의 마인드 대신 이 문제를 지적하면 수정 공수가 줄어드는가? 수정 공수를 줄이기 위해 지적해야 할 문제를 모두 찾아냈는가? 같은 자문을 계속한다.

2. 작성자 마인드

- 내가 새로 하는게 낫겠다 라는 마인드로 새로 만들어주면 정작 문제 검출에 대한 내용은 소홀해진다.

3. 두 마리 토끼 잡기

- 한꺼번에 여러가지 관점을 보려고 하면 끝내지 못하거나 중요한 문제를 놓치기 쉽다.

- 리뷰 관점을 설정하여 한번에 하나씩 체크한다. (ex : 에러 처리의 정의, 기능 간의 의존관계, 자원 누수, 화면 표준과의 정합성, 다른 시스템과의 인터페이스 등)

4. 시간부족

- 초반에 집중하다가 시간 부족으로 마감시간즈음에는 마지막까지 훑어보지 못한 채 끝날 수 있다.

- 문제 검출 작업을 여러개로 분할하고, 각 검출 작업에 대한 대략적인 시간을 예상해서 계획을 세우는 것이 좋다.

[리뷰의 4단계]

1. 리뷰 준비(1시간 정도로 잡는다)

- 시나리오 순서 정하기

- 시나리오 확인과 참고 문서 준비

- 검출 방법과 체크할 범위 검토

- 문제 검출 타이밍 결정

2. 문제 검출

- 시나리오에 따라 검출하기

- 검출한 문제 메모하기

- 시나리오 다시 보기

- 문제기록표 작성(문제 유형-시나리오, 문서상의 위치, 문제 내용)

3. 문제 지적

4. 수정 및 확인

[문제의 재발방지]

1. 문제의 재확인과 분류

- 문제 카테고리

- 간단한 리뷰에서 검출한 문제

2. 문제의 발생 원인 분석

- 구체적인 원인을 찾는 것에 중점을 둔다.

3. 재발 방지책 검토

- 문제 발생 원인 분석에서 거론된 각각의 원인에 대해 재발 방지책을 검토한다.

- 구체적으로 실행할 수 있는 재발 방지책을 생각한다.