若手エンジニアのコードレビュー 〜斜め上の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とは、、、 斜め上の先輩（１つ上の新卒のエンジニア）のPRです！ （自分(24卒)であれば、斜め上の先輩→23卒） - レビュアーがどこをみているか - レビューコメントのちょっとした作法 - レビュイーが指摘された箇所を先回りして予習 16 スピード感を持ってレビューができるようになるには？

まとめ

● 若手がレビューするメリットは、「レビューにレビューをもらえる」 ● コードレビューの第一歩目は「質問」、「承認」 ● レビューの知識は斜め上のPRを見て学ぼう！ 18 まとめ