Document/ReviewCheckList

http://sakura.qp.land.to/?Document%2FReviewCheckList


Top > Document > ReviewCheckList

レビューチェックリスト

コーディング等のレビューの際のお供にご利用ください。 みなさんのレビュー観点があれば追加してください。

共通

  • 誤字・脱字・わかりにくい表現・誤解を招く表現はないか
  • 要求事項がすべて網羅されているか

データ定義

  • データ型、桁数は適切か
  • データのキー項目は妥当か
  • グローバルデータ・ローカルデータの区分は適切か
  • グローバルデータ・ローカルデータの命名規則を区別しているか
  • 内部データに無駄な項目はないか
  • 内部データに不足している項目はないか

モジュール化

  • 似たような機能がないか
  • 異なる処理をひとつのモジュールに混在させていないか
  • パラメータ設計は適切か
  • ローカル変数の定義は適切か
  • 処理が複雑すぎないか
  • 関数名と処理内容が一致しているか
  • ルーチン内でグローバル変数の使用を抑えているか

汎用モジュール

  • 汎用モジュールの使用方法は適切か
  • 汎用モジュールと同じものを作っていないか
  • 汎用性を阻害する処理が存在しないか
  • 性能を考慮しているか
  • パラメータ設計は適切か
  • 保守が容易なロジックになっているか
  • 例外処理は考慮されているか
  • コメントは十分か

データアクセス

  • 同じデータを二重に読み込むことはないか
  • 検索条件は適切か
  • 同じデータに対し異なったアクセスをしていないか
  • 標準的なデータへのアクセスが汎用化されているか
  • 無駄なデータを読み込んでいないか
  • 読み込んだデータを格納するバッファは十分か
  • 排他処理は適切か
  • エラー処理を行っているか
  • 性能を考慮しているか

アルゴリズム

  • 性能を考慮しているか
  • 保守が容易な簡単なロジックになっているか

ユーザインタフェース

  • ユーザビリティを考慮しているか
  • 無効なデータに対する処理がされているか
  • エラーメッセージはわかりやすいか

その他

  • ToDo事項が明確になっているか
  • 将来の拡張性を考慮しているか
  • セキュリティに対する考慮がされているか

リロード   新規 編集 凍結 差分 添付 複製 名前変更   ホーム 一覧 単語検索 最終更新 バックアップ リンク元   ヘルプ   最終更新のRSS
Last-modified: 2008-03-09 (日) 18:29:39 (3846d)