Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.

コードレビュー用資料

2,160 views

Published on

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

Published in: Technology
  • Be the first to comment

コードレビュー用資料

  1. 1. 俺のコードがこんなに汚いわけがない 2013/01/31(水)
  2. 2. コードレビューの目的・品質の向上・開発スキルの底上げ
  3. 3. レビュー観点1. 機能性2. 保守性3. 効率性
  4. 4. 機能性要件を満たしていることを確認する・入力チェックしているか・NULLやundefのチェックしているか・車輪の再開発してないか・関数のボリュームは適切か・例外発生時の挙動を考慮しているか・インデントや括弧の位置などに統一感があるか
  5. 5. 保守性他の人が理解しやすいか、仕様変更しやすいかを確認する・関数名や変数名が適切か・コメントがちゃんとあるか・マニアックなオレオレプログラムになってないか・重複している処理がないか・鬼みたいな量のif文はないか・雑なコピペで強引なドリブルをしていないか
  6. 6. 効率性性能に影響がありそうな部分を確認する・ループの中でオブジェクトの動的生成・ループの中でAPI発行・ループの中でDB検索・ループの中でファイルのI/O・巨大な配列やオブジェクトを実体コピー
  7. 7. レビューの進め方どういうプログラムなのかを説明コードを読んで質疑応答など
  8. 8. コードレビューの注意点・構文エラーがあるコードは見ない。⇒商用で動かないものを見るのは非効率。・あらさがしをして、人をボコボコしない⇒糞コードを憎んで人を憎まず!・レビューは1日1時間⇒それ以上だと集中力が持たない・・・

×