社内で発表したときに作った資料。

1. 俺のコードがこんなに
汚いわけがない
2013/01/31(水)

2. コードレビューの目的
・品質の向上
・開発スキルの底上げ

3. レビュー観点
1. 機能性
2. 保守性
3. 効率性

4. 機能性
要件を満たしていることを確認する
・入力チェックしているか
・NULLやundefのチェックしているか
・車輪の再開発してないか
・関数のボリュームは適切か
・例外発生時の挙動を考慮しているか
・インデントや括弧の位置などに統一感があるか

5. 保守性
他の人が理解しやすいか、
仕様変更しやすいかを確認する
・関数名や変数名が適切か
・コメントがちゃんとあるか
・マニアックなオレオレプログラムになってないか
・重複している処理がないか
・鬼みたいな量のif文はないか
・雑なコピペで強引なドリブルをしていないか

6. 効率性
性能に影響がありそうな部分を確認する
・ループの中でオブジェクトの動的生成
・ループの中でAPI発行
・ループの中でDB検索
・ループの中でファイルのI/O
・巨大な配列やオブジェクトを実体コピー

7. レビューの進め方
どういうプログラムなのかを説明
コードを読んで質疑応答など

8. コードレビューの注意点
・構文エラーがあるコードは見ない。
⇒商用で動かないものを見るのは非効率。
・あらさがしをして、人をボコボコしない
⇒糞コードを憎んで人を憎まず！
・レビューは１日１時間
⇒それ以上だと集中力が持たない・・・