코드 검토 목록
3807 단어 codereview
나는 내 블로그에 우리가 코드 심사를 어떻게 하는지에 관한 글을 발표하고 이를 배합 프로그래밍과 비교했다.
내가 주의한 것은 모든 팀과 팀원들이 코드 심사를 할 때 서로 다른 명세서를 가지고 있다는 것이다.이 문장에서 나는 내가 원하는 것을 열거할 것이다.이것은 절대로 완전한 명세서가 아니다.검사가 부족하다고 생각되면 댓글에 추가해서 알려주십시오.
설계
사용자가 사용하는 프로그래밍 언어나 플랫폼에 따라 새 코드가 정확한 설계를 따르는지 확인하십시오.
무엇이 정확한 설계인지는 팀의 동의 여부에 달려 있습니까?새로운 추상을 도입하고 새로운 디자인을 따를 때 개발자들 사이에서 미리 논의하는 것이 좋다.코드 심사는 토론한 대로 실현될 수 있도록 일종의 방법으로 확보할 수 있다.이것은 많은 재작업이 없을 것을 확보할 것이다.그러나 새로운 것을 실시하고 피드백을 받고 싶을 때 면제된다.생각을 전달하기 위해 충분히 많이 쓰도록 하세요.
다음은 체크해야 할 사항입니다. (이 목록에서 대부분의 내용을 제공해 주셔서 감사합니다.)
테스트
expect(subject).to include(:customer) #is better than:
expect(subject.key? :customer).to be\_true
QA 모자를 쓰다
This is something which most people miss or probably think not as important. Working of the code is as important as its quality. Even if you have a testing team, catching bugs at code review will reduce the cost of fixing it.
다양한 기능 요구 사항 (비기능 요구)
성능, 보안, 분석, 로깅, 경고 등 다양한 기능에 대한 요구 사항도 고려됩니다.
연속 제공
연속적인 교부를 따를 때, 새로운 변경이 생산 데이터와 기능을 파괴하지 않도록 확보해야 합니다.
따로
의견에 의견을 추가하십시오.만약 당신이 이 게시물을 좋아한다면 다른 사람들이 그것을 찾아 토론에 참여할 수 있도록 추천해 주십시오.
Reference
이 문제에 관하여(코드 검토 목록), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다 https://dev.to/uday_rayala/code-review-checklist-20c6텍스트를 자유롭게 공유하거나 복사할 수 있습니다.하지만 이 문서의 URL은 참조 URL로 남겨 두십시오.
우수한 개발자 콘텐츠 발견에 전념 (Collection and Share based on the CC Protocol.)