회사,it's 토론을 이메일의 어떤 기능을 구현하고 어떤 종류의 버그가 수정되이 보낸 하나가 쓰는 코드입니다. 고 평가 받은 메일을 검토 코드고 논의 품질 및 편집하는 방법에 코드를 그의 의견입니다. 무엇 표준 검토 코드가 포함되어 있습니까?
내 경험에서 대부분의 공식적인 코드 리뷰 양도로 스타일을 검사하기 때문에 그것이's 쉽습니다. 경우에도 당신은 공급 체크리스트의 일을 보고,그's 을 위해 아주 쉽게 눈을 시작 윤이 나는 끝났습니다.
나는've 는 것을 발견 유닛 테스트 검토를 제공합 더 많은 혜택입니다. 대부분의 개발자 나는've 와 함께 일했지't 있는 방법을 알고 단위 테스트는 제대로 일단 그들이 그"Aha!"순간의 나머지 자신의 코드를 시작한 개선뿐만 아니라. 여기's 힌트:'s 지 단위 테스트해야 하는 경우에는 사용자를 검사하는 무언가를,그리고 그's 지 단위 테스트하는 경우 당신은 무언가를 시작하에서 실행하는 디버거.
하는 경향이 있에 따라 달라질 수 있습니다. 많은 시간은 그's 간단한 고무-스탬프입니다. "기's 무엇이 문제,에서 보이는 라인이 여기,그's 분명한 것's 잘못된 것,그리고 여기's where I 고정머니를 찾을 수 있습니다.quot;"Yup,는's 분명합니다. 가서 그것은을 확인합니다."
그러나 때가 더 많은 관여하는가에,그것은 일반적으로 다음과 같이 진행됩니다
-터치패드가 활성화되어 있는지 확인에 대한 수정 사항에 TortoiseSVN 의 목록을 얻으려면 파일을 변경 합니다. -가으로 평가할 수 있습니다. -문제를 설명과 함께,크롬에서 버그 추적 시스템을 엽니다. -아래로 이동 파일의 목록에서 TortoiseSVN 열고,그들의 각각에서BeyondCompare서 변경 내용을 표시합니다. -만약 검토자는't 의 변화를 이해하고,설명하고 당신이 무엇을 했는지 그리고 왜. -로 검토할 수 있는지't 좋아 보인다. 그렇다면,그것을 논의에 도달할 때까지 계약에는지 여부를 변경해야합니다. (있다면 간단하여 변경이 필요할 수 있습도 파일을 편집하 내부의 BeyondCompare.) -이 있는 경우에 어떤 변화를 다시 컴파일하고 있는지 확인을 구축! -프로그램을 실행하는 증명을 평가하는 해결 실제로 작동합니다. -을 확인합니다.
내가 선호하는 코드를 검토하는 데브 앉아 있으로 검토를 통해 코드 라인 라인에 의해 설명이다. 자주 사용이 보는 문제에 대한 설명을 검토하지 않을 수 있습이 아직 보이는 이유합니다. 나는 또한 코드가 없는 곳에 나도 전송 코드를 읽고 내 자신에 그것들의 의견,그러나 내가 찾는 사람들을하는 경향이 더 이상(검토와 초안의 의견하고 보내는 개발자들을 읽고요 그녀는 것을 의미하고 이메일을 나에게 다시 설명하고 두 개 또는 세 개의 라운드 나중에 우리는 함께 나 화면에 내가 무엇을 의미하고,간,"오래 지금은 그것을 참조하십시오.")고 생산성을 덜 수 있으므로 적은 진짜 토론하고 더 많은,"당신이 잘못입니다."
그것은 또한 중요한 표준을 강제 적용에 코드를 검토하지만 그들만의 초점입니다.
그러나 코드를 전송되지 않을 생산할 때까지 코드를 검토자는 행복하거나 관리하지 않는(dev)은 기각 그 또는 그녀의(코드는 검토가 너무 잘못된). 이것은 중요하거나 코드를 검토하는 것은 단지 관료 프로세스를 추가하지 않 값이면 코드의 검토를 승인해야 하는 최종기 전에 코드를 푸시됩니다.
첫째로 당신은 코딩 할 필요가 표준과 이들 보다 더 단순한 구문입니다. 사람들이 시작할 때에 회사는 그들이 배워야 하는 지침을 당신의 회사는 가능한 한 많기 전에 그들은 코드를 시작. 는 경우에는 검토 프로세스는 모든 종류의 위반을 찾은 그들은 가능성이 높:
-되지 않을 수정 시간 제약으로 인해 -찾을 수상에 어떤 지침서가 가치가
의 가이드라인을 감고 있어야 적절한 장식새김을 찾 위반과 리팩터링을 최대한 쉽게 갈 수 있어야 합니다. 항상 봐의 목표는 지침 및 코드 검토
목표는 내 마음에서는 코드로 균일한 가능한 문제점을 발견으로 유지 관리와 가독성이 향상됩니다. 보조 목표 될 수 있는 많은 사람을 얻기 위해서는 최대 속도와 특정 소프트웨어입니다.
이 지침에 내 마음 수 있는 예를들면이 존재하의:
-일반적인 구문 및 코딩 가이드라인(중 하나를 선택하는 이미 존재하고 사용하여 공구를 검사하는 자동으로) -적절한 예외 처리 -적절한 로깅 좋은 사용의 패러다임의 언어에 대한(고체 위해 OO 언어) -명백하고 잘 생각 간의 종속성 요소(같은 도구를 사용 NDepend) -작업 스크립트 구축 -문서 현재(개발자의 시작,설치 매뉴얼) -내부 라이브러리를 사용하여 -회사 정책 -타인 툴지 않는't 허용 -단위는 존재 테스트 및 실패 비 코드 범위의 90% -...
는 장소에 코드를 검토으로 구성되는 소프트웨어에 대해 확인 지침:
-논의 위반으로한 프로그래머 수정 불필요한 위반이 -코멘트 데 필요한 위반이