コードレビュー

コードレビュー（ピアレビューとも呼ばれる）は、ソフトウェアの品質保証活動であり、実装後または開発プロセス中に、1人または複数の人がコンピュータプログラムのソースコードを検査します。チェックを行う人（作成者を除く）は「レビュー担当者」と呼ばれます。少なくとも1人のレビュー担当者は、コードの作成者であってはなりません。^{[ 1 ] [ 2 ]}

コードレビューは、静的コード分析、自己チェック、テスト、ペアプログラミングといった関連するソフトウェア品質保証手法とは異なります。静的分析は主に自動化ツールに依存し、自己チェックは作成者のみが関与し、テストはコードの実行を必要とし、ペアプログラミングは開発中に別個のステップとしてではなく継続的に実行されます。^{[ 1 ]}

品質問題の直接的な発見が主な目的となることが多いが、^{[ 3 ]}コードレビューは通常、複数の目的を達成するために実施される。^{[ 4 ] [ 5 ]}

• コード品質の向上–可読性、統一性、理解しやすさを向上させることで、内部コードの品質と保守性を向上させる。 
• 欠陥の検出–外部的な側面、特に正確性に関する品質を向上させるだけでなく、パフォーマンスの問題、セキュリティの脆弱性、注入されたマルウェアなどの問題も発見します。 
• 学習／知識移転–コードベースの知識、解決策、品質に関する期待値を、レビュー担当者と作成者の両方に共有する。 
• 相互責任感を高める–コードの共同所有意識と連帯感を高める 
• より良い解決策を見つける–手元の特定のコードにとらわれず、新しいより良い解決策やアイデアを生み出す 
• 品質保証ガイドライン、ISO/IEC規格への準拠–航空交通管制ソフトウェアや安全性が重要なソフトウェアなど、一部の分野ではコードレビューが必須です。 

コードレビュープロセスにはいくつかのバリエーションがあり、IEEE 1028では追加のタイプが規定されています。^{[ 6 ]}

• 経営陣のレビュー
• 技術レビュー
• 検査
• ウォークスルー
• 監査

Capers Jones による 12,000 を超えるソフトウェア開発プロジェクトを分析する継続的な研究では、正式な検査では潜在的な欠陥の発見率が 60 ～ 65% であるのに対し、非公式な検査では欠陥の 50% 未満しか検出されないことがわかりました。ほとんどのテスト形式の潜在的な欠陥の発見率は約 30% です。^{[ 13 ] [ 14 ]} 『 Best Kept Secrets of Peer Code Review』という書籍に掲載されたコードレビューのケーススタディはCapers Jones の研究と矛盾しており、^{[ 13 ]}軽量レビューは正式なレビューと同じくらい多くのバグを発見でき、しかもより速く、より低コストであることがわかりました。^{[ 15 ]}

研究によると、コードレビューのコメントの最大 75% は機能ではなくソフトウェアの進化可能性と保守性に影響を与えることが示されており、^{[ 16 ] [ 17 ] [ 4 ] [ 18 ]}コードレビューは製品またはシステムのライフサイクルが長いソフトウェア企業にとって優れたツールであることを示唆しています。^{[ 19 ]}したがって、コードレビューで議論される問題の 15% 未満がバグに直接関係しています。^{[ 20 ]}

• コミッター
• ソフトウェアレビュー
• ソフトウェア品質
• コーディングのベストプラクティス
• ソフトウェア開発の理念一覧

• コードレビューにおける5つのアンチパターン
• Java Magazine、2020年ベスト