2014/11/5に開催した「俺のコードレビュー」勉強会のライトニングトーク資料。パターンの文脈でコードレビューについて話してみました。

1. C O D E R E V I E W
俺のコードレビュー
@TAKAKING22
TAKAO OYOBE
及部敬雄
2014.11.5
俺のコードレビュー勉強会

2. コードレビューしてますか？

3. コードレビューの目的は？

4. 品質の担保
認識合わせ
教育

5. Really？

6. 品質の担保
認識合わせ
教育
- CI、静的解析、テスト …
- コード規約、ペアプロ …
- ペアプロ、家でやれy …

7. 品質の担保
認識合わせ
教育
- CI、静的解析、テスト …
コードレビューじゃなくても
できる方法はある
- コード規約、ペアプロ …
- ペアプロ、家でやれy …

8. まさか…
品質の担保
認識合わせ
教育
偉い人が見るため
偉い人が知るため
偉い人が新人を…(ry

9. あるいは…
品質の担保
認識合わせ
教育
した気になってる
した気になってる
した気になってる

10. コードレビューの儀式化

11. コードレビューは
問題解決の一手段
でしかない

12. 隣のコードレビューを
そのまま真似しても
効果はあまり出ないかも

13. なぜなら…
よそはよそ、うちはうち
成功した事例には、
コンテキストや制約が存在し、
それがあなたのチームに
当てはまるとは限らないから

14. うわ…
うちのコードレビュー
効果なさすぎ…？
こうならないために
どうすればいよいのか？
http://www.pakutaso.com/20120806242post-1877.html

15. パターンランゲージ
http://books.rakuten.co.jp/rb/12432779/

16. パターン＝再現可能な解決策
http://www.slideshare.net/kawaguti/20111207-u

17. パターンは組み合わせ可能
http://www.slideshare.net/kawaguti/20111207-u

18. 組み合わせて意味を持つ
http://www.slideshare.net/kdmsnr/fearlessjourneypostudy

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 D E R E V I E W
俺のコードレビュー

32. 今のチームにとって
必要なコードレビューを
やっている

33. ぶっ壊したい!!
今のチームにとって
必要なコードレビューを
やっている

34. こうじゃなくて、
チェック
確認学び

35. こうなるように
チェック
確認
学び

36. コードレビューで
もっとわくわくできるハズ

37. コードレビューで
もっと上を目指せるハズ

38. コードレビューは必要なのか？

39. お前のコードレビューはどうだ？