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
Sponsored
·
Your Podcast. Everywhere. Effortlessly.
Share. Educate. Inspire. Entertain. You do you. We'll handle the rest.
→
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
MIXIで活躍できるエンジニアを 若手社員目線で考えてみる
mixi_engineers
PRO
0
140
LLM・AIエージェントシステムベストプラクティス
shibuiwilliam
6
1.7k
コーチングの奥義 何もしないテクニック
jinwatanabe
0
150
形式手法特論:Hyperproperty とモデル検査 #kernelvm / Kernel VM Study Tokyo 19th
ytaka23
0
730
dbt と Snowflake と tag
kevinrobot34
1
190
生成 AI の基礎 〜 サンプル実装で学ぶ基本原理
enakai00
7
4.6k
サイボウズ 開発本部採用ピッチ / Cybozu Engineer Recruit
cybozuinsideout
PRO
12
85k
フィジカルAIの知識ゼロの僕でも AIを使えばここまでできた
tatsuya1970
0
160
AIに持続⼒を与える 判断の⻑期記憶設計
eiei114
1
680
医療の現場を変革に挑戦した半年間の軌跡 - PythonとAIで現場を変える / From Code to Care
soudai
PRO
1
590
Sets in Go
ramalho
1
1.2k
地震情報アプリを作ってみた
yama3133
1
110
Featured
See All Featured
HU Berlin: Industrial-Strength Natural Language Processing with spaCy and Prodigy
inesmontani
PRO
0
660
Breaking role norms: Why Content Design is so much more than writing copy - Taylor Woolridge
uxyall
0
380
Automating Front-end Workflow
addyosmani
1369
210k
Exploring anti-patterns in Rails
aemeredith
3
470
DevOps and Value Stream Thinking: Enabling flow, efficiency and business value
helenjbeal
1
360
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
mongodb
49
10k
Amusing Abliteration
ianozsvald
1
260
Design of three-dimensional binary manipulators for pick-and-place task avoiding obstacles (IECON2024)
konakalab
0
550
Discover your Explorer Soul
emna__ayadi
2
1.3k
Jamie Indigo - Trashchat’s Guide to Black Boxes: Technical SEO Tactics for LLMs
techseoconnect
PRO
0
630
What's in a price? How to price your products and services
michaelherold
247
13k
The SEO identity crisis: Don't let AI make you average
varn
0
530
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 まとめ 自身のレビューを振り返ったことで課題が明確になりまし た!
レビューはコメントは色んな人の知識と経験の集合体 だからこそ、ちゃんと振り返れば自分の足りない部分がどこ かにはある