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
980
コードレビューをレビューできてますか?(code review)
great tower
December 16, 2021
Tweet
Share
Other Decks in Technology
See All in Technology
人間の尊厳、幸福、アクセシビリティ / 第116回「WEB TOUCH MEETING」アクセシビリティSP
nulabinc
PRO
2
180
マルチアカウント環境への発見的統制の導入
ch1aki
1
1.3k
シン・Kafka / shin-kafka
oracle4engineer
PRO
7
2.7k
アプリがつくるNOT A HOTELブランド
hokuts
1
450
なぜ NOT A HOTEL が Web3 に取り組むのか - NOT A HOTEL TECH TALK
ynunokawa
0
160
Tebiki株式会社 エンジニア採用資料
tebiki
0
4.1k
元インフラエンジニアに成る / Human Resources to Human Relations
bobtani
3
770
スタートアップの技術顧問を3年間続けて発生した事と気付き
biwakonbu
0
160
Hands-on / Kaname Frusawa / Cloud Compare Users Meetup 2024 at University of Tokyo on April 17
paraworld
2
470
DevOpsメトリクスとアウトカムの接続にトライ!開発プロセスを通して計測できるメトリクスの活用方法
ham0215
1
190
Apple Vision Pro trial session
akkeylab
0
120
巨大なテーブルのテーブル定義を無停止で安全に誰でも変更できるようにする / Table-definitions-for-huge-tables-can-be-modified-by-anyone-safely-and-non-disruptively
freee
1
740
Featured
See All Featured
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
1
1.3k
Why You Should Never Use an ORM
jnunemaker
PRO
50
8.6k
Designing Experiences People Love
moore
136
23k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
9
8.3k
Git: the NoSQL Database
bkeepers
PRO
422
63k
Designing for humans not robots
tammielis
247
25k
GraphQLとの向き合い方2022年版
quramy
31
12k
RailsConf 2023
tenderlove
2
530
Designing for Performance
lara
602
67k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
morganepeng
115
18k
Fight the Zombie Pattern Library - RWD Summit 2016
marcelosomers
226
16k
Web Components: a chance to create the future
zenorocha
305
41k
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 まとめ 自身のレビューを振り返ったことで課題が明確になりまし た!
レビューはコメントは色んな人の知識と経験の集合体 だからこそ、ちゃんと振り返れば自分の足りない部分がどこ かにはある