若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう！〜

仕事とお酒を愛でる「ソースコード」レビューまつり！2024/3/29

Transcript

1. 若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう！〜 仕事とお酒を愛でる「ソースコード」レビューまつり！ 2024/3/29 株式会社HRBrain 中嶋将太
2. 2 AGENDA • 自己紹介 • 若手エンジニアのコードレビューについて • 若手エンジニアがレビューするメリットとは？ • コードレビューの第一歩目 • レビューができるようになるには？ • まとめ
3. 自己紹介
4. 4 自己紹介 • 中嶋 将太（なかじ）
 • HRbrain 24卒インターン - 半年　PHPでインターン - もう少しで　HRBrainでインターン1年 • Go言語で新規事業開発をおこなっています • #ゴルフ󰝱#麻雀🀄　#海外旅行✈ @nakampany
5. • 今回の内容記事にしてます！(ややばずったのかな？) • https://qiita.com/nakampany/items/49fa90cf584fd9ad418d 5 若手エンジニアのコードレビュー 〜斜め上のPRを見て学ぼう！〜
6. 若手エンジニアのコードレビュー
7. • 「コードレビューは自分にはハードルが高いなー🙁」 • 「知識ないから、先輩のPRにコメントするの恐れ多い😨」 などと思っているor思ったことありませんか？？ 自分は、当然に 先輩エンジニアやチームメンバーのコードをレビューするのは恐れ多く、 ワンステップ先だと思っていました〜 7 若手エンジニアのコードレビュー
8. 最近は、積極的に先輩エンジニアやチームメンバーのPRにコメントをつけている （全く完璧ではなく、なんちゃってですが😅） ですが、 『的を得たコメントできているんだろうか？😟』 と思うことが多くあります。 8 若手エンジニアのコードレビュー
9. 先輩エンジニアとCTOからアドバイスをもとに、 若手エンジニアのコードレビューについての 考えをまとめました〜 9 若手エンジニアのコードレビュー
10. 若手エンジニアがレビューする メリットとは？
11. 11 若手エンジニアがレビューするメリットとは？ • 1. 自己のスキル向上 • 2. チーム内コミュニケーションの活発化 • 3. レビューにレビューをもらえる レビューコメント Goはキャメルケースなのに、 URLが入ってる！（Url→URL） 見やすいのでいいですね！ レビューのレビュー 大文字が推奨されています companyURLName := fetchCompanyURLName()
12. 12 若手エンジニアがレビューするメリットとは？ 　　若手がレビューすることはいいことだってわかるけど、 そんないきなりできんわい！
13. 13 コードレビューの第一歩目 • 指摘（多くの人が想像しているコードレビュー） • 質問（この実装どういう意図で？） • 承認（褒める、この実装すごい！） 「質問」、「承認」から入ると良いです！ 　「PRに上記のコメントがついて、嫌がる先輩やチームメンバーいない！😇」 　　　　　というマインドをもち、果敢にコメントをつけると良いと思います！
14. スピード感を持ってレビューが できるようになるには？
15. 斜め上のPRで学ぶ！ 15 スピード感を持ってレビューができるようになるには？
16. 斜め上のPRとは、、、 斜め上の先輩（１つ上の新卒のエンジニア）のPRです！ （自分(24卒)であれば、斜め上の先輩→23卒） - レビュアーがどこをみているか - レビューコメントのちょっとした作法 - レビュイーが指摘された箇所を先回りして予習 16 スピード感を持ってレビューができるようになるには？
17. まとめ
18. • 若手がレビューするメリットは、「レビューにレビューをもらえる」 • コードレビューの第一歩目は「質問」、「承認」 • レビューの知識は斜め上のPRを見て学ぼう！ 18 まとめ