2012-01-05 4 views
2

여기 사지로 나가기. 개발자가 입력을 제공하기 위해 잘못한 것을 판단 할 수있을만큼 코드의 일부 클래스와 코드 단위를 샘플링하여 코드 검토를 수행하고 있습니까? 이것은 단지 당신이 시간을 누르고 그 (것)들이 쓴 모든 단일 행을 검토 할 수없는 경우에 해당합니다. 나는 개발자들이 같은 유형의 실수를 도처에 복제 할 수도 있다는 가정하에 작업하고있다. 이것이 효과적인 접근 방법입니까? 또는 부분적으로 효과가 있습니까? 아니면 전혀 효과가 있습니까?코드 검토 접근 방식

어떤 다른 접근 방법이 더 좋습니까? 코드 라인을 작성한 각 개발자에게 왜 그들이 그렇게 특별한 방법을 사용했는지 묻습니다. 또는 혼자서 해본 후 이메일을 통해 조사 결과의 개요를 보냅니다.

일반적인 15 일 스프린트에서 모든 코드 기반을 검토해야 할 때 주니어 개발자/동료가 작성한 코드 검토 방법에 어떻게 접근합니까?

답변

3

몇 가지 클래스와 단위를 선택하는 것이 코드가 수용 가능하다는 것을 아는 것으로 충분하지 않다고 생각합니다. 완벽하게 작성되고 큰 문제를 놓치지 않는 코드 부분을 선택하는 것이 가능합니다.

시간이 제한적이라면 검토하는 방법을 최적화해야합니다. 첫 번째로 일반적인 오류 또는 코딩 표준 편차를 자동으로 검토하기 위해 실행할 수있는 도구 (StyleCop, FxCop 등)가 있습니다.

그런 다음 중요한 기능이 무엇인지 알아 내고 줄별로 검토해야한다고 생각합니다. 시놉시스를 보내고 필요하다면 개발자들과 함께 앉아서하는 것이 현명한 접근 방법이 될 것입니다.

전체 검토가 필요한 첫 번째 코드 집합 일 것입니다. 변경 한 내용을 살펴보면 일반적으로 그렇게 나쁘지는 않지만 팀 크기에 따라 다릅니다.

1

주니어 개발자로서, 나는 앉아있는 것이 당신에게 시간이 있다면 최선이라고 생각합니다. 나 자신이 직접 해보려 던 일이 무엇인지 설명하기가 쉽지 않습니다.

작성된 모든 코드 행은 잔인합니다. 이유는 QA가 존재하기 때문입니다. 우리가 회사의 코드 규칙을 따르고 있는지, 우리가 알아볼 수있는 충분한 경험이 어리석은 바보 같은 것들을 막고 있는지 확인하려고합니다.