Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Sign up for free
Menu
Search
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Pricing
Search
Sign in
Sign up for free
コードレビューをレビューできてますか?(code review)
Search
Sponsored
·
SiteGround - Reliable hosting with speed, security, and support you can count on.
→
great tower
December 16, 2021
Technology
1.7k
0
Share
Embed
Copy iframe code
Copy JS code
Copy link
Start on current slide
コードレビューをレビューできてますか?(code review)
great tower
December 16, 2021
Other Decks in Technology
See All in Technology
分析の民主化を支えるメタデータ整備の実践
kakehashi
PRO
4
1.2k
今さら理解するウェブフォントver.2026
takanorip
3
510
検証フェーズはAIの回答を判断するための学習機会
toru_kubota
2
550
実践! SRE × AI - ABEMA における AIOps -
tetsuya28
7
2.5k
Google Cloud Next Tokyo 26登壇時のスクリプト
recruitengineers
PRO
0
210
Execution in the Kingdom of Agents: Reflections on Abstraction and Complexity
bcantrill
0
890
購入ドメインでの課題と取り組み
ykagano
0
150
メルペイ 会計システム概要と歴史
mewuto
0
200
TinyGoでオブザーバビリティ?OpenTelemetryはTinyGoで使えるのか / Is OpenTelemetry available in TinyGo environment?
ymotongpoo
2
120
SREでアラート疲れを 解決しよう!
kairim0
1
220
LocalStack を使ったサーバーレスアプリケーション開発 / Serverless Development with LocalStack
kakakakakku
1
340
初対面のメンバーがチームを組み、Claude Codeでゼロからサービスをリリース
mkdev10
1
110
Featured
See All Featured
Building a Scalable Design System with Sketch
lauravandoore
464
34k
From Legacy to Launchpad: Building Startup-Ready Communities
dugsong
0
350
We Have a Design System, Now What?
morganepeng
55
8.4k
We Analyzed 250 Million AI Search Results: Here's What I Found
joshbly
2
2k
Why Our Code Smells
bkeepers
PRO
340
58k
Conquering PDFs: document understanding beyond plain text
inesmontani
PRO
4
3.2k
Measuring & Analyzing Core Web Vitals
bluesmoon
9
1k
Optimising Largest Contentful Paint
csswizardry
37
4k
Are puppies a ranking factor?
jonoalderson
2
4k
Building Better People: How to give real-time feedback that sticks.
wjessup
370
20k
Breaking role norms: Why Content Design is so much more than writing copy - Taylor Woolridge
uxyall
1
450
ラッコキーワード サービス紹介資料
rakko
1
5.2M
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 まとめ 自身のレビューを振り返ったことで課題が明確になりまし た!
レビューはコメントは色んな人の知識と経験の集合体 だからこそ、ちゃんと振り返れば自分の足りない部分がどこ かにはある