ウチでは昔から新規、修正された全ソースに対してソースレビューを実施している。これ自体は特に珍しいことではなく、やってるとこはやってることだと思う。ソースレビューと書いたが、仕様レビュー、結合テストレビューを含める。 レビューすべき内容 レビューでチェックするのは、ざっと、以下のような点かなと思う。 (1)要件に合致しているか 仕様レビューで指摘するべき (2)漏れがなく、重複が無いか 結合テストレビューで指摘するべき (3)バグ等が無いか 様々なレイヤでの原因が考えられるため、仕様、結合テスト、ソースのすべてのレビューで指摘が発生する。 (4)デグレードが無いか これも上記と同様 (5)運用上の懸念点が無いが これも上記と同様 (6)パフォーマンス上の懸念は無いか 仕様、ソースレビューで指摘するべき (7)コーディング規約上の違反事項は無いか ソースレビューで指摘するべき （指摘優先順位順