Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
少し変わったコードレビュー(KGDC)
Search
Amakza
March 19, 2021
Programming
0
680
少し変わったコードレビュー(KGDC)
コードレビューを改善すれば、勉強会にもなり、チーム全体の技術力アップ出来ます。
Amakza
March 19, 2021
Tweet
Share
Other Decks in Programming
See All in Programming
2025 年のコーディングエージェントの現在地とエンジニアの仕事の変化について
azukiazusa1
24
12k
パッケージ設計の黒魔術/Kyoto.go#63
lufia
3
430
CloudflareのChat Agent Starter Kitで簡単!AIチャットボット構築
syumai
2
490
How Android Uses Data Structures Behind The Scenes
l2hyunwoo
0
440
はじめてのMaterial3 Expressive
ym223
2
310
プロパティベーステストによるUIテスト: LLMによるプロパティ定義生成でエッジケースを捉える
tetta_pdnt
0
320
AIでLINEスタンプを作ってみた
eycjur
1
230
[FEConf 2025] 모노레포 절망편, 14개 레포로 부활하기까지 걸린 1년
mmmaxkim
0
1.6k
実用的なGOCACHEPROG実装をするために / golang.tokyo #40
mazrean
1
270
アプリの "かわいい" を支えるアニメーションツールRiveについて
uetyo
0
260
複雑なドメインに挑む.pdf
yukisakai1225
5
1.1k
250830 IaCの選定~AWS SAMのLambdaをECSに乗り換えたときの備忘録~
east_takumi
0
390
Featured
See All Featured
Done Done
chrislema
185
16k
Mobile First: as difficult as doing things right
swwweet
224
9.9k
VelocityConf: Rendering Performance Case Studies
addyosmani
332
24k
"I'm Feeling Lucky" - Building Great Search Experiences for Today's Users (#IAC19)
danielanewman
229
22k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
morganepeng
131
19k
The Language of Interfaces
destraynor
161
25k
The Power of CSS Pseudo Elements
geoffreycrofte
77
6k
Visualization
eitanlees
148
16k
RailsConf & Balkan Ruby 2019: The Past, Present, and Future of Rails at GitHub
eileencodes
139
34k
StorybookのUI Testing Handbookを読んだ
zakiyama
31
6.1k
Build your cross-platform service in a week with App Engine
jlugia
231
18k
[RailsConf 2023] Rails as a piece of cake
palkan
57
5.8k
Transcript
少し変わったコードレビュー 2021/03/17 KGDC Tech Conference Carlos Planter
NAME: COUNTRY: COMPANY: POSITION: CARLOS PLANTER MEXICO mediba.inc FRONT-END ENGINEER
自己紹介
プロダクト紹介 auゲーム ゲームポータル
チーム体制 フロントエンド バックエンド インフラ
•自己組織チーム •技術力強化 チームとしての目標
どうやってできるのか?
コードレビュー使いましょう!
よくあるコードレビュー① Main coder そもそもBE わかんない… ふーーーん LGTM!! 俺の素晴らしいコード 見て頂戴! 赤井さんはいいと
思っているから、 僕もそうしよう!
よくあるコードレビュー① Main coder ふーーーん LGTM!! 俺の素晴らしいコード 見て頂戴! 赤井さんはいいと 思っているから、 僕もそうしよう!
Problem • 言語や環境わからないとコードレ ビューできない • わかるようになりたいけど、Github で質問しづらい
よくあるコードレビュー② Main coder/Driver ここでこうやって、ああ やって、難しかったけど この関数を使って先々週 あった問題も回避できて、 最高なコード書けたぜ! ふーーーん よくわからないけど、
質問していいのかな…
よくあるコードレビュー② Main coder ここでこうっやて、ああ やって、難しかったけど この関数を使って先々週 あった問題も回避できて、 最高なコード書けたぜ! ふーーーん これわからないけど、
聞いていいのかな… Problem • メインコーダーは皆が自分と同じ知 識があると前提してしまい喋り出す • 一方的なコミュニケーションにな りがち
改善してみましょう!
Main coder/Driver Driver Main coder 1) メインコーダーはタスクの条 件と期待結果を説明する。画面 で見せたり、気になったところ も説明する。
2) ドライバーを指名して、メイ ンコーダーと交代する。 3) ドライバーはコードを説明し ようとして、わからないところ をメインコーダーに聞いてみる。 少し変わったコードレビュー
Improvement ① ・コードを書いていない人が話し 手なので理解できていないと説明 できない!! → 質問しやすい!! 説明する中で自然と質問が湧 いてくる
Improvement ② ・メインコーダーは質問への回答 に集中できる
Improvement ③ ・いつもより少し時間がかかりま すが、コードレビュー+勉強会だ と思ったら時間がとても効率 的!!
結果
インフラ フロントエンド チームの技術力アップ バックエンド
ご視聴ありがとうございました
Muchas Gracias