Download as PDF, PPTX
Uploaded byTakao Oyobe
20141105 俺のコードレビュー(lightning talk) #devraku
This document discusses code review patterns and improving code review practices. It describes several code review problems or constraints teams commonly face, such as unclear review contexts, inefficient review coordination, and lack of review metrics. For each problem, it proposes a potential pattern or solution, such as defining review contexts upfront, conducting team reviews to improve quality, and tracking review counts and time spent. The document advocates adapting and combining these patterns to fit each team's unique needs rather than copying practices from other teams. It also encourages challenging existing practices and aiming higher with code reviews.
20141105 俺のコードレビュー(lightning talk) #devraku
- 1. C O DE R E V I E W 俺のコードレビュー @TAKAKING22 TAKAO OYOBE 及部敬雄 2014.11.5 俺のコードレビュー勉強会
- 2.
- 3.
- 4.
- 5.
- 6. 品質の担保 認識合わせ 教育 - CI、静的解析、テスト … - コード規約、ペアプロ … - ペアプロ、家でやれy …
- 7. 品質の担保 認識合わせ 教育 - CI、静的解析、テスト … コードレビューじゃなくても できる方法はある - コード規約、ペアプロ … - ペアプロ、家でやれy …
- 8. まさか… 品質の担保 認識合わせ 教育 偉い人が見るため 偉い人が知るため 偉い人が新人を…(ry
- 9. あるいは… 品質の担保 認識合わせ 教育 した気になってる した気になってる した気になってる
- 10.
- 11.
- 12.
- 13. なぜなら… よそはよそ、うちはうち 成功した事例には、 コンテキストや制約が存在し、 それがあなたのチームに 当てはまるとは限らないから
- 14. うわ… うちのコードレビュー 効果なさすぎ…？ こうならないために どうすればいよいのか？ http://www.pakutaso.com/20120806242post-1877.html
- 15.
- 16.
- 17.
- 18.
- 19. まとめると、 •事例だけでは役に立たない •事例からパターンに落とす•パターンを組み合わせて適用する •組み合わせることで効果が出る http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
- 20. つまり、 •自分のチームの問題・課題を見える化 •事例からパターンに変換する•パターンを取捨選択する •パターン組み合わせて実施する 点ではなく、 線・面で問題解決する http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
- 21.
- 22. 実はぶっちゃけ… でも教科書通りじゃ うまくいかねーんだよ いろいろ考えてやろうぜ ってことを代弁してくれてて ちょっと好きになった
- 23.
- 24. Pull Request コンテキスト問題 レビューのログを残したい 人によって残し方が バラバラでレビューがつらい 制約解決 Gitの使い方を覚える必要ありPull Requestを用いる
- 25. レビューコンテキスト コンテキスト問題 レビューの目的と観点を 明確にしたい レビューの目的が不明確で 必要な準備が不足している 観点がつかみにくい 制約解決 時間がかかるレビューの最初に レビューコンテキストを 説明してからレビューを開始
- 26. チームレビュー コンテキスト問題 レビューの質をあげたい レビュワーを増やしたい つまらない指摘が多くて レビューに 改善が盛り込めない 制約解決 時間がかかるチームレビューを導入し、 全員がレビュワーになる （レビュワーの観点を知る）
- 27. レビューカウント コンテキスト問題 レビュー施策の効果を 計測したい 改善策はやっているが 客観的な効果計測が できていない 制約解決 レビューカウントが多いから といって質が悪いとは 一概には言えない 1つのチケットにかかった レビュー回数をハンコで 計測して見える化
- 28. レビューの見える化 コンテキスト問題 一日のレビュー状況と偏りを 見える化したい 自己中心的なレビュー予約が 増えてレビュワーの作業時間 が逼迫する 制約解決 変更の際は変更コストが発生かんばん＆付箋でレビューを 見える化して朝礼で確認
- 29. レビュー時間の固定化 コンテキスト問題 レビュー調整のコストを 下げたい レビュワーが忙しく レビュー時間の調整に時間が かかってしまう 制約解決 レビュワーの作業時間が 不確定 レビュータイムを朝夕に設けて 必要な人がその枠を予約する
- 30. 今紹介したのは、 自分の現場で実際にやっている コードレビューパターンの一例です。 今日たくさんパターンの原石が 見つかると思うので、 それを組み合わせて自分のチームの 改善に活かしてください。
- 31. C O DE R E V I E W 俺のコードレビュー
- 32.
- 33.
- 34.
- 35.
- 36.
- 37.
- 38.
- 39.