2018-04-03

開発現場に学ぶ、円滑なコードレビューに必要な8つの手法～手段から準備、実施時期まで徹底解説～

コードレビューによって解決される問題とは？そして、実際にチームでコードレビューを実施する上で気をつけるべきこととは？ソニックガーデンの取締役プログラマー西見公宏さんが、コードレビューのポイントを、実践に基づき解説します。

開発現場に学ぶ、円滑なコードレビューに必要な8つの手法～手段から準備、実施時期まで徹底解説～

ITを活用して事業の課題を解決するサービス「納品のない受託開発」を提供する会社、ソニックガーデンの西見公宏（にしみ・まさひろ／@mah_lab）です。お客様の「バーチャルCTO」として、サービスの企画からシステムの開発・運用まで、日夜幅広く関わらせていただいております。

皆さんは普段、ソースコードをどのくらい読んでいるでしょうか？

普段からソフトウェア開発をしている人であれば、何か問題が起こったときの原因調査のために他の人が書いたコードを読んだり、はたまた自分の書いたコードを読んだりする機会は多いですよね？オープンソースのライブラリも当たり前のように使っているでしょうから、実装の詳細を確認するためにライブラリのソースコードを読む機会も多いと思います。

このように、ソフトウェアの開発をする上では、ソースコードを書く技術もさることながらソースコードを読む技術も当然求められるものです。

では、ソースコードを読む技術はどのようにして鍛えられるのでしょうか？オープンソースのライブラリのソースコードを片っ端から読むのもひとつの手ではありますが、個人的な意見では、コードレビューこそ、ソースコードを読む技術を鍛える絶好の機会だと考えています。「レビュー」というと、通常はベテランが品質の維持・向上のために行う活動を指すため、若手にレビューの機会が回ってくることは少ないかもしれません。

しかし、コードレビューによってどのような問題が解決されるか？を考えると、「コードレビューはベテランがチェックするのが一番効率が良い」というアプローチが間違っているのではないか？ と私は考えています。

本稿ではコードレビューによって解決される問題の例を確認した後、実際にチームでコードレビューを実施する上で気をつけるべきことを紹介します。コードレビューをしていない現場から、ベテランが一方的にコードレビューを実施している現場、または、すでに実施しているがイマイチ効果が出ていない現場まで役に立つ情報だと思いますので、ぜひご参考にしてみてください。