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
56
若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜
仕事とお酒を愛でる「ソースコード」レビューまつり!2024/3/29
nakampany
March 29, 2024
Tweet
Share
More Decks by nakampany
See All by nakampany
自分発信のミーティングがノープランすぎて失敗しかけた話
nakampany
1
120
アドベントカレンダーで投稿するのはタイパが悪いのか?
nakampany
1
48
Featured
See All Featured
Adopting Sorbet at Scale
ufuk
72
8.9k
Reflections from 52 weeks, 52 projects
jeffersonlam
346
20k
Thoughts on Productivity
jonyablonski
65
4.2k
Building Adaptive Systems
keathley
36
2.1k
Visualization
eitanlees
142
15k
A Tale of Four Properties
chriscoyier
155
22k
Put a Button on it: Removing Barriers to Going Fast.
kastner
58
3.4k
Pencils Down: Stop Designing & Start Developing
hursman
118
11k
The Success of Rails: Ensuring Growth for the Next 100 Years
eileencodes
41
6.4k
Docker and Python
trallard
39
3k
Ruby is Unlike a Banana
tanoku
96
10k
Git: the NoSQL Database
bkeepers
PRO
425
64k
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 まとめ