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
66
若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜
仕事とお酒を愛でる「ソースコード」レビューまつり!2024/3/29
nakampany
March 29, 2024
Tweet
Share
More Decks by nakampany
See All by nakampany
自分発信のミーティングがノープランすぎて失敗しかけた話
nakampany
1
320
アドベントカレンダーで投稿するのはタイパが悪いのか?
nakampany
1
65
Featured
See All Featured
Reflections from 52 weeks, 52 projects
jeffersonlam
348
20k
Large-scale JavaScript Application Architecture
addyosmani
511
110k
The Psychology of Web Performance [Beyond Tellerrand 2023]
tammyeverts
46
2.3k
What's in a price? How to price your products and services
michaelherold
244
12k
Code Reviewing Like a Champion
maltzj
521
39k
The Straight Up "How To Draw Better" Workshop
denniskardys
232
140k
No one is an island. Learnings from fostering a developers community.
thoeni
21
3.1k
Documentation Writing (for coders)
carmenintech
67
4.6k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
251
21k
Improving Core Web Vitals using Speculation Rules API
sergeychernyshev
9
470
The Invisible Side of Design
smashingmag
299
50k
Site-Speed That Sticks
csswizardry
4
390
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 まとめ