少し変わったコードレビュー(KGDC)

コードレビューを改善すれば、勉強会にもなり、チーム全体の技術力アップ出来ます。

Transcript

1. 少し変わったコードレビュー 2021/03/17 KGDC Tech Conference Carlos Planter
2. NAME: COUNTRY: COMPANY: POSITION: CARLOS PLANTER MEXICO mediba.inc FRONT-END ENGINEER 自己紹介
3. プロダクト紹介 auゲーム ゲームポータル
4. チーム体制 フロントエンド バックエンド インフラ
5. •自己組織チーム •技術力強化 チームとしての目標
6. どうやってできるのか？
7. コードレビュー使いましょう！
8. よくあるコードレビュー① Main coder そもそもBE わかんない… ふーーーん LGTM!! 俺の素晴らしいコード 見て頂戴！ 赤井さんはいいと 思っているから、 僕もそうしよう！
9. よくあるコードレビュー① Main coder ふーーーん LGTM!! 俺の素晴らしいコード 見て頂戴！ 赤井さんはいいと 思っているから、 僕もそうしよう！ Problem • 言語や環境わからないとコードレ ビューできない • わかるようになりたいけど、Github で質問しづらい
10. よくあるコードレビュー② Main coder/Driver ここでこうやって、ああ やって、難しかったけど この関数を使って先々週 あった問題も回避できて、 最高なコード書けたぜ！ ふーーーん よくわからないけど、 質問していいのかな…
11. よくあるコードレビュー② Main coder ここでこうっやて、ああ やって、難しかったけど この関数を使って先々週 あった問題も回避できて、 最高なコード書けたぜ！ ふーーーん これわからないけど、 聞いていいのかな… Problem • メインコーダーは皆が自分と同じ知 識があると前提してしまい喋り出す • 一方的なコミュニケーションにな りがち
12. 改善してみましょう！
13. Main coder/Driver Driver Main coder 1) メインコーダーはタスクの条 件と期待結果を説明する。画面 で見せたり、気になったところ も説明する。 2) ドライバーを指名して、メイ ンコーダーと交代する。 3) ドライバーはコードを説明し ようとして、わからないところ をメインコーダーに聞いてみる。 少し変わったコードレビュー
14. Improvement ① ・コードを書いていない人が話し 手なので理解できていないと説明 できない!! → 質問しやすい!! 説明する中で自然と質問が湧 いてくる
15. Improvement ② ・メインコーダーは質問への回答 に集中できる
16. Improvement ③ ・いつもより少し時間がかかりま すが、コードレビュー＋勉強会だ と思ったら時間がとても効率 的！！
17. 結果
18. インフラ フロントエンド チームの技術力アップ バックエンド
19. ご視聴ありがとうございました
20. Muchas Gracias