レビューチェックリスト
コーディング等のレビューの際のお供にご利用ください。
みなさんのレビュー観点があれば追加してください。
共通
- 誤字・脱字・わかりにくい表現・誤解を招く表現はないか
- 要求事項がすべて網羅されているか
データ定義
- データ型、桁数は適切か
- データのキー項目は妥当か
- グローバルデータ・ローカルデータの区分は適切か
- グローバルデータ・ローカルデータの命名規則を区別しているか
- 内部データに無駄な項目はないか
- 内部データに不足している項目はないか
モジュール化
- 似たような機能がないか
- 異なる処理をひとつのモジュールに混在させていないか
- パラメータ設計は適切か
- ローカル変数の定義は適切か
- 処理が複雑すぎないか
- 関数名と処理内容が一致しているか
- ルーチン内でグローバル変数の使用を抑えているか
汎用モジュール
- 汎用モジュールの使用方法は適切か
- 汎用モジュールと同じものを作っていないか
- 汎用性を阻害する処理が存在しないか
- 性能を考慮しているか
- パラメータ設計は適切か
- 保守が容易なロジックになっているか
- 例外処理は考慮されているか
- コメントは十分か
データアクセス
- 同じデータを二重に読み込むことはないか
- 検索条件は適切か
- 同じデータに対し異なったアクセスをしていないか
- 標準的なデータへのアクセスが汎用化されているか
- 無駄なデータを読み込んでいないか
- 読み込んだデータを格納するバッファは十分か
- 排他処理は適切か
- エラー処理を行っているか
- 性能を考慮しているか
アルゴリズム
- 性能を考慮しているか
- 保守が容易な簡単なロジックになっているか
ユーザインタフェース
- ユーザビリティを考慮しているか
- 無効なデータに対する処理がされているか
- エラーメッセージはわかりやすいか
その他
- ToDo事項が明確になっているか
- 将来の拡張性を考慮しているか
- セキュリティに対する考慮がされているか