나는 나 자신을 슈퍼 스타 개발자라고 부르지는 않지만 상대적으로 숙련 된 전문가라고 부를 것이다. 코드 품질을 높은 수준으로 유지하려고 노력하고 항상 코딩 스타일을 개선하고, 코드를 효율적이고 읽기 쉽고 일관성있게 유지하면서 팀이 패턴 및 방법론을 따라 일관성을 유지하도록 장려합니다. 또한 품질과 속도의 균형이 필요하다는 것도 알고 있습니다.
이를 달성하기 위해 동료 검토라는 개념을 팀에 소개했습니다. 병합을위한 github pull-request에서 두 개의 엄지 손가락. 위대하지만 내 의견으로는 좋지 않습니다.
나는 종종 같은 동료의 동료 리뷰 의견을 보았습니다-
- 뒤에 공백을 추가하는 것이 좋을 것입니다
<INSERT SOMETHING HERE>
- 메소드 사이에 불필요한 추가 라인
- docblock의 주석 끝에는 완전 정지를 사용해야합니다.
내 관점에서 볼 때, 검토자는 코드 미학을 피상적으로보고 있으며 실제로 코드 검토를 수행하지는 않습니다. 화장품 코드 검토는 오만한 / 정신 주의적 사고 방식으로 나에게 온다. 그것은 실체가 없지만 검토자가 기술적으로 정확 하기 때문에 너무 많이 논쟁 할 수는 없습니다 . 위와 같은 종류의 리뷰는 많지 않지만 다음과 같은 리뷰는 더 많이 볼 수 있습니다.
- 순환 복잡성을 줄일 수 있습니다 ...
- 일찍 종료하고 if / else를 피하십시오
- DB 쿼리를 리포지토리에 추상화
- 이 논리는 실제로 여기에 속하지 않습니다
- 자신을 반복하지 마십시오-추상과 재사용
X
메소드의 인수로 전달 되면 어떻게Y
됩니까?- 이것에 대한 단위 테스트는 어디에 있습니까?
나는 화장품 유형의 리뷰를 제공하는 사람들이 항상 같은 종류의 사람들이고, 제 생각에 "Quality & Logic based"동료 리뷰를 제공하는 사람들과 같은 유형의 사람들이라는 것을 알았습니다.
동료 검토에 대한 올바른 접근 방법은 무엇입니까? 그리고 실제 코드 결함이 아닌 철자 오류 및 미적 결함을 찾는 코드를 기본적으로 훑어 보는 동일한 사람들에 좌절감을 느끼는 것이 맞습니까?
내가 맞다면-동료들에게 실제로 화장 용 터치 업 제안과 균형을 이루어 코드에서 결함을 찾도록 격려하는 방법은 무엇입니까?
내가 틀렸다면-깨달아주세요. 실제로 좋은 코드 검토를 구성하는 규칙에 대한 규칙이 있습니까? 코드 리뷰가 무엇인지 요점을 놓쳤습니까?
내 관점에서-코드 검토는 코드에 대한 책임을 공유하는 것입니다. 논리, 가독성 및 기능을 다루거나 확인하지 않고 코드에 엄지 손가락을 올리는 것은 편안하지 않습니다. 누군가가 doc-block에서 완전히 멈추지 않았다는 것을 알면 견고한 코드 조각에 대한 병합을 차단하는 것을 귀찮게하지 않을 것입니다.
코드를 검토 할 때 500 Loc 당 15-45 분이 소요될 수 있습니다. 이 얕은 리뷰가 그들이 수행하는 리뷰의 깊이라면 10 분 이상 걸리는 것을 상상할 수 없습니다. 또한 얕은 검토 자의 엄지 손가락 업 가치는 얼마입니까? 확실히 이것은 모든 엄지 손가락의 무게가 같지 않으며 2 패스 검토 프로세스가 필요할 수 있음을 의미합니다. 심도있는 검토를위한 하나의 엄지 손가락과 "연마"를위한 두 번째 엄지 손가락?