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
0
1.3k
コードレビューをレビューできてますか?(code review)
great tower
December 16, 2021
Tweet
Share
Other Decks in Technology
See All in Technology
継続的にアウトカムを生み出し ビジネスにつなげる、 戦略と運営に対するタイミーのQUEST(探求)
zigorou
0
830
怖くない!ゼロから始めるPHPソースコードコンパイル入門
colopl
0
160
TSKaigi 2024 の登壇から広がったコミュニティ活動について
tsukuha
0
170
Server-Side Engineer of LINE Sukimani
lycorp_recruit_jp
0
360
宇宙ベンチャーにおける最近の情シス取り組みについて
axelmizu
0
120
GitHub Copilot のテクニック集/GitHub Copilot Techniques
rayuron
39
16k
DUSt3R, MASt3R, MASt3R-SfM にみる3D基盤モデル
spatial_ai_network
2
260
レンジャーシステムズ | 会社紹介(採用ピッチ)
rssytems
0
280
小学3年生夏休みの自由研究「夏休みに Copilot で遊んでみた」
taichinakamura
0
180
ゼロから創る横断SREチーム 挑戦と進化の軌跡
rvirus0817
3
280
スタートアップで取り組んでいるAzureとMicrosoft 365のセキュリティ対策/How to Improve Azure and Microsoft 365 Security at Startup
yuj1osm
0
240
メンタル面でもつよつよエンジニアになる/登壇資料(井田 献一朗)
hacobu
0
120
Featured
See All Featured
Embracing the Ebb and Flow
colly
84
4.5k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
251
21k
The Pragmatic Product Professional
lauravandoore
32
6.3k
The Cult of Friendly URLs
andyhume
78
6.1k
JavaScript: Past, Present, and Future - NDC Porto 2020
reverentgeek
47
5.1k
Designing for humans not robots
tammielis
250
25k
A designer walks into a library…
pauljervisheath
205
24k
Build your cross-platform service in a week with App Engine
jlugia
229
18k
Product Roadmaps are Hard
iamctodd
PRO
50
11k
Designing for Performance
lara
604
68k
Learning to Love Humans: Emotional Interface Design
aarron
274
40k
Gamification - CAS2011
davidbonilla
80
5.1k
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 まとめ 自身のレビューを振り返ったことで課題が明確になりまし た!
レビューはコメントは色んな人の知識と経験の集合体 だからこそ、ちゃんと振り返れば自分の足りない部分がどこ かにはある