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
若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜
Search
nakampany
March 29, 2024
1
59
若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜
仕事とお酒を愛でる「ソースコード」レビューまつり!2024/3/29
nakampany
March 29, 2024
Tweet
Share
More Decks by nakampany
See All by nakampany
自分発信のミーティングがノープランすぎて失敗しかけた話
nakampany
1
140
アドベントカレンダーで投稿するのはタイパが悪いのか?
nakampany
1
56
Featured
See All Featured
Chrome DevTools: State of the Union 2024 - Debugging React & Beyond
addyosmani
2
170
"I'm Feeling Lucky" - Building Great Search Experiences for Today's Users (#IAC19)
danielanewman
226
22k
Java REST API Framework Comparison - PWX 2021
mraible
28
8.3k
A better future with KSS
kneath
238
17k
The Cult of Friendly URLs
andyhume
78
6.1k
Documentation Writing (for coders)
carmenintech
66
4.5k
The Straight Up "How To Draw Better" Workshop
denniskardys
232
140k
ピンチをチャンスに:未来をつくるプロダクトロードマップ #pmconf2020
aki_iinuma
111
49k
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
mongodb
44
9.3k
Building Better People: How to give real-time feedback that sticks.
wjessup
365
19k
Product Roadmaps are Hard
iamctodd
PRO
49
11k
How to Think Like a Performance Engineer
csswizardry
22
1.2k
Transcript
若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜 仕事とお酒を愛でる「ソースコード」レビューまつり! 2024/3/29 株式会社HRBrain 中嶋将太
2 AGENDA • 自己紹介 • 若手エンジニアのコードレビューについて • 若手エンジニアがレビューするメリットとは? • コードレビューの第一歩目
• レビューができるようになるには? • まとめ
自己紹介
4 自己紹介 • 中嶋 将太(なかじ) • HRbrain 24卒インターン - 半年 PHPでインターン
- もう少しで HRBrainでインターン1年 • Go言語で新規事業開発をおこなっています • #ゴルフ#麻雀🀄 #海外旅行✈ @nakampany
• 今回の内容記事にしてます!(ややばずったのかな?) • https://qiita.com/nakampany/items/49fa90cf584fd9ad418d 5 若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜
若手エンジニアのコードレビュー
• 「コードレビューは自分にはハードルが高いなー🙁」 • 「知識ないから、先輩のPRにコメントするの恐れ多い😨」 などと思っているor思ったことありませんか?? 自分は、当然に 先輩エンジニアやチームメンバーのコードをレビューするのは恐れ多く、 ワンステップ先だと思っていました〜 7 若手エンジニアのコードレビュー
最近は、積極的に先輩エンジニアやチームメンバーのPRにコメントをつけている (全く完璧ではなく、なんちゃってですが😅) ですが、 『的を得たコメントできているんだろうか?😟』 と思うことが多くあります。 8 若手エンジニアのコードレビュー
先輩エンジニアとCTOからアドバイスをもとに、 若手エンジニアのコードレビューについての 考えをまとめました〜 9 若手エンジニアのコードレビュー
若手エンジニアがレビューする メリットとは?
11 若手エンジニアがレビューするメリットとは? • 1. 自己のスキル向上 • 2. チーム内コミュニケーションの活発化 • 3.
レビューにレビューをもらえる レビューコメント Goはキャメルケースなのに、 URLが入ってる!(Url→URL) 見やすいのでいいですね! レビューのレビュー 大文字が推奨されています companyURLName := fetchCompanyURLName()
12 若手エンジニアがレビューするメリットとは? 若手がレビューすることはいいことだってわかるけど、 そんないきなりできんわい!
13 コードレビューの第一歩目 • 指摘(多くの人が想像しているコードレビュー) • 質問(この実装どういう意図で?) • 承認(褒める、この実装すごい!) 「質問」、「承認」から入ると良いです! 「PRに上記のコメントがついて、嫌がる先輩やチームメンバーいない!😇」
というマインドをもち、果敢にコメントをつけると良いと思います!
スピード感を持ってレビューが できるようになるには?
斜め上のPRで学ぶ! 15 スピード感を持ってレビューができるようになるには?
斜め上のPRとは、、、 斜め上の先輩(1つ上の新卒のエンジニア)のPRです! (自分(24卒)であれば、斜め上の先輩→23卒) - レビュアーがどこをみているか - レビューコメントのちょっとした作法 - レビュイーが指摘された箇所を先回りして予習 16
スピード感を持ってレビューができるようになるには?
まとめ
• 若手がレビューするメリットは、「レビューにレビューをもらえる」 • コードレビューの第一歩目は「質問」、「承認」 • レビューの知識は斜め上のPRを見て学ぼう! 18 まとめ