レビューする目的を明確にする
- 開発のどの段階の、どういう文脈でのレビューなのか
- 開発初期とリリース前では、どこにどの程度着目すべきかは違ってくる
- プロジェクトにとって何が重要なのか
- コストと効果を意識する
- 分類
- リファクタリング
- パターンやライブラリ
- バグの指摘
- ロジックの整合性、仕様通りかどうか
- パフォーマンス
- 互換性、拡張性、メンテナンス性
- 一貫性
- 読みやすさ、言語やSDKのルール
人間にしか出来ないことに集中する
- 機械にまかせられるようなチェックは機械にまかせる
- インデントの間違い
- スペースや改行の挿入位置
- 静的解析
- コーディング規約：無駄に悩むことを減らす
内容によっては他の手法が適している場合もある
- ペアプロ
レビューは改善のために行うもの
- 品質を向上させる
- 共通の認識を共有する
- メンバーのスキル向上
- 人に読まれることを意識するようになる