무엇 표준 검토 코드가 포함되어 있습니까?

내 경험에서 대부분의 공식적인 코드 리뷰 양도로 스타일을 검사하기 때문에 그것이'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.) -이 있는 경우에 어떤 변화를 다시 컴파일하고 있는지 확인을 구축! -프로그램을 실행하는 증명을 평가하는 해결 실제로 작동합니다. -을 확인합니다.

IMO 코드가 없는 기능 또는 버그,하지만에 집중한 코드의 품질 및 테스트를 작성한다.

그래서,당신은 옆에 앉아 당신의 피고 있는 그를 설명하는 코드를 사용할 필요가 있음을 주목해야한 코드고 그것을 통해,어떤 상황에 대한 호출.

그것은 모두가 프로그램에 대한 동일한 표준을 사용 하는 경우 도구 fxCop 를 자동화하는 프로세스의 일부입니다.

내가 선호하는 코드를 검토하는 데브 앉아 있으로 검토를 통해 코드 라인 라인에 의해 설명이다. 자주 사용이 보는 문제에 대한 설명을 검토하지 않을 수 있습이 아직 보이는 이유합니다. 나는 또한 코드가 없는 곳에 나도 전송 코드를 읽고 내 자신에 그것들의 의견,그러나 내가 찾는 사람들을하는 경향이 더 이상(검토와 초안의 의견하고 보내는 개발자들을 읽고요 그녀는 것을 의미하고 이메일을 나에게 다시 설명하고 두 개 또는 세 개의 라운드 나중에 우리는 함께 나 화면에 내가 무엇을 의미하고,간,"오래 지금은 그것을 참조하십시오.")고 생산성을 덜 수 있으므로 적은 진짜 토론하고 더 많은,"당신이 잘못입니다."

그것은 또한 중요한 표준을 강제 적용에 코드를 검토하지만 그들만의 초점입니다.

그러나 코드를 전송되지 않을 생산할 때까지 코드를 검토자는 행복하거나 관리하지 않는(dev)은 기각 그 또는 그녀의(코드는 검토가 너무 잘못된). 이것은 중요하거나 코드를 검토하는 것은 단지 관료 프로세스를 추가하지 않 값이면 코드의 검토를 승인해야 하는 최종기 전에 코드를 푸시됩니다.

첫째로 당신은 코딩 할 필요가 표준과 이들 보다 더 단순한 구문입니다. 사람들이 시작할 때에 회사는 그들이 배워야 하는 지침을 당신의 회사는 가능한 한 많기 전에 그들은 코드를 시작. 는 경우에는 검토 프로세스는 모든 종류의 위반을 찾은 그들은 가능성이 높:

-되지 않을 수정 시간 제약으로 인해 -찾을 수상에 어떤 지침서가 가치가

의 가이드라인을 감고 있어야 적절한 장식새김을 찾 위반과 리팩터링을 최대한 쉽게 갈 수 있어야 합니다. 항상 봐의 목표는 지침 및 코드 검토

목표는 내 마음에서는 코드로 균일한 가능한 문제점을 발견으로 유지 관리와 가독성이 향상됩니다. 보조 목표 될 수 있는 많은 사람을 얻기 위해서는 최대 속도와 특정 소프트웨어입니다.

이 지침에 내 마음 수 있는 예를들면이 존재하의:

-일반적인 구문 및 코딩 가이드라인(중 하나를 선택하는 이미 존재하고 사용하여 공구를 검사하는 자동으로) -적절한 예외 처리 -적절한 로깅 좋은 사용의 패러다임의 언어에 대한(고체 위해 OO 언어) -명백하고 잘 생각 간의 종속성 요소(같은 도구를 사용 NDepend) -작업 스크립트 구축 -문서 현재(개발자의 시작,설치 매뉴얼) -내부 라이브러리를 사용하여 -회사 정책 -타인 툴지 않는't 허용 -단위는 존재 테스트 및 실패 비 코드 범위의 90% -...

는 장소에 코드를 검토으로 구성되는 소프트웨어에 대해 확인 지침:

-논의 위반으로한 프로그래머 수정 불필요한 위반이 -코멘트 데 필요한 위반이