Google code review guide

 

 

UI변경이 합리적이고 보기 좋은지

병렬작업이 안전하게 실행되는지 

 

 

• 설계(Design): 코드가 잘 설계되었고 시스템에 적합한가?
• 기능(Functionality): 코드가 작성자의 의도대로 동작하는가? 사용자에게 적합하게 동작하는가?   지금 당장 필요없는 기능을 개발하진 않았는가?
• 복잡성(Complexity): 더 간단하게 만들 수 있는가? 나중에 코드를 다른 개발자가 보았을 때 쉽게 이해하고 사용 가능한가?
• 테스트(Tests): 잘 설계된 자동 테스트가 있는가? 유닛 테스트는 적절항가?
• 작명(Naming): 개발자가 변수, 클래스, 메소드 등에 명확한 이름을 선택했는가?
• 주석(Comments): 주석이 명확하고 유용한가? '왜'에 대해 잘 나타내고 있는가 
• 스타일(Style): 스타일 가이드(코딩 컨벤션)를 따르고 있는가?
• 문서화(Documentation): 개발자가 관련 문서도 업데이트 했는가?