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

by yatsuhashi

Slide 1

Slide 1 text

コードレビューを受ける新人の心構えと準備 2023/12/11 遠藤光敏(dosan)

Slide 2

Slide 2 text

自己紹介

Slide 3

Slide 3 text

遠藤光敏(dosan) X：@yatsuhashi168 ● 26歳 ● 2022年4月ソニックガーデン入社 ● 居場所：岡山 ● 趣味：コーヒー、ゲーム(FGO、オクトラ、原神、etc…）

Slide 4

Slide 4 text

伝えたいこと

Slide 5

Slide 5 text

良いコードレビューにするために

Slide 6

Slide 6 text

レビュイーもできることがある

Slide 7

Slide 7 text

動機 ● 最初はレビューが怖かった ● レビューは採点では無い ● より良いコードにしていくためのもの　　　レビューは成長する機会

Slide 8

Slide 8 text

心構え 1. コメントの口調は気にしない 2. 「なぜ」を考える準備 1. 同じレビューをされない 2. 自分以外のレビューを見る

Slide 9

Slide 9 text

準備

Slide 10

Slide 10 text

同じレビューをされない

Slide 11

Slide 11 text

No content

Slide 12

Slide 12 text

レビュアーの負担を減らそう

Slide 13

Slide 13 text

レビュアーの負担を減らそう・レビューにかかる時間が増える・重要な指摘箇所を見落とす可能性がある

Slide 14

Slide 14 text

凡ミスに多い傾向・orderのつけ忘れ・()、{}の空白が揃ってない・シングルクォートとダブルクォートがごっちゃになっている・etc… セルフレビューで防ごう

Slide 15

Slide 15 text

自分以外のレビューを見る

Slide 16

Slide 16 text

・同期、後輩・レベルが近いので読みやすい・お互いにレビューするのもあり　　その指摘、自分も踏む可能性があります

Slide 17

Slide 17 text

心構え

Slide 18

Slide 18 text

コメントの口調は気にしない

Slide 19

Slide 19 text

こんなコメントが飛んでくるかもしれません

Slide 20

Slide 20 text

気にしない！！！

Slide 21

Slide 21 text

・攻撃したいわけではない・忙しいと口調まで気にかけられなかったりする・慣れる必要以上に怖がることはない

Slide 22

Slide 22 text

「なぜ」を考える

Slide 23

Slide 23 text

<% if user.role == 'admin' %> 管理者 <% elsif user.role == 'advisor' %> アドバイザー <% elsif user.role == 'other' %> その他 <% end %>

Slide 24

Slide 24 text

「モデルにメソッドを作った方がいいよ」

Slide 25

Slide 25 text

def role_text case self.role when 'admin' '管理者' when 'advisor' 'アドバイザー' when 'mentor' 'メンター' else 'その他' end end

Slide 26

Slide 26 text

<%= user.role_text %>

Slide 27

Slide 27 text

<% if user.role == 'admin' %> 管理者 <% elsif user.role == 'advisor' %> アドバイザー <% elsif user.role == 'mentor' %> メンター <% end %>