品質向上と言うことで、レビュー強化が進んでいます。
リリースまでの各プロセスで、レビューが入りますが、
ソースレビューについて、チーム内で取り組んでいたこと
■実装場所確認しましょう
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
リリースまでの各プロセスで、レビューが入りますが、
ソースレビューについて、チーム内で取り組んでいたこと
■実装場所確認しましょう
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
コメント
コメントを投稿