Upgrade to Pro — share decks privately, control downloads, hide ads and more …

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

E17011aa45f8d923495d381b2b65afe2?s=47 Amakza
March 19, 2021

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

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

E17011aa45f8d923495d381b2b65afe2?s=128

Amakza

March 19, 2021
Tweet

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