コードレビューを受ける新人の心構えと準備

2023/12/11に行われた『エンジニアのためのスキルアップ勉強会#1「妥協しないコードレビュー」』で登壇した際に使用したスライドです

Transcript

1. コードレビューを受ける新人の 心構えと準備 2023/12/11 遠藤光敏(dosan)
2. 自己紹介
3. 遠藤光敏(dosan) X：@yatsuhashi168 • 26歳 • 2022年4月ソニックガーデン入社 • 居場所：岡山 • 趣味：コーヒー、ゲーム(FGO、オクトラ、原神、etc…）
4. 伝えたいこと
5. 良いコードレビュー にするために
6. レビュイーも できることがある
7. 動機 • 最初はレビューが怖かった • レビューは採点では無い • より良いコードにしていくためのもの 　　　レビューは成長する機会
8. 心構え 1. コメントの口調は気にしない 2. 「なぜ」を考える 準備 1. 同じレビューをされない 2. 自分以外のレビューを見る
9. 準備
10. 同じレビューをされない
12. レビュアーの負担を減らそう
13. レビュアーの負担を減らそう ・レビューにかかる時間が増える ・重要な指摘箇所を見落とす可能性がある
14. 凡ミスに多い傾向 ・orderのつけ忘れ ・()、{}の空白が揃ってない ・シングルクォートとダブルクォートがごっちゃになっている ・etc… セルフレビューで防ごう
15. 自分以外のレビューを見る
16. ・同期、後輩 ・レベルが近いので読みやすい ・お互いにレビューするのもあり 　　その指摘、自分も踏む可能性があります
17. 心構え
18. コメントの口調は気にしない
19. こんなコメントが飛んでくるかもしれません
20. 気にしない！！！
21. ・攻撃したいわけではない ・忙しいと口調まで気にかけられなかったりする ・慣れる 必要以上に怖がることはない
22. 「なぜ」を考える
23. <div> <% if user.role == 'admin' %> 管理者 <% elsif user.role == 'advisor' %> アドバイザー <% elsif user.role == 'other' %> その他 <% end %> </div>
24. 「モデルにメソッドを作った方がいいよ」
25. def role_text case self.role when 'admin' '管理者' when 'advisor' 'アドバイザー' when 'mentor' 'メンター' else 'その他' end end
26. <div> <%= user.role_text %> </div>
27. <div> <% if user.role == 'admin' %> 管理者 <% elsif user.role == 'advisor' %> アドバイザー <% elsif user.role == 'mentor' %> メンター <% end %> </div> <div> <%= user.role_text %> </div>
28. めでたしめでたし
29. ではない • もったいない • 全く同じ場合にしか使えない 　　　　「なぜ」を考えよう
30. 「なぜ」を考える • ビューの構造がわかりづらい • テストが書きにくい 抽象度をあげて自分のストックにする
31. まとめ
32. 心構え 1. コメントの口調は気にしない 2. 「なぜ」を考える 準備 1. 同じレビューをされない 2. 自分以外のレビューを見る
33. レビュイーも できることがある
34. ご清聴ありがとうございました