コードレビューとは?
コードレビューとは、ソフトウェア開発活動の中の一つで、色々な目的ながあります。
もっとも一般的な目的は、コードを自分の目で確認して不具合等が入り込んでいないかを確かめることです。
自分の描いたコードを他の開発者が確認したり、他の開発者が書いたコードを自分で確認したりします。
そのほかの目的として、他のメンバーのコードを理解して、自分のコードとの間で齟齬がないかを確認したり、問題がある部分の代替案を考えたりする場合もあります。1回のレビューでこれらすべてを目的にすることはあまりなく、1回目のレビューでは欠陥がないかを確かめ、2回目のレビューでは代替案がないかを確かめる、といった具合に使い分けるのが一般的です。
コードレビュー時に必要なスキル
レビューのスキルには大きく分けて以下の2つが存在します。
①問題を見つけるスキル
②見つけた問題をうまく指摘するスキル
どちからがかけていると、問題がわかっているものの相手になかなか意味が汲み取ってもらえず、問題からさらなる問題を引き起こす場合もあります。
またそれをちゃんと伝えるためのコミュニケーションコストがさらに発生してしまうこともあり得ます。
コードレビューの読み進め方
基本的には以下の2つになるかと思います。
①バグが潜んでいないか
②可読性があるか
1つ目は、そのままなのですが期待している処理通りに進むようなコードになっているのか?
2つ目は、将来コードを拡張する際に、コードを読む人を勘違いさせやすいコードです。コードを読む人にとって、紛らわしい書き方をしていないかどうかを確認するための読み進め方を紹介します。
基本的にはこれらのポイントを念頭に読み進めていくことが大事だと思います。