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
Shota Nakajima
March 29, 2024
1
39
若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう!〜
仕事とお酒を愛でる「ソースコード」レビューまつり!2024/3/29
Shota Nakajima
March 29, 2024
Tweet
Share
More Decks by Shota Nakajima
See All by Shota Nakajima
アドベントカレンダーで投稿するのはタイパが悪いのか?
nakampany
1
24
Featured
See All Featured
Pencils Down: Stop Designing & Start Developing
hursman
117
11k
Building Applications with DynamoDB
mza
88
5.7k
Build The Right Thing And Hit Your Dates
maggiecrowley
25
2k
Distributed Sagas: A Protocol for Coordinating Microservices
caitiem20
323
20k
Product Roadmaps are Hard
iamctodd
45
9.8k
A designer walks into a library…
pauljervisheath
201
23k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
14
8.4k
Navigating Team Friction
lara
179
13k
Facilitating Awesome Meetings
lara
43
5.6k
GitHub's CSS Performance
jonrohan
1025
450k
Web Components: a chance to create the future
zenorocha
306
41k
Raft: Consensus for Rubyists
vanstee
133
6.3k
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 まとめ