少し変わったコードレビュー(KGDC)

by Amakza

Slide 1

Slide 1 text

少し変わったコードレビュー 2021/03/17 KGDC Tech Conference Carlos Planter

Slide 2

Slide 2 text

NAME: COUNTRY: COMPANY: POSITION: CARLOS PLANTER MEXICO mediba.inc FRONT-END ENGINEER 自己紹介

Slide 3

Slide 3 text

プロダクト紹介 auゲームゲームポータル

Slide 4

Slide 4 text

チーム体制フロントエンドバックエンドインフラ

Slide 5

Slide 5 text

•自己組織チーム •技術力強化チームとしての目標

Slide 6

Slide 6 text

どうやってできるのか？

Slide 7

Slide 7 text

コードレビュー使いましょう！

Slide 8

Slide 8 text

よくあるコードレビュー① Main coder そもそもBE わかんない… ふーーーん LGTM!! 俺の素晴らしいコード見て頂戴！赤井さんはいいと思っているから、僕もそうしよう！

Slide 9

Slide 9 text

よくあるコードレビュー① Main coder ふーーーん LGTM!! 俺の素晴らしいコード見て頂戴！赤井さんはいいと思っているから、僕もそうしよう！ Problem • 言語や環境わからないとコードレビューできない • わかるようになりたいけど、Github で質問しづらい

Slide 10

Slide 10 text

よくあるコードレビュー② Main coder/Driver ここでこうやって、ああやって、難しかったけどこの関数を使って先々週あった問題も回避できて、最高なコード書けたぜ！ふーーーんよくわからないけど、質問していいのかな…

Slide 11

Slide 11 text

よくあるコードレビュー② Main coder ここでこうっやて、ああやって、難しかったけどこの関数を使って先々週あった問題も回避できて、最高なコード書けたぜ！ふーーーんこれわからないけど、聞いていいのかな… Problem • メインコーダーは皆が自分と同じ知識があると前提してしまい喋り出す • 一方的なコミュニケーションになりがち

Slide 12

Slide 12 text

改善してみましょう！

Slide 13

Slide 13 text

Main coder/Driver Driver Main coder 1) メインコーダーはタスクの条件と期待結果を説明する。画面で見せたり、気になったところも説明する。 2) ドライバーを指名して、メインコーダーと交代する。 3) ドライバーはコードを説明しようとして、わからないところをメインコーダーに聞いてみる。少し変わったコードレビュー

Slide 14

Slide 14 text

Improvement ① ・コードを書いていない人が話し手なので理解できていないと説明できない!! → 質問しやすい!! 説明する中で自然と質問が湧いてくる

Slide 15

Slide 15 text

Improvement ② ・メインコーダーは質問への回答に集中できる

Slide 16

Slide 16 text

Improvement ③ ・いつもより少し時間がかかりますが、コードレビュー＋勉強会だと思ったら時間がとても効率的！！

Slide 17

Slide 17 text

結果

Slide 18

Slide 18 text

インフラフロントエンドチームの技術力アップバックエンド