コードレビュー

コードレビュー（）は、ソフトウェア開発工程で見過ごされた誤りを検出・修正することを目的としてソースコードの体系的な検査（査読）を行う作業のこと。
概要.
書き下ろされたばかりのソースコードや十分なテストがされていないコードは、潜在的にバグやセキュリティホールなどの不具合や問題が入り込んでいることが多い。また、直接的な不具合はなくとも、命名規則に従っていなかったり、モジュール分割のような構造設計が不適切だったりと、可読性やメンテナンス性に問題があることもある。最適化されていないコードは、メモリやプロセッサ時間といったリソース（計算資源）を無駄に消費して性能低下を招くような問題を抱えていることもある。ソフトウェア品質を高めるためにはこのような不具合や問題を除去していく必要がある。潜んでいる不具合や問題の数は一般的に、ソースコードを記述した開発者（プログラマ）の設計スキルや実装スキルにも左右されるが、これらを発見し修正するための方法のひとつが、本人または他者によりソースコードの査読を行うこと、すなわちコードレビューである。
オンラインのソフトウェアリポジトリ（匿名のCVSなど）を使うと、複数の個人が共同でコードレビューを行うことができる。バージョン管理システム（VCS）およびホスティングサービスを利用したソフトウェア開発では、コード修正を含む分岐先ブランチのプルリクエスト（pull request）を提出する際にレビューを受けて、レビューを通過したコードのみが分岐元ブランチにマージされるようにすることで、修正により別の不具合が混入するリスクを低減させることができる。
コードレビューを自動化するソフトウェアを使うと、ソフトウェア開発者の代わりに典型的なセキュリティホールを見つける作業を行ってくれる。そのようなソフトウェアの例として、Flawfinder や Rough Auditing Tool for Security (RATS) などがある。GitHubと連携するSiderのような自動化サービスもある。
効果.
コードレビューを実施することにより以下のような効果が期待できる。
ただし、その性質から開発工程上の問題点も多く、批判もある（#批判の項目を参照）。
批判.
コードレビューよりもコーディングにあたっての規則や方法論を整備することのほうが重要であるとの見方もある。エクストリーム・プログラミング (XP) という技法では、ペアプログラミングというプラクティスを推奨しており、コーディングの最中に同時にコードレビューを行うようになっている。XP の信奉者は、リファクタリングやコードの前にテストを書くといったXPのプラクティスによってレビュー/書き直しが不要なコードを作成することでソフトウェア開発がスピードアップすると主張する。
DOD-STD-2167A では、コードレビューは「労多く益少なし」としている。Lausen と Younessi（IEEE Software, July/Aug 1998, pg 69-73）では、行単位のコードレビューはほとんど価値がないと結論付けている。問題点を除去するという意味では、プログラマに行単位のコードレビューをさせることは、他の手法よりも生産性が低い。
コードレビューにより一定時間拘束されるため、担当作業の遅延が発生する可能性があるとの批判もある。
レビューの精度や粒度は属人的なスキルにも依存する。また、レビューの指摘項目や内容を組織内で共有できなければ、同じ指摘が相手を変えて何度も繰り返されるおそれがある。事前にレビューの目的や到達点を明確にして、開発メンバーの間で共有することも必要である。
コンパイラからの警告や静的コード解析の結果は、潜在的な問題点を指摘していることも多い。警告のレベルを上げ、これらを無視せずにコードを正しく修正していけば、属人的で時間のかかるコードレビューに頼らずとも、問題点のいくつかは解消できることがある。