ソースレビューの結果は文章に

品質向上と言うことで、レビュー強化が進んでいます。

リリースまでの各プロセスで、レビューが入りますが、
ソースレビューについて、チーム内で取り組んでいたこと


■実装場所確認しましょう
MVCに準拠した、実装場所に記載します

■テーブル・カラムの追加または、変更が、ありましたか?
テーブル・カラムの順は、整理されていますか?
pk、indexは、考慮されていますか?
適用するSQL文(ALTER)は、開発環境で問題なく適用できましたか?
適用するSQL文(ALTER)は、開発環境で実行時間は、どのくらいでしたか?
コードを追記する際、テーブル・カラムの順番に整理して、mdlを記載できていますか?
DBドキュメントは、更新しましたか?

■SQL文のリクエスト時間(EXPLAIN)を確認しましたか?

■共通部品化を考慮しましたか?
共通部品化を考慮しましょう
共通関数として、libに実装するべき関数を、prg側に都度記載していませんか?

■セキュリティーに関係する情報が、露出されていませんか?
IDなどが、パラメータに直接露出している場合は、
暗号化(cryptなど)を使用しましょう

■類似施策の拡張性を考慮していますか?
if分岐などは、switch分岐に変更する

■不要コメント削除されていますか?


上記に、今後追加をしていきます。

また、取り組んではいましたが、意識していなかったことがありました。
文章で残すことが大事。と、記載しているサイトがありました。
http://d.hatena.ne.jp/kameid/20091028/1256721583

コメント