【れびゅー】

レビュー とは?

💡 他人の目で品質を磨き上げるチェック工程
📌 このページのポイント
コードレビューのフロー コード作成 (開発者A) PR作成 レビュー依頼 レビュー (開発者B) 承認・マージ 本番反映 指摘があれば修正して再レビュー チェック: バグ / 可読性 / 設計 / セキュリティ / パフォーマンス
コードレビューのイメージ
ひよこ ひよこ

コードを書いたらすぐ本番に出していいの?

ペンギン先生 ペンギン先生

その前にレビューを通すのが基本だよ。他の人にコードを見てもらって問題がないか確認するんだ

ひよこ ひよこ

自分で確認するだけじゃダメなの?

ペンギン先生 ペンギン先生

自分だと見落としやすいバグや、もっと良い書き方に気づけないことが多いんだよね。別の視点が入ると品質がぐっと上がるよ

ひよこ ひよこ

レビューで指摘されるとへこみそうだな

ペンギン先生 ペンギン先生

指摘は攻撃じゃなくてチームで良いものを作るためのものだよ。レビューする側も勉強になるし、お互いに成長できる文化が大事だね

ひよこ ひよこ

コードレビューで気まずくならないコツってある?

ペンギン先生 ペンギン先生

指摘するときは「コード」に対してコメントして「人」を否定しないのがポイントだよ。「この書き方は〜の理由で○○にしたほうがいいかも」と提案スタイルにする。あと「LGTM(Looks Good To Me)」と良い点を褒めるのも大事だね。

ひよこ ひよこ

レビューってどのくらいの量を見ればいいの?

ペンギン先生 ペンギン先生

Googleの調査によると、1回のレビューで見るコードは200〜400行が最適で、それ以上だとレビューの質が急激に落ちるとされているよ。つまりPull Requestは小さく分割するのがベストプラクティス。大きなPRは「レビューする側の負担が大きい→後回しにされる→マージが遅れる」という悪循環を生むんだ。

ペンギン
まとめ:ざっくりこれだけ覚えればOK!
「レビュー」って出てきたら「成果物を他の人にチェックしてもらって品質を上げること」と思えればだいたいOK!
📖 おまけ:英語の意味
「Review」 = 見直す・再検討する
💬 re(再び)とview(見る)が組み合わさった言葉で、成果物をもう一度別の視点で確認するという意味がある
← 用語集にもどる