少し変わったコードレビュー(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