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

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

Embed Size (px)

DESCRIPTION

SonicGarden Study #11で放送された資料から一部スライドを抜いたものになります。 http://sonicgarden.doorkeeper.jp/events/13229 ----- 優れたプログラマだけが優れたソースコードを書くことができます。 では優れたプログラマになるにはどうすれば良いでしょうか。 自分の書いたコードを、優れたプログラマに指摘してもらうことが一番の近道です。それがコードレビューです。たった一人でコードレビューも受けずに、ただ書き続けてもクソコードはクソコードのままなのです。 そこで今回は、良いコードが書けるプログラマになるための、コードレビューを上手に実践する秘訣を話します。

Citation preview

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

SonicGarden  Study  11  #sg_study  

いつまでクソコードを 書き続けるの?

デキるプログラマだけが知っている コードレビュー7つの秘訣 ソニックガーデン 西見 公宏

1  

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

SonicGarden  Study  11  #sg_study  

西見 公宏 Nishimi Masahiro

2  

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

自己紹介

SonicGarden  Study  11  #sg_study  

西見 公宏 Masahiro Nishimi @mah_lab 昭和58年生まれ 東京育ち 2児(双子)の父親です

3  

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

今日お話する内容

1. クソコードとは何か? 2. 優れたプログラマとは何か? 3. コードレビューが改善の近道 4. コードレビュー7つの秘訣

SonicGarden  Study  11  #sg_study   4  

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

SonicGarden  Study  11  #sg_study  

1. クソコードとは何か?

5  

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

SonicGarden  Study  11  #sg_study  

「クソコード」とは・・・

6  

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

SonicGarden  Study  11  #sg_study  

「クソコード」とは・・・

読む人を怒りの渦に 叩きこむコードである

7  

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

つまり

•  「クソコード」とは読んだ人の怒りを体現した言葉である。

SonicGarden  Study  11  #sg_study   8  

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

クソコード3つの特徴 1. 読めないコード

–  変数名が暗号/制御フローが無駄に複雑/メソッド名と処理の内容が合ってない etc...

2. 要領の悪いコード –  言語レベルで用意されている機能を素直に使わない(例:Go

を使っているのにゴルーチンを使わない) etc...

3. 意図がわからないコード –  フレームワークのレールに従っていない etc...

SonicGarden  Study  11  #sg_study   9  

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

クソコードかどうかは 読まれるまで分からない •  シュレディンガーの猫 •  読んだ人の当事者感によって度合いが変わる(例:「え、このコード、俺がメンテするの!?」)

•  自分で読んでダメだと思うなら、最初から直して・・・。

SonicGarden  Study  11  #sg_study   10  

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

SonicGarden  Study  11  #sg_study  

2. 優れたプログラマとは何か?

11  

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

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

SonicGarden  Study  11  #sg_study   12  

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

優れたプログラマを見極める簡単な試験 書いたコードの1行1行に対して、 以下の項目を質問する。

l どういう処理なのか? l どういう意図で書いたのか?

SonicGarden  Study  11  #sg_study   13  

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

SonicGarden  Study  11  #sg_study  

3. コードレビューが改善の近道

14  

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

まずは読まれなければはじまらない

SonicGarden  Study  11  #sg_study   15  

読む人に怒りをもたらすのが クソコード

ならば

読む人がいなくては はじまらない

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

誰に読まれるのがベストか?

SonicGarden  Study  11  #sg_study   16  

優れたプログラマ

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

何故優れたプログラマが読むべきなのか?

SonicGarden  Study  11  #sg_study   17  

•  単純に同じレベルのプログラマが指摘しても、指摘の内容に限界があるため。

限界の壁

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

優れたプログラマは限界を引き上げます

SonicGarden  Study  11  #sg_study   18  

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

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

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

ただし・・・

SonicGarden  Study  11  #sg_study   19  

・・・

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

•  ソニックガーデンでも新しく入ったプログラマを教育する手段としてコードレビューを取り入れています。

•  その中で気付いたポイントをいくつかピックアップしました。

•  それが今からお伝えする「7つの秘訣」です。

より効果を上げるために

SonicGarden  Study  11  #sg_study   20  

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

SonicGarden  Study  11  #sg_study  

4. コードレビュー7つの秘訣

21  

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

1. レビューの観点を明確にすること

SonicGarden  Study  11  #sg_study   22  

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

1. レビューの観点を明確にすること •  リリース前レビューの話なのか、コード品質の話なのか。

•  読む側も漠然と読んで、漠然とコメントを書くのでは効果を生まない。 – 観点を明確にしてレビューすることで指摘する側の精度も上がります。

SonicGarden  Study  11  #sg_study   23  

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

2. 我が身に返ることを恐れずに指摘すること

SonicGarden  Study  11  #sg_study   24  

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

2. 我が身に返ることを恐れずに指摘すること •  自分のことは棚に上げる! •  鋭くツッコミを入れられないなら、レビューの意味は無い。

•  指摘して自分の耳が痛いなら、自分にとっても成長のチャンスだと捉える。

SonicGarden  Study  11  #sg_study   25  

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

3. 何故悪いコードなのかを論理的に 説明すること

SonicGarden  Study  11  #sg_study   26  

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

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

•  論理的に説明できないなら指摘をすべきではない。

SonicGarden  Study  11  #sg_study   27  

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

4. 良いコードについて共通認識を持つこと

SonicGarden  Study  11  #sg_study   28  

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

4. 良いコードについて共通認識を持つこと •  コードレビューのあとに雰囲気が悪くなるのはチームで「良いコード」の認識が揃っていない証拠。

•  「良いコード」を共有することでチームとして成長する。

•  認識を揃えるためには、あるコードを題材にディスカッションの機会を設けると良い。

SonicGarden  Study  11  #sg_study   29  

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

SonicGarden  Study  11  #sg_study   30  

チームでディスカッションの 機会を持つ例

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

5. 小さい単位でレビューを繰り返すこと

SonicGarden  Study  11  #sg_study   31  

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

5. 小さい単位でレビューを繰り返すこと

•  大きな単位だとレビューする側もされる側も出番が回ってきにくい。 – 機会を増やすことでみんながレビュー体験をする/小口化することで習慣化する

•  チーム全体で誰かの書いたコードをレビューするのも有効。

SonicGarden  Study  11  #sg_study   32  

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

6. 指摘は素直な気持ちで受け入れること

SonicGarden  Study  11  #sg_study   33  

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

6. 指摘は素直な気持ちで受け入れること

•  指摘を素直に受け止められないと成長できない。

•  受け入れた後は同じ失敗を繰り返さない。 – 漠然とコードレビューを受けて、その場はその場で何とかやり過ごして、また同じ失敗を繰り返すメンタルモデルではダメ。

SonicGarden  Study  11  #sg_study   34  

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

7. 指摘は人格否定でないことを理解すること

SonicGarden  Study  11  #sg_study   35  

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

7. 指摘は人格否定でないことを理解すること •  自分が一生懸命書いたコードに対して手厳しく指摘を受けるのは辛いこと。。

•  でも、指摘するのはコードであって、あなたの人格を否定しているわけではありません!

•  お互いに「コード」の話をしている、ということを理解した上で指摘をしましょう。

SonicGarden  Study  11  #sg_study   36  

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

SonicGarden  Study  11  #sg_study  

まとめ

37  

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

今日お話した内容

1. クソコードとは何か? 2. 優れたプログラマとは何か? 3. コードレビューが改善の近道 4. コードレビュー7つの秘訣

SonicGarden  Study  11  #sg_study   38  

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

今日お話した内容 •  クソコードとは読む人を怒りの渦に叩きこむコードである。

•  優秀なプログラマに読まれる機会があれば、その人のコードは劇的に改善する。  (※ 効用には個人差があります)

SonicGarden  Study  11  #sg_study   39  

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

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

SonicGarden  Study  11  #sg_study   40  

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

SonicGarden  Study  11  #sg_study  

ご視聴ありがとうございました! 41