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
コードレビューをレビューできてますか?(code review)
Search
great tower
December 16, 2021
Technology
1.7k
0
Share
コードレビューをレビューできてますか?(code review)
great tower
December 16, 2021
Other Decks in Technology
See All in Technology
さくらのクラウドでつくるCloudNative Daysのオブザーバビリティ基盤
b1gb4by
0
140
Oracle AI Database@Google Cloud:サービス概要のご紹介
oracle4engineer
PRO
6
1.3k
今年60歳のおっさんCBになる
kentapapa
1
360
Data Enabling Team立ち上げました
sansantech
PRO
0
300
終盤で崩壊させないAI駆動開発
j5ik2o
0
420
プロダクトを触って語って理解する、チーム横断バグバッシュのすすめ / 20260411 Naoki Takahashi
shift_evolve
PRO
1
250
プロジェクトマネジメントは AIでどう変わるか?
mkg5383
0
140
Cortex Code君、今日から内製化支援担当ね。
coco_se
0
320
NgRx SignalStore: The Power of Extensibility
rainerhahnekamp
0
190
🀄️ on swiftc
giginet
PRO
0
290
AIがコードを書く時代の ジェネレーティブプログラミング
polidog
PRO
3
670
"まず試す"ためのDatabricks Apps活用法 / Databricks Apps for Early Experiments and Validation
nttcom
1
230
Featured
See All Featured
Why Mistakes Are the Best Teachers: Turning Failure into a Pathway for Growth
auna
0
110
How to Ace a Technical Interview
jacobian
281
24k
Game over? The fight for quality and originality in the time of robots
wayneb77
1
160
We Analyzed 250 Million AI Search Results: Here's What I Found
joshbly
1
1.1k
More Than Pixels: Becoming A User Experience Designer
marktimemedia
3
370
Ruling the World: When Life Gets Gamed
codingconduct
0
190
Raft: Consensus for Rubyists
vanstee
141
7.4k
How to Talk to Developers About Accessibility
jct
2
170
Technical Leadership for Architectural Decision Making
baasie
3
310
Hiding What from Whom? A Critical Review of the History of Programming languages for Music
tomoyanonymous
2
680
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
254
22k
How to build an LLM SEO readiness audit: a practical framework
nmsamuel
1
710
Transcript
Copyright© Speee, Inc. All rights reserved. Confidential 株式会社Speee 2021/12/01 コードレビューをレビューできてますか?
Copyright© Speee, Inc. All rights reserved. 2 八木 塔偉(やぎ とうい)
@great_tower511 株式会社Speee エンジニア 2021年8月から 新規事業開発(toC&toB) 趣味 漫画(少年漫画) ゲーム(FPS) プロフィール
Copyright© Speee, Inc. All rights reserved. 3 伝えたいこと ・なんとなくフレームワーク、日々の業務に則って開発、レビューしている方
・とりあえず動けばOKな環境でエンジニアリングしている方 ・自分のレビューに疑いを持っている方 ・レビュー力は経験で片付けてしまっている方 が自分からもっと良いレビューできるようにしようと動き出すきっかけになっていた だけたら嬉しい!!
Copyright© Speee, Inc. All rights reserved. 4 背景 1~2年目の子達 僕
3ヶ月前まで 10年選手のエンジニア 僕 現在
Copyright© Speee, Inc. All rights reserved. 5 ちゃんとコードレビューできていなかったなぁ
環境が変わって気がついた。
Copyright© Speee, Inc. All rights reserved. 6 経験・・・?
どうすればレビューが自信を持ってできるようになるのか
Copyright© Speee, Inc. All rights reserved. 7 愚直に過去のPRについているレビューコメントを集めてみる
やってみたこと
Copyright© Speee, Inc. All rights reserved. 8 やってみたこと(具体例)
当たり前だよねっていうところも含めて見たものは全部集めてみた
Copyright© Speee, Inc. All rights reserved. 9 集めてどうしたか・・ ・このコメントはできないな
・そもそもその機能やライブラリを知らない? ・実装方法が思いつかない? ・このコメントは僕もできるな ・本当にレビュー依頼がきたときできてた?
Copyright© Speee, Inc. All rights reserved. 10 やってみたことのまとめ ・僕に一番足りないなとわかったこと
・設計の原理原則が身に染み付いているからできている ・一旦置いておく ・もっといい命名できている ・言語やフレームワーク、ライブラリの機能でもっといいものを知っている
Copyright© Speee, Inc. All rights reserved. 11 まとめ 自身のレビューを振り返ったことで課題が明確になりまし た!
レビューはコメントは色んな人の知識と経験の集合体 だからこそ、ちゃんと振り返れば自分の足りない部分がどこ かにはある