#97prog_ja 14 コードレビュー
このエントリーに出会うまでに実施していたコードレビューでは大した成果が出なかった。チームメンバーから今まで行っていたコードレビューのやり方に関して以下の指摘を受けた。
- プログラムの全体がわからない
- 1行を入念に読んでいて読み切れない
このエントリーがいうコードレビューの目的は
- コードの質を上げ、欠陥を減らす
- チーム全員に同じ知識を共有させる
- コーディングにおいて全員が守るガイドラインを確立させる
ということだ。でもこれだけでも先に挙げた問題を解決するには足りない。
コードレビューの前に今回コードレビューを行うモジュール、もしくは
テストコードの概要説明を行ったほうが良い。
もしオブジェクト指向で構築しているのであればコードレビュー対象のクラスと
関係を持っているクラスを含めクラス図にすると良い。
最後にこのエントリーでは以下のように締めくくっている。
コードレビューを成功させるために最も有効な方法は、レビューを楽しいものにすることです。
レビューで最も大事なのは人です。もしレビューが辛く退屈なものだったとしたら、
誰も進んでレビューに参加する気にはならないはずです。
できるだけくだけた雰囲気を心がけ、
主目的をメンバー内で知識を共有することに置くようにするといいでしょう。