レビューでいいのか検討
レビューのルール
- 事前
- レビューの観点
- 計画
- 日程調整
- レビュー方針の決定
- 終了条件の決定
- レビュー中
- 欠陥の指摘
- 欠陥の記録
- 欠陥を修正すべきか討論
- 修正(すべき)事項の決定
- 事後
チェックリスト
- 対象は？
- ドキュメント
- 記述漏れ
- 記述ミス
- 記述過多
- 上位ドキュメントとの整合性
- ソースコード
- 可読性
- タブとスペースの混在
- 適切な括弧付け
- 適切なコメント
- 保守性
- 同じような処理を行っている関数、メソッド
- 変数名、クラス名、関数名に規則性
- 極端に短い変数名、クラス名、関数名
- 例外処理の検討漏れ
- 初期化漏れ
- 車輪の再発明
- テストコードの欠如
- 多すぎる引数
- 適切なスコープ
- グローバル変数の多用
- 記述ミス
- 設計ミス
司会の手引きについて
- やること
- 事前
- レビューのスケジューリング
- 開始基準のチェック
- レビュー範囲の決定
- レビューの観点の設定
- キックオフ
- レビュー目的の確認
- レビュー対象の配布
- 全員の認識あわせ
- レビュー中
- 事後
- 心得