デキるプログラマだけが知っているコードレビュー7つの秘訣

1. SonicGarden Study 11 #sg_study いつまでクソコードを書き続けるの？デキるプログラマだけが知っているコードレビュー7つの秘訣ソニックガーデン西見公宏 1

2. SonicGarden Study 11 #sg_study 西見公宏 Nishimi Masahiro 2

3. 自己紹介 SonicGarden Study 11 #sg_study 西見公宏 Masahiro Nishimi @mah_lab 昭和58年生まれ東京育ち 2児（双子）の父親です 3

4. 今日お話する内容 1. クソコードとは何か？ 2. 優れたプログラマとは何か？ 3. コードレビューが改善の近道 4. コードレビュー7つの秘訣 SonicGarden Study 11 #sg_study 4

5. SonicGarden Study 11 #sg_study 1. クソコードとは何か？ 5

6. SonicGarden Study 11 #sg_study 「クソコード」とは・・・ 6

7. SonicGarden Study 11 #sg_study 「クソコード」とは・・・読む人を怒りの渦に叩きこむコードである 7

8. つまり •  「クソコード」とは読んだ人の怒りを体現した言葉である。 SonicGarden Study 11 #sg_study 8

9. クソコード3つの特徴 1. 読めないコード –  変数名が暗号／制御フローが無駄に複雑／メソッド名と処理の内容が合ってない etc... 2. 要領の悪いコード –  言語レベルで用意されている機能を素直に使わない（例：Go を使っているのにゴルーチンを使わない） etc... 3. 意図がわからないコード –  フレームワークのレールに従っていない etc... SonicGarden Study 11 #sg_study 9

10. クソコードかどうかは読まれるまで分からない •  シュレディンガーの猫 •  読んだ人の当事者感によって度合いが変わる（例：「え、このコード、俺がメンテするの！？」） •  自分で読んでダメだと思うなら、最初から直して・・・。 SonicGarden Study 11 #sg_study 10

11. SonicGarden Study 11 #sg_study 2. 優れたプログラマとは何か？ 11

12. 優れたプログラマ3つの特徴 1. 「読めないコード」を書かない。 –  読みやすいコードを書く。 2. 「要領の悪いコード」を書かない。 –  言語の特性を活かした要領の良いコードを書く。 3. 「意図のわからないコード」を書かない。 –  設計の意図が明確である。 SonicGarden Study 11 #sg_study 12

13. 優れたプログラマを見極める簡単な試験書いたコードの1行1行に対して、以下の項目を質問する。 l どういう処理なのか？ l どういう意図で書いたのか？ SonicGarden Study 11 #sg_study 13

14. SonicGarden Study 11 #sg_study 3. コードレビューが改善の近道 14

15. まずは読まれなければはじまらない SonicGarden Study 11 #sg_study 15 読む人に怒りをもたらすのがクソコードならば読む人がいなくてははじまらない

16. 誰に読まれるのがベストか？ SonicGarden Study 11 #sg_study 16 優れたプログラマ

17. 何故優れたプログラマが読むべきなのか？ SonicGarden Study 11 #sg_study 17 •  単純に同じレベルのプログラマが指摘しても、指摘の内容に限界があるため。限界の壁

18. 優れたプログラマは限界を引き上げます SonicGarden Study 11 #sg_study 18 •  レビューの中で小手先のテクニックに終始しない「優れたプログラマの考え方」を伝えていくことで、チームの限界を引き上げていきます。

19. •  いくら優れたプログラマと言えども、闇雲にレビューをするのでは効果がありません。ただし・・・ SonicGarden Study 11 #sg_study 19 ・・・

20. •  ソニックガーデンでも新しく入ったプログラマを教育する手段としてコードレビューを取り入れています。 •  その中で気付いたポイントをいくつかピックアップしました。 •  それが今からお伝えする「7つの秘訣」です。より効果を上げるために SonicGarden Study 11 #sg_study 20

21. SonicGarden Study 11 #sg_study 4. コードレビュー７つの秘訣 21

22. 1. レビューの観点を明確にすること SonicGarden Study 11 #sg_study 22

23. 1. レビューの観点を明確にすること •  リリース前レビューの話なのか、コード品質の話なのか。 •  読む側も漠然と読んで、漠然とコメントを書くのでは効果を生まない。 – 観点を明確にしてレビューすることで指摘する側の精度も上がります。 SonicGarden Study 11 #sg_study 23

24. 2. 我が身に返ることを恐れずに指摘すること SonicGarden Study 11 #sg_study 24

25. 2. 我が身に返ることを恐れずに指摘すること •  自分のことは棚に上げる！ •  鋭くツッコミを入れられないなら、レビューの意味は無い。 •  指摘して自分の耳が痛いなら、自分にとっても成長のチャンスだと捉える。 SonicGarden Study 11 #sg_study 25

26. 3. 何故悪いコードなのかを論理的に説明すること SonicGarden Study 11 #sg_study 26

27. 3. 何故悪いコードなのかを論理的に説明すること •  「何となく汚い」は指摘ではない。 •  悪いコードだと判断したなら、相手が納得できるように、論理的に説明する。 •  論理的に説明できないなら指摘をすべきではない。 SonicGarden Study 11 #sg_study 27

28. 4. 良いコードについて共通認識を持つこと SonicGarden Study 11 #sg_study 28

29. 4. 良いコードについて共通認識を持つこと •  コードレビューのあとに雰囲気が悪くなるのはチームで「良いコード」の認識が揃っていない証拠。 •  「良いコード」を共有することでチームとして成長する。 •  認識を揃えるためには、あるコードを題材にディスカッションの機会を設けると良い。 SonicGarden Study 11 #sg_study 29

30. SonicGarden Study 11 #sg_study 30 チームでディスカッションの機会を持つ例

31. 5. 小さい単位でレビューを繰り返すこと SonicGarden Study 11 #sg_study 31

32. 5. 小さい単位でレビューを繰り返すこと •  大きな単位だとレビューする側もされる側も出番が回ってきにくい。 – 機会を増やすことでみんながレビュー体験をする／小口化することで習慣化する •  チーム全体で誰かの書いたコードをレビューするのも有効。 SonicGarden Study 11 #sg_study 32

33. 6. 指摘は素直な気持ちで受け入れること SonicGarden Study 11 #sg_study 33

34. 6. 指摘は素直な気持ちで受け入れること •  指摘を素直に受け止められないと成長できない。 •  受け入れた後は同じ失敗を繰り返さない。 – 漠然とコードレビューを受けて、その場はその場で何とかやり過ごして、また同じ失敗を繰り返すメンタルモデルではダメ。 SonicGarden Study 11 #sg_study 34

35. 7. 指摘は人格否定でないことを理解すること SonicGarden Study 11 #sg_study 35

36. 7. 指摘は人格否定でないことを理解すること •  自分が一生懸命書いたコードに対して手厳しく指摘を受けるのは辛いこと。。 •  でも、指摘するのはコードであって、あなたの人格を否定しているわけではありません！ •  お互いに「コード」の話をしている、ということを理解した上で指摘をしましょう。 SonicGarden Study 11 #sg_study 36

37. SonicGarden Study 11 #sg_study まとめ 37

38. 今日お話した内容 1. クソコードとは何か？ 2. 優れたプログラマとは何か？ 3. コードレビューが改善の近道 4. コードレビュー7つの秘訣 SonicGarden Study 11 #sg_study 38

39. 今日お話した内容 •  クソコードとは読む人を怒りの渦に叩きこむコードである。 •  優秀なプログラマに読まれる機会があれば、その人のコードは劇的に改善する。（※ 効用には個人差があります） SonicGarden Study 11 #sg_study 39

40. 今日お話した内容 1.  レビューの観点を明確にすること 2.  我が身に返ることを恐れずに指摘すること 3.  何故悪いコードなのかを論理的に説明すること 4.  良いコードについて共通認識を持つこと 5.  小さい単位でレビューを繰り返すこと 6.  指摘は素直な気持ちで受け入れること 7.  指摘は人格否定でないことを理解すること SonicGarden Study 11 #sg_study 40

41. SonicGarden Study 11 #sg_study ご視聴ありがとうございました！ 41

デキるプログラマだけが知っているコードレビュー7つの秘訣

Vous êtes en train de lire un aperçu.

Consultez-les par la suite

デキるプログラマだけが知っているコードレビュー7つの秘訣

Plus De Contenu Connexe

Diaporamas pour vous (20)

Les utilisateurs ont également aimé (20)

Similaire à デキるプログラマだけが知っているコードレビュー7つの秘訣 (20)

Plus par Masahiro Nishimi (9)

Plus récents (20)